You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
DATM CPLHIST test needs to be fixed and then restored #408
I prematurely merged #405 without realizing that the DATM CPLHIST test (SMS_Ld5.f10_f10_mt232.1850_DATM%CPLHIST_SLND_SICE_SOCN_SROF_SGLC_SWAV_SESP.derecho_intel.datm-cplhist) is currently failing.
The problem is that a dtlimit is exceeded due to too big of a discrepancy between dtmin and dtmax:
I'm going to temporarily remove this test from the test list until this problem can be resolved. @ekluzek whenever you have time to resolve it (no rush!!!), it can be reinstated.
I could be wrong, but my impression is that the data set pointed to here was just for software testing purposes, so this shouldn't hold anything up... but I haven't been in the loop with the plan for the land model spinup so I could be wrong on that.
In ctsm5.4.040 (our latest master tag) I updated from cdeps1.0.93 to cdeps1.0.94 and ran the test: PASS (expected)
I updated from cdeps1.0.94 to cdeps1.0.95 and ran the test: FAIL with the error reported above (expected) AND TestStatus.log also includes a long list of warnings like this one:
I reverted the change from Erik's commit 0bdb38e and ran the test: PASS
@billsacks@ekluzek@wwieder I interpret this to mean that Erik's commit that looks for Cecile's cplhist data in /inputdata fails to find it there. I see two options, and will wait for feedback:
Revert Erik's commit and point to Cecile's /scratch directory again until the data gets copied
It seems like a simple enough issue to address, but I'll let more SE-minded collaborators decide how to maintain this long-term.
Let's shelve this until @ekluzek and @billsacks are back from leave to address this.
Browsing the list of missing files suggests that the model does not find 12 months of data (present in Cecile's /scratch) and finds only 1 month in /inputdata. For the full list of missing files, see /glade/derecho/scratch/slevis/SMS_Ld5.f10_f10_mt232.1850_DATM%CPLHIST_SLND_SICE_SOCN_SROF_SGLC_SWAV_SESP.derecho_intel.datm-cplhist.20260521_154000_oiao8p/TestStatus.log
Activity
billsacks commentedon May 21, 2026
This will mean backing out the changes in #409 .
Temporarily remove datm cplhist test
Merge pull request #409 from billsacks/remove_datm_cplhist_test
wwieder commentedon May 21, 2026
Does this mean we should make a cesm alpha09 tag that uses #409, or should we wait until this issue is resolved before making the alpha tag?
Second, Is this something that @slevis-lmwg could help with, or is it lower priority and can wait for @ekluzek to return?
billsacks commentedon May 21, 2026
I could be wrong, but my impression is that the data set pointed to here was just for software testing purposes, so this shouldn't hold anything up... but I haven't been in the loop with the plan for the land model spinup so I could be wrong on that.
slevis-lmwg commentedon May 21, 2026
@wwieder requested that investigate this:
@billsacks @ekluzek @wwieder I interpret this to mean that Erik's commit that looks for Cecile's cplhist data in /inputdata fails to find it there. I see two options, and will wait for feedback:
wwieder commentedon May 21, 2026
Thanks for looking into this, @slevis-lmwg.
It seems like a simple enough issue to address, but I'll let more SE-minded collaborators decide how to maintain this long-term.
Let's shelve this until @ekluzek and @billsacks are back from leave to address this.
slevis-lmwg commentedon May 21, 2026
Browsing the list of missing files suggests that the model does not find 12 months of data (present in Cecile's /scratch) and finds only 1 month in /inputdata. For the full list of missing files, see
/glade/derecho/scratch/slevis/SMS_Ld5.f10_f10_mt232.1850_DATM%CPLHIST_SLND_SICE_SOCN_SROF_SGLC_SWAV_SESP.derecho_intel.datm-cplhist.20260521_154000_oiao8p/TestStatus.logslevis-lmwg commentedon May 22, 2026
I will mark "done" in my Sprint accounting. Can always change later.
7 remaining items
Increase dtlimit for the CPLHIST options so that the CPLHIST test wil…