-
Notifications
You must be signed in to change notification settings - Fork 65
New test to demonstrate GNU failure #689
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
base: develop
Are you sure you want to change the base?
Conversation
Update main with develop branch
…an modules in the correct place (#4) Update CMakeLists.txt to: - Bring in updates from NCAR ccpp-framework main as needed and update CMakeLists.txt to modern cmake version 3 - Support NEPTUNE cmake build while retaining compatibility with UFS/SCM
…an modules in the correct place (NCAR#653) **Update src/CMakeLists.txt: use modern cmake features to install Fortran modules in the correct place** Update CMakeLists.txt to modern cmake version 3 and support NEPTUNE cmake build while retaining compatibility with UFS/SCM User interface changes?: No Fixes: [Github issue #s] n/a Testing: all CI tests pass, ran `capgen` and `prebuild` tests locally on my laptop - all pass test removed: unit tests: system tests: manual testing: Associated PRs: - ccpp-physics: ufs-community/ccpp-physics#259 - FV3: NOAA-EMC/ufsatm#940 - ufs-weather-model: ufs-community/ufs-weather-model#2659
… feature/pull_in_dev_20250724
… feature/pull_in_dev_20250724
gold2718
left a comment
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.
Can this PR be prioritized so that we can work towards a solution?
climbfuji
left a comment
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.
Obviously, we can't merge this because it breaks the CI tests. Is the intention to keep this PR open until a fix is in place and the test passes?
Fair. I suppose I can always cut a branch from this PR branch if I want to play with the problem. |
Correct, and we'll keep this branch up to date with develop so that you can always pull that in, too. |
New test to demonstrate GNU failure when passing unallocated fields into the Group cap.
See error HERE
User interface changes?: No
Related to #690
Testing:
test removed:
unit tests:
system tests:
manual testing: