Update regional_workflow for use with authoritative UFS_UTILS utilities and add support for global/regional grib2 functionality in chgres_cube #274
Conversation
…ow repository. Remove run-time make_grid module file for all platforms. Will be sourced from UFS_UTILS from now on.
…r FV3GFS and GSMGFS nemsio files
… into feature/chgres_cube
…odel/SDF combinations
This reverts commit b04ad0b.
|
The regional_014 test will be fixed as long as this PR makes its way into the feature/regional_release branch of Larissa's UFS_UTILS fork. |
|
Here's a summary of the latest failed tests. I'd say start with trying to fix nco_GSD_HRRR3km_HRRRX_RAPX because that's what GSL wants to run. The experiment directories for all the WE2E tests I ran for this PR can be found under here (on hera): /scratch2/BMC/det/Gerard.Ketefian/UFS_CAM/expt_dirs/PR274_UFS_UTILS_20200902 Below, I list the test name followed by the grid, suite, and external model for ICs/LBCs that the test uses. Then the bullet points state which task fails and other info. You can look in the log files for detailed error messages. GSD_RAP13km -- GSD_RAP13km, FV3_GSD_v0, FV3GFS/FV3GFS:
nco_GSD_HRRR3km_HRRRX_RAPX -- GSD_HRRR3km, FV3_GSD_SAR, HRRRX/RAPX:
regional_003 -- GSD_HRRR25km, FV3_GSD_v0, HRRRX/RAPX:
regional_004 -- GSD_HRRR25km, FV3_GSD_SAR, HRRRX/RAPX:
regional_005 -- GSD_HRRR_AK_50km, FV3_GSD_SAR, RAPX/RAPX:
regional_010 -- GSD_HRRR25km, FV3_CPT_v0, GSMGFS/GSMGFS:
regional_013 -- GSD_HRRR25km, FV3_GSD_SAR_v1, GSMGFS/GSMGFS:
regional_014 -- GSD_HRRR25km, FV3_GFS_2017_gfdlmp, GSMGFS/GSMGFS:
regional_016 -- GSD_HRRR25km, FV3_RRFS_v0, GSMGFS/GSMGFS:
|
… into feature/chgres_cube
… into feature/chgres_cube
|
@gsketefian Latest commit should fix your WE2E errors. |
|
@JeffBeck-NOAA I reran the failed tests. The NCO-mode one (nco_GSD_HRRR3km_HRRRX_RAPX) works but the others still fail because of the bugs in PR #280 that are currently in the HEAD of develop. I have a bugfix that I will push this afternoon which you'll have to merge and then we can rerun the community mode tests. |
|
@gsketefian, sounds good. I will merge as soon as it's available. |
|
@JeffBeck-NOAA I just checked this out but it doesn't have my latest PR. Can you merge the latest develop into this so I can retest? Thx. |
|
Yes. Doing that now.
…On Tue, Sep 15, 2020 at 2:21 PM gsketefian ***@***.***> wrote:
@JeffBeck-NOAA <https://github.com/JeffBeck-NOAA> I just checked this out
but it doesn't have my latest PR. Can you merge the latest develop into
this so I can retest? Thx.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/NOAA-EMC/regional_workflow/pull/274#issuecomment-692957391>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ANFE562IUA7LMRE64MTDH53SF7EDVANCNFSM4QGN2PFA>
.
|
… into feature/chgres_cube
…bcs scripts, from Gerard's PR
|
@JeffBeck-NOAA Can you update your PR#17 into ufs-srweather-app so that it uses the release/public-v2 branch? The current ufs-weather-model does not have the RRFS_v1beta suite defined yet (SDF is missing). |
|
@JeffBeck-NOAA Also, can you check if the FV3_CPT_v0 suite accidentally got deleted from the make_ics.sh and make_lbcs.sh scripts where the variable mapping table is set? This is the error I'm getting from the make_ics.sh script (from test regional_010, which uses the FV3_CPT_v0 suite): Same error from the make_lbcs.sh script. Thanks. |
|
@gsketefian I added the FV3_CPT_v0 SDF to the necessary lines in the make_ics/lbcs scripts. I just pushed the commit to this PR, so it should work now. |
|
@JeffBeck-NOAA Ok, the FV3_CPT_v0 issue is fixed. Also, the RRFV_v1beta suite is now available in ufs-weather-model. BUT, because we're now using regional_esg_grid instead of the old regional_grid to generate grids, the orography statistics files that Mike Toy made no longer work. Most importantly, there is an unknown grid number issue in the tests that use HRRRX or RAPX for the ICs, which is probably the most important problem to fix (it wasn't there before I pulled your latest commits). Below is a summary of the latest WE2E test results on hera. They can be found in this directory:
There is an earlier set of runs (from before when I pulled the latest updates to the feature/chgres_cube forks yesterday) in which the "unknown input file grid template number" error in make_ics and make_lbcs does not occur. Those runs are here:
Here's a summary of the latest runs (i.e. the ones in the first directory I listed above): |
|
@gsketefian Recent changes in the UFS_UTILS build.hera module file have broken wgrib2 on Hera. I just left a comment for the commit in UFS_UTILS that has these changes and detailed what the problem is. This is the source of the problem with an "unknown input file grid". Chgres_cube is unable to run wgrib2 on the input file to identify what kind of grid it is. |
…ow that these are proper modulefiles
|
As of 20200918, on hera all WE2E tests pass except for the following three, which are expected to fail: Since this PR needs to go in urgently, I will approve it along with with PR #17 to ufs-community/ufs-srweather-app. |
DESCRIPTION OF CHANGES:
This PR must be tested with PR #17 from ufs-srweather-app
Modifies j-jobs/exregional scripts to work with a single UFS_UTILS repository and the new version of chgres_cube, up-to-date with the authoritative UFS_UTILS repository.
Adds support for Thompson climatology code within chgres_cube when using external model data that does not contain the ice- and water-friendly aerosol fields for Thompson MP. Fixed file for Thompson climatology code needs to be propagated to other platforms (currently resides only on Jet and Hera in ${FIXam}).
Adds support for the new regional_grid without alpha and kappa parameters in the namelist (these variables are optimized in the new version of the code)
Removes all utilities (/sorc directory) from regional_workflow
Sources modulefiles from UFS_UTILS for build and run-time (removes "modulefiles/codes" directory and no longer uses fixed make_grid, mosaic_file, and global_equiv_resol modulefiles from "modulefiles/tasks")
Removes all unnecessary surface-related variables and processing from exregional_make_lbcs.sh
TESTS CONDUCTED:
Tested successfully on Hera. Needs to be tested on at least Jet, WCOSS, and Cheyenne.
Note: Not all external model data and SDF combinations are supported by chgres_cube. Combining RAP or HRRR external model data with a Noah MP SDF will fail, since interpolation of 9 to 4 soil layers has not yet been implemented.Fixed in subsequent commit to Larissa's fork of UFS_UTILS.Using FV3GFS model data with an SDF containing RUC-LSM (4 to 9 soil levels) has a known bug that will be fixed shortly.- Fixed in subsequent commit to this PR.CONTRIBUTORS (optional):
@LarissaReames-NOAA