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

NCAR-main PR#956 #623

Merged
merged 6 commits into from
Mar 2, 2023
Merged

NCAR-main PR#956 #623

merged 6 commits into from
Mar 2, 2023

Conversation

dustinswales
Copy link
Contributor

Description

This PR contains new interstitial variables in GFS_typedefs to accommodate changes in the underlying physics. These changes are mostly related to the removal of a radiation configuration(data) module, physparam.f. Runtime information is set in GFS_typedefs and provided through the ccpp infrastructure, instead of referencing physparam.f throughout the physics.

Issue(s) addressed

Detailed in NCAR #956

Testing

Tested on Cheyenne and Hera with Intel.
Updates need for the RRTMGP RTs (see conversation in NCAR/ccpp-physics#956)

@grantfirl
Copy link
Collaborator

@dustinswales ufs-community/ccpp-physics#38 has been merged. Please revert .gitmodules and update the ccpp-physics submodule pointer.

@jkbk2004
Copy link
Collaborator

jkbk2004 commented Mar 2, 2023

@dustinswales correct ccpp hash is ufs-community/ccpp-physics@4a75606

@jkbk2004 jkbk2004 merged commit c932926 into NOAA-EMC:develop Mar 2, 2023
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