Opened 9 years ago
Closed 7 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 9 years ago by mathiot
- Owner changed from NEMO team to jchanut
comment:2 Changed 8 years ago by nicolasmartin
- Keywords BDY added; bdy removed
comment:3 Changed 7 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.
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