-
Notifications
You must be signed in to change notification settings - Fork 164
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
Infrastructure improvements for radiation #193
Infrastructure improvements for radiation #193
Conversation
The PR description is incomplete, can that be fixed please? Also, associated PRs: NCAR/ccpp-physics#514 For regression testing information, see ufs-community/ufs-weather-model#262 |
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.
Straightforward!
Please update .gitmodules
temporarily to point to your ccpp-physics fork and your branch master-ncar-f3e6761? Thanks.
Submodule pointer for ccpp-physics is correct, this can be merged. |
This PR contains changes to accommodate improvements to both the RRTMG and RRTMGP radiation schemes.
The changes include new interstitial fields for use by the GP scheme, this includes new fields for configuration information.
Associated PRs:
NCAR/ccpp-physics#514
#193
ufs-community/ufs-weather-model#262
These changes were tested on Hera using Intel compilers. Existing RT's for the G scheme passed. New baselines for the GP RT's need to be created, for both v15p2 and