-
Notifications
You must be signed in to change notification settings - Fork 153
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
Ufs dev PR#251 #1112
Merged
Merged
Ufs dev PR#251 #1112
Changes from all commits
Commits
Show all changes
9 commits
Select commit
Hold shift + click to select a range
4ac7547
"update GF/C3 for SRW3.0 release"
haiqinli 578b17f
remove changes to ci and CMakeLists.txt that aren't needed
grantfirl 21469fd
update SCM doc link in README.md
grantfirl 1d0bcdc
Merge pull request #235 from grantfirl/rrfsv1-to-ufs/dev6
rhaesung b2bc10a
Merge branch 'ufs-community:ufs/dev' into ufs/dev-SRW3.0
haiqinli 2e275de
Merge branch 'ufs/dev' into NCAR_main_merge_20250206
grantfirl 585e8a1
Merge branch 'ufs/dev-SRW3.0' into NCAR_main_merge_20250206
grantfirl 412e7f6
Merge pull request #251 from grantfirl/NCAR_main_merge_20250206
grantfirl 71b1768
Merge branch 'ufs/dev' of https://github.com/ufs-community/ccpp-physi…
dustinswales File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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 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 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 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 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
Oops, something went wrong.
Oops, something went wrong.
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.
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.
Come to think of it, I'm not entirely sure that we need this file anymore since we're not maintaining our own branch of fv3atm? At least when this CI is run in the NCAR repo, it will fail henceforth I think. Wasn't the point of this test to sorta provide a "heads up" that the host (SCM) needs to change metadata somehow when physics PRs are introduced? It's trying to build the current fv3 with the PR branch physics. I guess that might still serve a purpose for the ufs/dev branch.
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.
Yeah, I think what we want to do is:
In the NCAR:ccpp-physics we should have a CI test that runs prebuild using the SCM metadata.
In the UFS:ccpp-physics we should have a CI test that runs prebuild using the FV3 metadata.