RRFS_dev1: Switch to FV3_HRRR physics#49
Merged
christinaholtNOAA merged 8 commits intoMar 25, 2021
Merged
Conversation
…uite in NCO mode (ufs-community#421) @chunhuazhou (Chunhua Zhou) reported that GWD_HRRRsuite_BASEDIR is not getting set correctly when running the FV3_HRRR suite in NCO mode. This PR is to fix that bug. ## DESCRIPTION OF CHANGES: ### Main changes: * In setup.sh, ensure that GWD_HRRRsuite_BASEDIR is set properly when running the FV3_HRRR suite in NCO mode, and add checks to make sure directories exist, etc. * Add 2 new WE2E tests for running the FV3_HRRR suite in NCO mode on the RRFS_CONUS_25km and RRFS_CONUS_3km grids. The test names are nco_grid_RRFS_CONUS_25km_ics_HRRR_lbcs_RAP_suite_HRRR and nco_grid_RRFS_CONUS_3km_ics_HRRR_lbcs_RAP_suite_HRRR. These run the 2020020800 case, which is a winter case that has a history of crashing. ### Improvements: * Remove the regional_workflow/env directory since it is unused. * Change the names of 5 of the existing NCO-mode WE2E tests to make it clearer what configurations they run (and to be consistent with the naming convention of community-mode tests). ## TESTS CONDUCTED: Ran the following WE2E tests (including the two new ones) on Hera. All were successful. * grid_RRFS_CONUS_25km_ics_HRRR_lbcs_RAP_suite_HRRR * nco_grid_RRFS_CONUS_25km_ics_HRRR_lbcs_RAP_suite_HRRR * nco_grid_RRFS_CONUS_3km_ics_HRRR_lbcs_RAP_suite_HRRR (both on 2020020800 and 2020081006)
christinaholtNOAA
requested review from
CurtisAlexander-NOAA,
JeffBeck-NOAA,
TrevorAlcott-NOAA,
chunhuazhou,
hu5970,
jaymes-kenyon and
terraladwig
as code owners
Collaborator
Author
|
So, interestingly my parallel runs with the same configuration are failing with AVOST messages in the forecast. 👎 The only differences I currently see are that the real time runs use kjet with more memory for chgres. Not sure if that's going to be a culprit or not. I am doing a few more tests with that in mind. |
hu5970
approved these changes
Mar 24, 2021
|
Looks good. |
19 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
DESCRIPTION OF CHANGES:
These are the changes necessary to switch RRFS_dev1 to FV3_HRRR physics. One bug fix was cherry-picked from RRFS_dev4 that was not previously required here.
These changes are already running on Jet, but a reconfigure should be done once this PR and the reservations PR (forthcoming) are merged. No rebuild is necessary.
Update (03/24):
After unsuccessful debugging of executable reproducibility on Jet, this set of configuration changes was bundled with changes needed for Jet usage and tracking. The new additions include:
RRFS_dev?_[area]where area is CONUS or AK (for now).TESTS CONDUCTED:
One cycle each of AK and CONUS are pending completion on Jet.
CONTRIBUTORS (optional):
@jaymes-kenyon @hu5970 @JeffBeck-NOAA @chunhuazhou
And so many others!