Add FATES tree recruitment scheme tests #3757
Add FATES tree recruitment scheme tests #3757glemieux wants to merge 6 commits intoESCOMP:b4b-devfrom
Conversation
|
Running the TRS test mods on |
|
The Alternatively, we could attempt to fix the issue and turn this PR into an API update associated with whichever FATES tag fixes the TRS issue. |
|
TO DO:
|
67c3762 to
c386a0e
Compare
|
Running |
|
Testing on Testing on @ekluzek @slevis-lmwg did you see these fail during the testing for the last Results ( |
I confirmed that the RX and SUBSETDATA tests passed in ctsm5.4.021 by looking at Teststatus here: |
|
Looking at the cesm.log error message it appears to be an instance of #3590 as I'm seeing the same thing as mentioned in #2930 (comment). I'll try rerunning these without defining a custom test-id. |
Looks like this is the cause of the issue I was seeing. Re-running these tests individually with the default test-id resulted in all three previous run failures passing b4b. |
Description of changes
This simple pull request adds tests for the two recruitment namelist options for the FATES tree recruitment scheme. There are two different modes for turning on the scheme, which now have a test. No FATES parameter file modifications are necessary to activate the test.
Resolves #2760.
Note that there is an known FATES mass balance issue, NGEET/fates#1246, that might be triggered by the
FatesColdTRStestmod. This will be added to the expected fail list if it comes up during testing.Specific notes
Contributors other than yourself, if any:
CTSM Issues Fixed (include github issue #): #2760
Are answers expected to change (and if so in what way)? No
Any User Interface Changes (namelist or namelist defaults changes)? No
Does this create a need to change or add documentation? Did you do so? No
Testing performed, if any:
(List what testing you did to show your changes worked as expected)
(This can be manual testing or running of the different test suites)
(Documentation on system testing is here: https://github.com/ESCOMP/ctsm/wiki/System-Testing-Guide)
(aux_clm on derecho for intel/gnu and izumi for intel/gnu/nag/nvhpc is the standard for tags on master)
NOTE: Be sure to check your coding style against the standard
(https://github.com/ESCOMP/ctsm/wiki/CTSM-coding-guidelines) and review
the list of common problems to watch out for
(https://github.com/ESCOMP/CTSM/wiki/List-of-common-problems).