Skip to content
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

Push EWOrg/cime to cime6.0.182 #7

Merged
merged 119 commits into from
Feb 4, 2024
Merged

Conversation

gdicker1
Copy link

Fetch upstream changes to allow MPAS-A to easily pick up GPU flags in the builds.

NOTE: This depends on #6 being merged first

jasonb5 and others added 30 commits August 28, 2023 18:32
Fix typo retart/restart

Just a minor typo fix.
One of the code paths for computing submit args was assuming
a whitespace separator was needed, which is not right when the flag
ends with '=' or ':'. This PR adds a new function that encapsulates
this concept and calls it in the appropriate places in env_batch.
env_batch: Fix flag/val separator

One of the code paths for computing submit args was assuming a whitespace separator was needed, which is not right when the flag ends with '=' or ':'. This PR adds a new function that encapsulates this concept and calls it in the appropriate places in env_batch.

Test suite: scripts-regression-tests
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)?:
This will be used to mark suites as performance test suites.
Moves check_input_data to SystemTestsCommon when running a test
…operty

Add new 'perf' test property

This will be used to mark suites as performance test suites.

I think this is the best way to approach the issue of performance testing. It will allow us to mark a handful of tests in our common test suites as performance tests so we can piggyback on the current testing jobs. The idea would be to select a few tests in the integration suite, pull them into their own suite, mark them as perf=True, then have the original suite inherit the new suite.

Test suite: doctests
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes [CIME Github issue #]

User interface changes?:

Update gh-pages html (Y/N)?:

Removes .* between hist_file_extension and suffix in get_all_hist_files.
This would cause ERIO test to fail by not finding the correct files to compare
due to the extra text allowed between the extension and suffix. If there is
expected text between the extension and suffix this should be defined in
hist_file_extension.

Test suite: pytest -vvv
Test baseline: n/a
Test namelist changes: n/a
Test status: n/a

Fixes n/a
User interface changes?: n/a
Update gh-pages html (Y/N)?: N
PFS tests should always save timing

Test suite: pre-commit
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)?:
This implementation makes some assumptions about the context
of /.../Macros.cmake, so I am only turning this on for
E3SM.
jedwards4b and others added 26 commits October 20, 2023 10:56
reduce the number of times env_batch is rewriten
Fixes failing tests failing due to E3SM cmake update.

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)?:
Fix bless test results

Fixes requiring --*-only flag
Fixes logic for default behavior of performing all bless types or only a single type
Adds exclude feature

Test suite: pytest -vvv CIME/tests/test_unit_*
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)?:
More bless_test_results fixes

Adds testing for bless_test_results function
Fixes --pes-file argument variable name
Fixes duplicate -g for create_test when blessing namelists
Fixes checking if memory/throughput phase failed
Fixes variable names causing blessing memory/throughput to be skipped
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)?:
Yet another btr update

Reverts to original behavior, bless_test_results with no flags will bless hist and nml
Adds performance flags --bless-tput, --bless-mem, and --bles-perf (both tput and mem)
Test suite: pytest -vvv CIME/tests/test_unit*
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)?:
Add GPU flags to MPAS dycore through the CIME infrastructure
Update EarthWorksOrg/cime with upstream work from
'ESCOMP/CESM/cesm2_3_alpha17a' version.
Get GPU flags for the MPAS builds from upstream.
@gdicker1
Copy link
Author

NOTE: This depends on #6 being merged first

@gdicker1 gdicker1 merged commit e8ec229 into ew-develop Feb 4, 2024
@gdicker1 gdicker1 deleted the update/post_alpha17a/cime branch April 23, 2024 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants