Local externals with new OTB defaults, minor bug fixes, remove unused code#22
Open
jtruesdal wants to merge 2 commits into
Open
Local externals with new OTB defaults, minor bug fixes, remove unused code#22jtruesdal wants to merge 2 commits into
jtruesdal wants to merge 2 commits into
Conversation
Collaborator
|
Hey John, yes, I'll run through my Running CAM-MPAS document and make sure things still are working correctly. |
Collaborator
|
@jtruesdal so sorry about the delay on this. It slipped passed my radar after I made that comment. I just ran my tests using my Running CAM-MPAS document and it all ran successfully! |
MiCurry
approved these changes
Jun 16, 2021
jtruesdal
pushed a commit
that referenced
this pull request
Jun 20, 2024
fix py3.11 issue in gitinterface
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Update for cam_mpas_dev to allow more compsets to work out of the box. F2000climo mpasa120 should work now. Removed unused code due to weak scaling updates and fixed a minor syntax error caught by NAG. @MiCurry could you run a minimal test that verifies that the runs you advertised to the MPAS development community still work. I'd like to put in some regression tests for those (maybe with a category of aux_cam_mpas for Cheyenne and Izumi) that we can run for each of our PR's. For now, I ran 2 tests out of cime/scripts on Cheyenne and Izumi: