Make "Bytnerowicz" the default nfix_method for clm6 #2972
Conversation
| <paramfile phys="clm4_5" >lnd/clm2/paramdata/clm45_params.c241119.nc</paramfile> | ||
| <paramfile phys="clm6_0" >lnd/clm2/paramdata/ctsm60_params_nfix.c241119.nc</paramfile> | ||
| <paramfile phys="clm5_0" >lnd/clm2/paramdata/clm50_params_nfix.c241119.nc</paramfile> | ||
| <paramfile phys="clm4_5" >lnd/clm2/paramdata/clm45_params_nfix.c241119.nc</paramfile> |
There was a problem hiding this comment.
@wwieder please review this change.
There was a problem hiding this comment.
do we want to add nfix to the title of the parameter file? This seems unusual to me? Also, do we typically have the same default parameter list for each physics version? This is naive on my part.
There was a problem hiding this comment.
I agree with @wwieder I'd nix nfix in the name of the file. We make so many changes to paramfiles you can't keep adding the list of new things to the filenames.
There was a problem hiding this comment.
Also, do we typically have the same default parameter list for each physics version? This is naive on my part.
The values of the parameters are different for each physics version. Hence, we need to have a different parameter file for each physics version. In CESM3.0 we are likely to have two parameter files for clm6_0, one for coupled and one for CRUJRA.
Does that answer your question?
| @@ -1,2 +1,2 @@ | |||
| paramfile = '$DIN_LOC_ROOT/lnd/clm2/paramdata/ctsm60_ciso_cwd_hr_params.c241119.nc' | |||
| paramfile = '$DIN_LOC_ROOT/lnd/clm2/paramdata/ctsm60_ciso_cwd_hr_params_nfix.c241119.nc' | |||
There was a problem hiding this comment.
@wwieder pls review this change, also. For this one I had to generate the new file, so could you check my work given past snafus that we've had with paramfiles?
There was a problem hiding this comment.
sorry, what am I checking for with this file?
| @@ -1409,7 +1409,7 @@ attributes from the config_cache.xml file (with keys converted to upper-case). | |||
| ic_ymd="20000101" sim_year="2000" use_excess_ice=".true." | |||
| ic_tod="0" glc_nec="10" use_crop=".true." irrigate=".true." | |||
| phys="clm6_0" use_init_interp=".true." | |||
| >lnd/clm2/initdata_esmf/ctsm5.3/ctsm53n04ctsm52028_f09_g17_BgcCrop_exice_hist.clm60.r.2000-01-01.nc | |||
| >lnd/clm2/initdata_esmf/ctsm5.3/ctsm53019_f09_BNF_hist.clm2.r.2000-01-01-00000.nc | |||
There was a problem hiding this comment.
Questions regarding the finidat updates:
- I updated 1850 and 2000 for f09 only. Should I update ne30 and f19 with the same f09 file or wait until we have spun up ne30 and f19 simulations?
- For ic_ymd="19790101" should I go back and generate a 1979 file or not worry about it for now (e.g. because we will generate finidat files again before all is said and done)? Right now the closest restart file we have is 1978 from the new simulation.
- Anything I've forgotten?
There was a problem hiding this comment.
I'm not sure we'll end up creating ne30 and f19 initial conditions with this file, as Linnia is going to have new suggestions from the PPE soon. Instead, maybe we point to the f09 initial conditions and set init_interp = true for other resolutions?
|
derecho testing In aux_clm, expect DIFF for clm6 only, I think. |
Description of changes
Same as PR's title, plus update paramfile and finidat accordingly.
Definition of DONE checklist in the issue.
Specific notes
Contributors other than yourself, if any:
CTSM Issues Fixed (include github issue #):
Fixes #1339
Are answers expected to change (and if so in what way)?
Yes, greater than roundoff, same climate.
Any User Interface Changes (namelist or namelist defaults changes)?
See Description of changes above.
Does this create a need to change or add documentation? Did you do so?
Likely, but not done.
Testing performed, if any:
None, yet.