forked from ESMCI/cime
-
Notifications
You must be signed in to change notification settings - Fork 17
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
update to cime6.0.222 #70
Merged
mvertens
merged 204 commits into
NorESMhub:noresm
from
mvertens:feature/update_to_cime6.0.222
Apr 1, 2024
Merged
update to cime6.0.222 #70
mvertens
merged 204 commits into
NorESMhub:noresm
from
mvertens:feature/update_to_cime6.0.222
Apr 1, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This update is based on existing code for cism but uses a python script rather than a shell script. The update is needed for use with GEOS-Chem chemistry so that run-time configuration files can be copied to the case directory upon case setup. Signed-off-by: Lizzie Lundgren <[email protected]>
This avoids a duplicate string formatting warning when using a linter. Signed-off-by: Lizzie Lundgren <[email protected]>
Signed-off-by: Lizzie Lundgren <[email protected]>
Signed-off-by: Lizzie Lundgren <[email protected]>
Add call in case_setup to run CAM python script
Fixes separating std and performance blessing When blessing performance diffs, the default hist and namelists would be run as well. This PR separates these, there is now two distinct blessing paths; standard (hist and namelists) and performance. Test suite: pytest CIME/tests/test_unit* Test baseline: n/a Test namelist changes: n/a Test status: n/a Fixes n/a User interface changes?: N Update gh-pages html (Y/N)?: N
- add more details to TestStatus logs for base, comp, threshold - append (not overwrite) to perf-baselines to keep history - record sha and date of new blesses
…chanism to checkout
…o remove_cprnc
Test using dynamic container Fix workflow to always build containers (utilizes cache for speedup) Testing always uses newest container Fix recursion error from case.preview call for E3SM Test suite: Test baseline: Test namelist changes: Test status: [bit for bit, roundoff, climate changing] Fixes [CIME Github issue #] User interface changes? Update gh-pages html (Y/N)?:
Standalone component
…c_tests E3SMv3 atmosphere model default configuration is changed to use 80 vertical levels. The atm initial conditions for PGN and TSC with ne4 grid need to be updated with those from an EAMv3-based F2010 run. The land initial conditions are also replaced from the same run. [NBFB] for PGN and TSC tests. Baseline comparison will have an exception as the number of vertical levels in atm history output files differs Test suite: chrysalis_atm_nbfb Test baseline: chrys/intel/master Test namelist changes: atm and lnd ICs (ncdata and finidat) Test status: [bit for bit, roundoff, climate changing]: climate changing for these specific tests.
We don't want test cases downloading data while on a compute node if it can be avoided. It's harmless to call check_all_input_data twice, so let's have the case.submit call handle the 99% of cases where the main case will download all the inputs needed for the case/test.
This PR fixes finding termination text in logs. Before this fix an S compset was considered a standalone component and for E3SM would not check for correct termination text. Now the log is checked for all possible termination text. Test suite: pytest CIME/tests/test* Test baseline: n/a Test namelist changes: n/a Test status: n/a Fixes n/a User interface changes?: Update gh-pages html (Y/N)?:
…g_setup case.submit: Always try to download input data We don't want test cases downloading data while on a compute node if it can be avoided. It's harmless to call check_all_input_data twice, so let's have the case.submit call handle the 99% of cases where the main case will download all the inputs needed for the case/test. We keep the download calls in system_tests_common.py just in case case2 of a compare-two test needs more inputs. Test suite: Test baseline: Test namelist changes: Test status: [bit for bit, roundoff, climate changing] Fixes [CIME Github issue #] User interface changes?: Update gh-pages html (Y/N)?:
Fixes testing workflow for prs from forked repos. Test suite: Test baseline: Test namelist changes: Test status: [bit for bit, roundoff, climate changing] Fixes [CIME Github issue #] User interface changes?: Update gh-pages html (Y/N)?:
gold2718
approved these changes
Mar 10, 2024
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.
Seems pretty straightforward although I wonder about the (not new) big commented block (TO DO) in config_files.xml starting at line 510.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This update to cime6.0.222 is needed in order to update NorESMhub CMEPS to newer versions in ESCOMP - which in turn is needed in order to create a data glc component.
Testing: verified that the following test was bfb:
SMS_D_Ln9_P256.f19_f19_mtn14.F2000climo.betzy_intel.cam-outfrq9s
Fixes: None
User interface changes?: None
Update gh-pages html (Y/N)?: N