Opened 6 years ago

Closed 3 years ago

#1413 closed Bug (wontfix)

bug in cumulate surface flux in bdyvol ?

Reported by: mathiot Owned by: jchanut
Priority: low Milestone:
Component: OCE Version: trunk
Severity: Keywords: BDY
Cc:

Description

In bdyvol.F90, NEMO calculates the cumulate surface Flux z_cflxemp (m3/s) over all the domain:

z_cflxemp = SUM ( ( emp(:,:)-rnf(:,:) ) * bdytmask(:,:) * e1t(:,:) * e2t(:,:) ) / rau0
IF( lk_mpp )   CALL mpp_sum( z_cflxemp )     ! sum over the global domain

I think it should be :

USE lib_fortran
...
z_cflxemp = glob_sum( ( ( emp(:,:)-rnf(:,:) ) * bdytmask(:,:) * e1t(:,:) * e2t(:,:) )  / rau0 )

to avoid the halos as in sbcfwb.F90.

I think also snwice_fmass(:,:) should be added to z_cflxemp as in sbcfwb.F90.

Commit History (0)

(No commits)

Change History (3)

comment:1 Changed 6 years ago by mathiot

  • Owner changed from NEMO team to jchanut

In case in vvl option, surface is not bdy surface is not updated at each time step. Computation of surface should be move or copy in bdyvol.F90

comment:2 Changed 4 years ago by nicolasmartin

  • Keywords BDY added; bdy removed

comment:3 Changed 3 years ago by clevy

  • Resolution set to wontfix
  • Status changed from new to closed

Suppressing tickets opened on the trunk since more than 2 years… cleaning started!

Note: See TracTickets for help on using tickets.