Opened 7 years ago
Closed 6 years ago
#1945 closed Defect (fixed)
Require e3t filtered through depth not just surface in tranxt/dynnxt for rivers distributed through depth
Reported by: | deazer | Owned by: | jamesharle |
---|---|---|---|
Priority: | low | Milestone: | |
Component: | TRA | Version: | trunk |
Severity: | minor | Keywords: | rivers time |
Cc: |
Description
Context
Bug in Time Filtering with respect to Rivers specified through water column not just at the surface.
Problem code is in subroutine tra_nxt_vvl: here e3t only updated at surface:
301 IF( jk == mikt(ji,jj) ) THEN ! first level 302 ze3t_f = ze3t_f - zfact2 * ( (emp_b(ji,jj) - emp(ji,jj) ) & '''303 & - (rnf_b(ji,jj) - rnf(ji,jj) ) &''' 304 & + (fwfisf_b(ji,jj) - fwfisf(ji,jj)) ) 305 ztc_f = ztc_f - zfact1 * ( psbc_tc(ji,jj,jn) - psbc_tc_b(ji,jj,jn) ) 306 ENDIF
Analysis
The code here only works if the rivers are specified at the surface. There needs to be a case
to deal with rivers specified through depth. Otherwise the tracer content will be inconsistent with the level volume.
Fix
Create cases based on if ln_rnf_depth is true or false. If it is false then the existing code can be used but
if it is true then the rnf needs to be distributed through all the levels for which the river is specified to enter into the ocean.
jk <= nk_rnf(ji,jj)
possible similar fix also required in dynnxt where e3t is updated?
Commit History (0)
(No commits)
Change History (4)
comment:1 Changed 7 years ago by clevy
- Owner changed from deazer to jamesharle
- Severity set to minor
- Status changed from new to assigned
- Type changed from Bug to Defect
comment:2 Changed 7 years ago by nemo
- Keywords filter removed
comment:3 Changed 7 years ago by nemo
- Keywords depth removed
comment:4 Changed 6 years ago by clem
- Resolution set to fixed
- Status changed from assigned to closed
done