-
Notifications
You must be signed in to change notification settings - Fork 119
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
[develop] Bug fix for FV3_GFS_v17_p8 physics suite #1055
[develop] Bug fix for FV3_GFS_v17_p8 physics suite #1055
Conversation
The
The cold bias noted in issue #1004 (and seen in the first image in the Tests Conducted section above) is no longer present following this update: Approving this PR now. |
Test passed
2m temp ant hour 06: Approving. |
As discussed at the meeting today, I will submit the Jenkins tests, but manually submit the fundamental tests on the platforms that have no allocations/fairshare. |
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.
Thanks for your work in fixing this issue!
The Jenkins tests have successfully passed on Derecho and Hercules. One test failed for both the Gaea and Jet tests. For Jet, the
On Gaea, the
As expected, the Jenkins tests are sitting in queue on both Hera and Orion. Thankfully, I was able to successfully run the WE2E coverage tests on Hera Intel using the Rocky 8 nodes:
Given that the Orion |
DESCRIPTION OF CHANGES:
A SRW App user noticed an issue with the
FV3_GFS_v17_p8
physics suite, that the surface temperatures were dropping unrealistically throughout the forecast. This PR addresses that issue by updating theFV3_GFS_v17_p8
physics suite in theparm/FV3.input.yml
file.This issue was discovered in the SRW App v2.2.0, but since the
FV3_GFS_v17_p8
physics suite is not officially supported for the release, the change will only go into in the develop branch.Type of change
TESTS CONDUCTED:
I used a WE2E test to ensure the changes worked. The experiment can be found on Derecho here: /glade/work/esnyder/ufs-case-studies/expt_dirs/fan-update-3/grid_RRFS_CONUS_25km_ics_FV3GFS_lbcs_FV3GFS_suite_GFS_v17_p8_plot
User Issue (before the PR) - sfc temp at fcst hour 6
PR Changes - sfc temp at fcst hour 6
DEPENDENCIES:
DOCUMENTATION:
No documentation is required since the physics suites aren't thoroughly explained in the SRW App documentation.
ISSUE:
This PR addresses Issue 1004.
CHECKLIST
LABELS (optional):
A Code Manager needs to add the following labels to this PR:
CONTRIBUTORS (optional):
Thanks to @ulmononian, @ligiabernardet, Man Zhang, and Fanglin Yang for their help troubleshooting this problem!