ctsm5.4.055: CDEPS/CMEPS updates for cplhist and 1850 ndep from DATM - #4180
ctsm5.4.055: CDEPS/CMEPS updates for cplhist and 1850 ndep from DATM#4180ekluzek wants to merge 22 commits into
Conversation
…T case in the cplhist test mod
Merge b4b-dev 20260817 PR ESCOMP#4177
…cing set by DATM, and make the matrix and non-matrix version both have Crop on
|
aux_cdeps testing is looking as expected. aux_clm testing showed problems with these: which has to do with the new 1850 CO2 file in CDEPS |
Resolve problems with running PLUMBER2 sites Not b4b for PLUMBER2 sites only. Details in PR ESCOMP#4092 Conflicts: .gitmodules
|
Bringing in the updated CMEPS I see that share also has to be updated, and #4182 shows that it's safe to do that. |
ekluzek
left a comment
There was a problem hiding this comment.
A couple comments to myself.
| <default_value>FALSE</default_value> | ||
| <values> | ||
| <value compset="_CAM" >TRUE</value> | ||
| <value compset="_DATM%CPLHIST" >TRUE</value> |
There was a problem hiding this comment.
Hmmm. Should this just be for 1850 and with CLM60 or all the time? We assume CPLHIST data in CDEPS come from recent cases likely run with CLM60, so this is probably right...
| <value compset="_BGC%BDRD" >diagnostic</value> | ||
| <value compset="_BGC%BPRP" >prognostic</value> | ||
| <value compset="HIST.*_DATM" >diagnostic</value> | ||
| <value compset="1850.*_DATM%CPLHIST" >diagnostic</value> |
There was a problem hiding this comment.
The CO2 settings need to be consistent in CLM, DATM and CMEPS. There should be some error checking to check on that. There are various problems that happen if they aren't.
|
OK, I have changes to make in CDEPS from Bill, so will be working on that. |
|
Given that this PR is almost ready (except for #4183) and #4183 isn't really required for our CPL_HIST spinup for CESM3, I wonder if this tag could be merged without the CO2 checks to enable a quicker turn around for CESM3 spinup |
I think that's a good idea. I can do that as a seperate thing, and maybe just do it on b4b-dev. The holdup for this right now isn't that so much, as getting the CDEPS PR to pass review. There's a tricky thing I ran into there requested on the review, that I'll give an update on there. In the meantime I'm going to merge the submodule update first, since this updates a bunch of submodules and may also need a cime update. It's good to do as a previous step to show that the submodule update is b4b to get ready for just updating the submodules for the changes needed here. |
…nd NorESM CPLHIST for DATM
Update submodules to the latest needed for cesm3_0_alpha10a Update submodules: ccs_config, cime, cdeps, cmeps, share Current latest submodules that will be coming in cesm3_0_beta10 cesm3_0_beta09 had: cime6.4.1, share1.0.21, cdeps1.0.101, cmeps1.1.54 Also fix the two multi-instance tests that were failing, by making sure DGLC couples less times than ROF. And add new ctsm_release testlist which runs every test. And make sure the list of tests that should be subsets of aux_clm or ctsm_sci are so (as well as fates, and crop_cal). And bring in a script that checks the testlist for these features and will also fix them. And bring that script into a github workflow so the testlist is checked for basic requirements. Conflicts: .git-blame-ignore-revs .gitmodules cime components/cmeps
Description of changes
Update CDEPS with changes for CPLHIST mode. Turn on CLM_NDEP_FROM_CPL for CPLHIST mode. change cplhist test mod to use a different case. This should have everything needed to start CPLHIST spinups out of the box.
Also for 1850 with CLM60 use ndep from DATM
Specific notes
Contributors other than yourself, if any:
CTSM issues resolved or otherwise addressed, if any:
If answers are expected to change, describe (delete this line otherwise): Yes, just for CPLHIST mode
Any user interface changes (namelist or namelist defaults changes)?
Testing planned or performed, if any:
Requirements before merge: