Repository navigation
ctsm5.4.063: Move CTSM tests from t232 to t233 - #4223
Conversation
|
Blocked by ESMCI/ccs_config_cesm#305 and subsequent CTSM .gitmodules update in ESCOMP master #4232. |
|
@ekluzek only the LII2 tests raise a question in this PR. In particular, these tests require special handling to pass COMPARE_base_no_interp. My preference would be to return these to t232 and resolve later AND/OR move these tests to a grid less likely to change any time soon. What do you think? |
There was a problem hiding this comment.
Hey @slevis-lmwg this is great.
I did a "git grep t232" and noticed a few more things to handle...
cime_config/testdefs/ExpectedTestFails.xml:
cime_config/testdefs/testlist_clm.xml:
doc/source/users_guide/setting-up-and-running-a-case/customizing-the-clm-namelist.rst:.. I copied the following file from the baseline for ctsm5.4.047 for the SMS_D_Ld5_PS.f09_f09_mt232.I1850Clm60BgcCrop.derecho_gnu.clm-f09_ObscureStreamOpts test.
On the first two you might as well remove both of those which will complete #4238. The last one might not need anything done with it, but thought it would be good to look at.
|
|
ctsm5.4.063: Move CTSM tests from t232 to t233 Updating testlist_clm.xml, ExpectedTestFails.xml, config_compsets.xml, and documentation that needs updating. NB: Moving tests from t232 to t233 but postpone the change for LII2 tests (issue ESCOMP#4245). Changes answers relative to the baseline, only for cases with grids that changed to the t233 grid, and only due to the grid update. PR ESCOMP#4223 Resolves issue ESCOMP#4238 by removing NorESM test Contributes to ESCOMP#2540 Move testing from POP to MOM masks slevis resolved conflicts: cime_config/testdefs/testlist_clm.xml
ctsm5.4.063: Move CTSM tests from t232 to t233 Updating testlist_clm.xml, ExpectedTestFails.xml, config_compsets.xml, and documentation that needs updating. NB: Moving tests from t232 to t233 but postpone the change for LII2 tests (issue ESCOMP#4245). Changes answers relative to the baseline, only for cases with grids that changed to the t233 grid, and only due to the grid update. PR ESCOMP#4223 Resolves issue ESCOMP#4238 by removing NorESM test Contributes to ESCOMP#2540 Move testing from POP to MOM masks slevis resolved conflicts: bld/unit_testers/build-namelist_test.pl
ctsm5.4.063: Move CTSM tests from t232 to t233 Updating testlist_clm.xml, ExpectedTestFails.xml, config_compsets.xml, and documentation that needs updating. NB: Moving tests from t232 to t233 but postpone the change for LII2 tests (issue ESCOMP#4245). Changes answers relative to the baseline, only for cases with grids that changed to the t233 grid, and only due to the grid update. PR ESCOMP#4223 Resolves issue ESCOMP#4238 by removing NorESM test Contributes to ESCOMP#2540 Move testing from POP to MOM masks
ctsm5.4.063: Move CTSM tests from t232 to t233 Updating testlist_clm.xml, ExpectedTestFails.xml, config_compsets.xml, and documentation that needs updating. NB: Moving tests from t232 to t233 but postpone the change for LII2 tests (issue #4245). Changes answers relative to the baseline, only for cases with grids that changed to the t233 grid, and only due to the grid update. PR #4223 Resolves issue #4238 by removing NorESM test Contributes to #2540 Move testing from POP to MOM masks
Description of changes
Updating
testlist_clm.xml
ExpectedTestFails.xml
config_compsets.xml
.rst and README files that need the update
RTM, MOSART, and mizuRoute testlists do not include references to 232
Specific notes
Contributors other than yourself, if any:
CTSM issues resolved or otherwise addressed, if any:
Contributes to #2540
Resolves #4238
Testing planned or performed, if any:
./run_sys_tests -s aux_clm -c ctsm5.4.058 --skip-generatewith only ccs_config updated to tag 92 (additional submodules are getting updated on master)LII2FINIDATAREAS_D_P256x2_Ld1.f09_t233.I1850Clm60BgcCrop.derecho_intel.clm-defaulttests require special handling to pass COMPARE_base_no_interp. Return these to t232 to resolve when we are ready to finalize ICs. Open issue and link to ICs issue.Requirements before merge: