Skip to content
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

Bring back clm5-cam6 init_interp_attributes for ne0CONUS grid #2901

Open
wants to merge 1 commit into
base: cesm3_0_beta04_changes
Choose a base branch
from

Conversation

slevis-lmwg
Copy link
Contributor

Description of changes

Same as title, plus bring back the corresponding tests in the ctsm_sci test-suite.

Specific notes

Contributors other than yourself, if any:
@ekluzek

CTSM Issues Fixed (include github issue #):
Resolves #2544

Are answers expected to change (and if so in what way)?
No

Any User Interface Changes (namelist or namelist defaults changes)?
Added clm5-cam6 init_interp_attributes to namelist_defaults_ctsm.xml

Does this create a need to change or add documentation? Did you do so?
No

Testing performed, if any:
As posted in #2544 the next four tests PASS with this PR's code mods:

SMS_Ln9.ne0POLARCAPne30x4_ne0POLARCAPne30x4_mt12.IHistClm50Sp.derecho_intel.clm-clm50cam6LndTuningMode_1979Start--clm-nofireemis
SMS_Ln9.ne0CONUSne30x8_ne0CONUSne30x8_mt12.IHistClm50Sp.derecho_intel.clm-clm50cam6LndTuningMode_2013Start--clm-nofireemis
SMS_Ln9.ne0ARCTICne30x4_ne0ARCTICne30x4_mt12.IHistClm50Sp.derecho_intel.clm-clm50cam6LndTuningMode_1979Start--clm-nofireemis
SMS_Ln9.ne0ARCTICGRISne30x8_ne0ARCTICGRISne30x8_mt12.IHistClm50Sp.derecho_intel.clm-clm50cam6LndTuningMode_1979Start--clm-nofireemis

@slevis-lmwg slevis-lmwg self-assigned this Dec 5, 2024
@@ -2084,6 +2084,16 @@
<option name="comment" >Also --nofireemis because this is a SP compset</option>
</options>
</test>
<test name="SMS_Ln9" grid="ne0POLARCAPne30x4_ne0POLARCAPne30x4_mt12" compset="IHistClm50Sp" testmods="clm/clm50cam6LndTuningMode_1979Start--clm/nofireemis">
<machines>
<machine name="derecho" compiler="intel" category="ctsm_sci"/>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ekluzek did you prefer to make these aux_clm tests instead of ctsm_sci?

@slevis-lmwg
Copy link
Contributor Author

slevis-lmwg commented Dec 5, 2024

PASS ./build-namelist_test.pl

@ekluzek ekluzek added this to the cesm3_0_beta06 milestone Dec 5, 2024
@ekluzek ekluzek self-requested a review December 5, 2024 17:49
@ekluzek ekluzek added bfb bit-for-bit testing additions or changes to tests labels Dec 5, 2024
@slevis-lmwg slevis-lmwg added the PR status: awaiting review Work on this PR is paused while waiting for review. label Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bfb bit-for-bit PR status: awaiting review Work on this PR is paused while waiting for review. testing additions or changes to tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ne0ARCTICne30x4 grid transient failure Failing ne0CONUSne30x8_ne0CONUSne30x8_mt12 in CESM testing
2 participants