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

release/public-v2: bugfixes for GNU 10 crashes, update modulefile for cheyenne.gnu #355

Conversation

climbfuji
Copy link
Collaborator

@climbfuji climbfuji commented Jan 4, 2021

Description

This PR updates the submodule pointer for fv3atm and ccpp-physics to fix crashes of the model run with GNU 10. It also updates the modulefile for cheyenne.gnu to use GNU 10.1.0 instead of GNU 9.1.0 so that we have one tier-1 platform for the UFS SRW App release that uses the GNU 10 compilers.

Issue(s) addressed

These PRs address step 2 of #288 (but not step 1).

Testing

With these changes, all regression tests pass with GNU 10.1.0 on Cheyenne (create new baseline and verify against it).

Regression tests passed on all tier-1 platforms for the release/public-v2 branch, logs updated in the PR.

Dependencies

NCAR/ccpp-physics#540
NOAA-EMC/fv3atm#221
#355

@climbfuji climbfuji added Baseline Updates Current baselines will be updated. Waiting for Reviews The PR is waiting for reviews from associated component PR's. release/public-v2 labels Jan 4, 2021
@climbfuji climbfuji added the Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. label Jan 5, 2021
@climbfuji
Copy link
Collaborator Author

Ready to merge!

@climbfuji climbfuji merged commit f5d8a32 into ufs-community:release/public-v2 Jan 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Baseline Updates Current baselines will be updated. Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. Waiting for Reviews The PR is waiting for reviews from associated component PR's.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants