-
Notifications
You must be signed in to change notification settings - Fork 321
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add option for CRUJRA2024 add some compsets for Clm60 for it as well as a corresponding compset for Clm5 #2956
base: master
Are you sure you want to change the base?
Conversation
First attempt at testing: |
Second set of tests on derecho:
Checked the last two suites with these commands:
In aux_cdeps, this test fails here but also in vanilla ctsm5.3.021:
|
3rd round to test the "clm5" changes
|
TODOs
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@slevis-lmwg and I met and we went over the things to do and there's a conversation for them.
Since, we aren't doing the answer changing part here -- this could be put onto b4b-dev. What do you think about that @slevis-lmwg?
on izumi IN PROGRESS with the revisions only (did not update to ctsm5.3.024, yet) UPDATE |
derecho testing started before 0fad468 commit to my cdeps branch
derecho failures now resolved
The FUNIT test passes with vanilla 024; a subsequent commit fixes this failure. izumi testing started after last commit to my cdeps branch derecho additional testing started after the changes to subset_data |
Change choice of pressure in CLMU building energy model In the CLMU Building Energy Model (BEM), the choice of pressure for calculating indoor air density is changed from standard pressure to bottom layer of atmosphere pressure of the corresponding grid cell. Answers change. PR ESCOMP#2758 Issue ESCOMP#2755
@jedwards4b I get this unexpected FAIL that I have attributed (after testing) to the share1.1.8 update: Compare the first (failing one) against the second (passing baseline) in /glade/derecho/scratch/slevis: In run/funit.log this message may be the explanation: Shall I let you resolve this? |
python/ctsm/subset_data.py
Outdated
@@ -571,7 +571,11 @@ def setup_files(args, defaults, cesmroot): | |||
abort("inputdata directory does not exist") | |||
|
|||
# DATM data | |||
datm_type = "datm_gswp3" | |||
# TODO issue #1895: allow datm_crujra, which also affects |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change TODO to say "make this option available at the command line"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other TODOs
- Update .gitmodules when cdeps tag is ready
- Finish ChangeLog/Sum
@slevis-lmwg It looks like this unit test is not currently linking esmf. I suggest you check with @billsacks |
@@ -202,7 +227,7 @@ | |||
</compset> | |||
<compset> | |||
<alias>I1850Clm60BgcCropCmip6</alias> | |||
<lname>1850_DATM%GSWP3v1_CLM60%BGC-CROP-CMIP6DECK_SICE_SOCN_MOSART_SGLC_SWAV</lname> | |||
<lname>1850_DATM%CRUJRA2024_CLM60%BGC-CROP-CMIP6DECK_SICE_SOCN_MOSART_SGLC_SWAV</lname> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description of changes
Change the default datm input from GSWP3v1 to CRUJRA2024 for Clm6.
Add a CRUJRA2024 compset for clm5.
See #1895 for definition of DONE.
Specific notes
Contributors other than yourself (I hope I didn't forget any):
@wwieder @adrifoster @swensosc @djk2120 @ekluzek @olyson
CTSM Issues Fixed (include github issue #):
Resolves #1895
Are answers expected to change (and if so in what way)?
No. We will make a separate answer-changing PR to address Clm6 tests that do not specify the datm input, so that they get CRUJRA2024 instead of GSWP3v1.
Any User Interface Changes (namelist or namelist defaults changes)?
Adding Crujra to compset names.
Adding 2 new tests that explicitly include Crujra in their compset names.
Clm60 tests that do not include a datm option in their compset will default to Crujra in a later PR.
Does this create a need to change or add documentation? Did you do so?
It does, and I did not, yet.
Testing performed, if any:
See relevant posts below.