-
Notifications
You must be signed in to change notification settings - Fork 249
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
Third reconciliation PR from production/RRFS.v1 #2482
Third reconciliation PR from production/RRFS.v1 #2482
Conversation
@grantfirl can you sync up branch? we can start working on this pr. |
@jkbk2004 OK, should be good to go. |
Changes also occurred in intelllvm cases + |
I would like to reiterate my request that someone from EMC needs to review all PR's before merge. I was in the middle of my review when it was merged. |
All code changes are pretty much at physics level and all approved ok at sub components. |
Commit Queue Requirements:
Description:
This is mainly work from @AndersJensen-NOAA. This PR is identical to #2161, but targets develop instead of the RRFSv1 release branch.
This PR is the first of many that will modernized, modularize, and streamline Thompson-Eidhammer microphysics.
This PR includes:
The addition of parameterized kind: REAL -> real(kind_phys)
Consistent indentation
Removal of GOTO statements
Add physical constant metadata needed by Thompson MP
Commit Message:
Priority:
Git Tracking
UFSWM:
Sub component Pull Requests:
UFSWM Blocking Dependencies:
Changes
Regression Test Changes (Please commit test_changes.list):
All tests that use Thompson MP will have non bit-for-bit results due to high significant digit changes in physical constants. There is no substantial change in results as tested by @AndersJensen-NOAA in #2161.
Input data Changes:
Library Changes/Upgrades:
Testing Log: