Opened 4 years ago
Closed 3 years ago
#2642 closed Bug (fixed)
possible bugs: salflx output unit and ln_rnf_icb has no effect
Reported by: | mathiot | Owned by: | systeam |
---|---|---|---|
Priority: | low | Milestone: | |
Component: | ICB | Version: | v4.0.* |
Severity: | minor | Keywords: | output, ICB, |
Cc: |
Description
Context
- unit of the saltflx variable is in 1e-3/m2/s in field_def_oce.xml. I wonder if it should be instead in 1e-3.kg/m2/s as specified in the comments of ice.F90 for the variable sfx ?
- I am not convince the icb climatology is used. icb climatology is read in sbcrnf in fwfisf, output as iceberg_cea or hflx_icb_cea (latent heat) but I am unable to find where this is added to rnf or emp in a forced simulation. If cpl, there is in sbccpl a line rnf = rnf + fwfisf. I did a try with ORCA2 and I have the same result whatever the value of ln_rnf_icb.
- more general question/comments on ln_rnf_icb: I found confusing that the definition of the iceberg climatology is done in the runoff namelist instead of the icb namelist. A master flag ln_icb with ln_icb_clim for the use of the climatology and ln_icb_lagr for the use of the icb lagragian model is maybe more logical. This point could be discussed at the same time of the ticket #2467.
Analysis
It affect NEMO 4 and I didn't check the trunk.
Fix
point 1: If issue on saltflx confirmed, change in field_def_oce.xml is required.
point 2: add fwficb to rnf and add fwficb * rLfus to the runoff heat content flux.
Commit History (3)
Changeset | Author | Time | ChangeLog |
---|---|---|---|
14773 | clem | 2021-04-30T12:23:51+02:00 | trunk: correct units of some icb diags (also related to ticket #2642) |
14718 | clem | 2021-04-16T11:43:50+02:00 | trunk: correctly handle diagnostics of mass, salt and heat budgets (see ticket #2652). And fix Pierre ticket #2642 |
14717 | clem | 2021-04-16T11:42:56+02:00 | 4.0-HEAD: correctly handle diagnostics of mass, salt and heat budgets (see ticket #2652). And fix Pierre ticket #2642 |
Change History (5)
comment:1 Changed 4 years ago by clem
comment:2 Changed 4 years ago by clem
In 14717:
comment:3 Changed 4 years ago by clem
In 14718:
comment:4 Changed 4 years ago by clem
In 14773:
comment:5 Changed 3 years ago by mathiot
- Resolution set to fixed
- Status changed from new to closed
Thank you Clem for the fix in NEMO
The remaining general point mentioned has been moved to a specific ticket: #2740
I am currently checking this stuff for proper budgets and I agree with Pierre.
=> These can be fixed relatively painlessly as Pierre suggested with the difference that I would suggest to put the latent heat of icebergs into qns as for coupled simulations (and not in hflx_rnf). Only sensible heat should be in hflx_rnf.
But I also have a couple of head scratching problems: