-
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] Component hash updates, FV3_HRRR namelist changes #906
[develop] Component hash updates, FV3_HRRR namelist changes #906
Conversation
…ease_public_v1 test.
…ilable staged on disk for original date
f159832
to
fc8e8d9
Compare
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.
@mkavulich - Thanks for updating the component hashes as part of your FV3_HRRR namelist changes! These changes look good to me!
I have gone ahead and run the fundamental tests on Hera:
----------------------------------------------------------------------------------------------------
Experiment name | Status | Core hours used
----------------------------------------------------------------------------------------------------
grid_RRFS_CONUScompact_25km_ics_HRRR_lbcs_RAP_suite_RRFS_v1beta COMPLETE 8.24
nco_grid_RRFS_CONUS_25km_ics_FV3GFS_lbcs_FV3GFS_timeoffset_suite_ COMPLETE 11.31
grid_RRFS_CONUS_25km_ics_FV3GFS_lbcs_FV3GFS_suite_GFS_v15p2 COMPLETE 6.62
grid_RRFS_CONUS_25km_ics_FV3GFS_lbcs_FV3GFS_suite_GFS_v17_p8_plot COMPLETE 12.52
grid_RRFS_CONUScompact_25km_ics_HRRR_lbcs_HRRR_suite_HRRR COMPLETE 25.19
grid_SUBCONUS_Ind_3km_ics_HRRR_lbcs_RAP_suite_WoFS_v0 COMPLETE 12.96
grid_RRFS_CONUS_25km_ics_NAM_lbcs_NAM_suite_GFS_v16 COMPLETE 18.77
----------------------------------------------------------------------------------------------------
Total COMPLETE 95.61
as well as the aqm_grid_AQM_NA13km_suite_GFS_v16 test, which also successfully passed:
----------------------------------------------------------------------------------------------------
Experiment name | Status | Core hours used
----------------------------------------------------------------------------------------------------
aqm_grid_AQM_NA13km_suite_GFS_v16 COMPLETE 1039.44
----------------------------------------------------------------------------------------------------
Total COMPLETE 1039.44
Approving this work now.
Manual run of the WE2E coverage tests on Hera Intel all successfully passed:
|
The manual testing of Hera GNU WE2E coverage tests has successfully passed:
Once the current reruns of Gaea and Jet successfully pass, this work can be merged. |
The Jenkins tests have successfully passed on both Gaea:
and Jet:
Moving forward with merging this PR now. |
DESCRIPTION OF CHANGES:
This is a multi-purpose PR, introducing updates related to CCPP physics, a few WE2E test changes that didn't make it into #871) and updating external hashes.
Updating component hashes
Physics updates
WE2E test changes
GST_release_public_v1
(see issue GST_release_public_v1 test fails on Hera in latest develop #874)custom_ESGgrid_SF_1p1km
to use ICs from HRRR and LBCs from RAP. Previously all custom grid tests used FV3GFS data, so this adds additional diversity to the tests.Type of change
TESTS CONDUCTED:
Ran a large number of relevant tests on Hera, and Orion, all passed.
custom_ESGgrid_SF_1p1km
test.DEPENDENCIES:
This PR incorporates changes from #892 which will be closed, since these changes required a hash update anyway it made sense to combine to consolidate testing efforts.
Includes changes from PR #871, so that must be merged before this one.DOCUMENTATION:
In progress.
ISSUE:
CHECKLIST
CONTRIBUTORS:
Thanks to @MichaelLueken for letting me roll his hash update changes into this PR
Thanks to Tanya Smirnova (CIRES/NOAA-GSL) for her help in updating the FV3_HRRR namelist settings.