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

Update Regression Test to include gfsv17 prototypes that are after prototype 8, and turn on prognostic closure #598

Merged
merged 21 commits into from
Dec 5, 2022

Conversation

lisa-bengtsson
Copy link
Contributor

@lisa-bengtsson lisa-bengtsson commented Oct 28, 2022

Description

These PR’s in ufs-weather-model, fv3atm and ccpp/physics does the following:
Creates a set of _gfsv17 RT’s to reflect tests that are beyond prototype 8 targeted for GFSv17
Activates prognostic closure by setting namelist progsigma = true in *gfsv17 and pointing to new field_table in the RT’s
Addresses additional parametric tuning in progclosure_calc.F90
Adds a new gfsv17 field_table and diag_table

Issue(s) addressed

ufs-community/ufs-weather-model#1477

Testing

New ufs-weather-model RT tests have been carried out on Hera

Dependencies

ufs-community/ufs-weather-model#1480
ufs-community/ccpp-physics#18

@jkbk2004
Copy link
Collaborator

jkbk2004 commented Dec 5, 2022

@lisa-bengtsson can you update ccpp submodule pointer and revert the gitmodules branch?

@lisa-bengtsson
Copy link
Contributor Author

@jkbk2004 I updated the pointer in .gimodules

@jkbk2004
Copy link
Collaborator

jkbk2004 commented Dec 5, 2022

@lisa-bengtsson correct ccpp hash is ufs-community/ccpp-physics@befc336. Can you update the ccpp submodule poiinter?

@lisa-bengtsson
Copy link
Contributor Author

@jkbk2004 how can I see the submodule hashes ?

@jkbk2004
Copy link
Collaborator

jkbk2004 commented Dec 5, 2022

@jkbk2004 how can I see the submodule hash
The ccpp hash on your branch is pointing to correct one now. All looks fine to merge in here.

@lisa-bengtsson
Copy link
Contributor Author

@jkbk2004 how can I see the submodule hash
The ccpp hash on your branch is pointing to correct one now. All looks fine to merge in here.

Ok, thank you for confirming.

@jkbk2004
Copy link
Collaborator

jkbk2004 commented Dec 5, 2022

@DusanJovic-NOAA @BrianCurtis-NOAA you can go ahead to approve this pr.

@BrianCurtis-NOAA BrianCurtis-NOAA merged commit 92b0386 into NOAA-EMC:develop Dec 5, 2022
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