Github Action script to run preview_namelists #305
Closed
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
Fails in python 3.9 for G and B compsets because CICE's buildnml relies on glob.glob API introduced in 3.10. Fails all 3.12 and 3.13 tests because distutils has been removed from the standard library.
jedwards4b
requested changes
Nov 22, 2024
There was a problem hiding this comment.
@mnlevy1981 I believe that if you update to the latest cime it should fix the distutils issue.
| - uses: actions/checkout@v4 | ||
|
|
||
| - name: Setup python ${{ matrix.python-version }} | ||
| uses: actions/setup-python@v4 |
There was a problem hiding this comment.
should be actions/setup-python@v5
I'll leave that to Chris when he pushes the newest alpha tags to |
|
@mnlevy1981 can you give me permission to push to your fork please? |
|
Closed in favor of #307 |
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.
Description of changes
Adds Github Action to create cases using a range of compsets (currently
B,C,Cwith MARBL,D,I,Q, andT) and runpreview_namelists. I intentionally set up the matrix to turn this into 35 parallel jobs (7 compsets and 5 versions of python)... it might be more efficient to have a separate job for each version of python that creates all 7 compsets in that same job, but I think that would make it harder to parse the results. If aBcompset is failing because of issues in MOM6, CAM, and CLM, then we should see failures inB, bothCs,I, andQtests while a single job per python version would just abort after the first failure and not catch the other four.Specific notes
Contributors other than yourself, if any: @manishvenu was helpful in getting past some issues I had with github actions
Fixes: [Github issue #s] And brief description of each issue.
User interface changes?: [ No/Yes ]
[ If yes, describe what changed, and steps taken to ensure backward compatibility ]
Testing performed (automated tests and/or manual tests): These tests have run on my branch at https://github.com/mnlevy1981/cesm/actions/runs/11946703920 -- note that I had 3.14 in the python version matrix, but those tests failed with
So I removed it from this PR. Test results:
buildnmlrelies onglob.globAPI introduced in 3.10distutilshas been removed from the standard library