-
Notifications
You must be signed in to change notification settings - Fork 251
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 datm-mom6-cice6 and coupled-model tests on Gaea. #406
Conversation
Is there any interest in having WW3 ported to Gaea so that the wave tests can also be run there? |
@JessicaMeixner-NOAA Yes. We can't run tests with ww3 because the ww3 gaea porting changes are not available. Please let us know if ww3 code changes are ready. |
The new baselines have been added to the develop-20210128 baseline on Gaea, right? |
I don't see them there. Bin, if you tell me the location I can copy them over - assuming this goes in as the next PR or at least that there is no other PR with baseline changes in between. |
@climbfuji The baseline location is /lustre/f2/scratch/ncep/Bin.Li/RT/NEMSfv3gfs/develop-20210128/INTEL. |
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.
This looks good.
Is this the next PR to go in? |
It was not intended to be next but I haven't figured out why the cpld_2thread test is not passing w/ the updated FMS. That is the pending commit I believe. The CMEPS update scheduled for 2/5 will not change baselines. Bin Liu told me he wanted to go ahead w/ the CMEPS commit and that their required NEMS commit for HAFS would come later. |
Added to PR #401 |
The SRW_WE2E_COMPREHENSIVE_TESTS variable was incorrectly set by a sh block that proceeded the call to srw_test.sh. This issue was hidden because Jenkins sets the parameters to environment variables, which are accessible from sh blocks. However, the parameters do not seem to be set as environment variables the first time a multi-branch pipeline is initialized. This resulted in an unbound variable error when the SRW_WE2E_COMPREHENSIVE_TESTS variable was accessed by the srw_test.sh script. This update sets SRW_WE2E_COMPREHENSIVE_TESTS in the same sh block as the srw_test.sh script call, while ensuring the WORKSPACE variable isn't evaluated until the sh block.
Description
New baseline is created on Gaea for datm-mom6-cice6 and the coupled model. This PR will not change answers for existing tests.
Issue(s) addressed
Testing
What compilers / HPCs was it tested with?
Intel compiler/Gaea
Are the changes covered by regression tests?
Yes.
Dependencies
No.