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

Ninth reconciliation PR from production/RRFS.v1 #899

Merged
merged 12 commits into from
Jan 17, 2025

Conversation

grantfirl
Copy link
Collaborator

@grantfirl grantfirl commented Dec 5, 2024

Description

This PR includes both #864 and #868 and is the work of @JiliDong-NOAA. No baseline changes are expected.

This PR also includes #902, which was combined for efficiency reasons.

Issue(s) addressed

None

Testing

See ufs-community/ufs-weather-model#2532 for testing using UFS WM RTs on Hera.

Dependencies

Requirements before merging

  • All new code in this PR is tested by at least one unit test
  • All new code in this PR includes Doxygen documentation
  • All new code in this PR does not add new compilation warnings (check CI output)

@grantfirl
Copy link
Collaborator Author

@FernandoAndrade-NOAA This has been updated after the ccpp-physics merge and is ready to review/merge.

@jkbk2004 jkbk2004 merged commit b40f99d into NOAA-EMC:develop Jan 17, 2025
2 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants