Modify/add predefined grids#305
Merged
gsketefian merged 20 commits intoOct 8, 2020
Merged
Conversation
…RRR25km and GSD_HRRR13km grids so that they include all of the contiguous US; add new WE2E test for the GSD_HRRR13km grid (previously, none existed).
… expected by the latest version of tests/run_experiments.sh.
… Lambert conformal write-component grid.
The -f flag to readlink requires that the last "component" of the path passed to it exist. In the way readlink is used in ush/setup.sh, the path may not yet exist. Thus, change the -f to a -m, which states that none of the components of the path need to exist.
…with the new regional_esg_grid code.
…files from user-staged directories.
…with the new regional_esg_grid code.
…ese are now automatically optimized by the new regional_esg_grid code.
…possible while still fitting inside the HRRRX domain.
…ion of an RRFS_CONUS_3km grid.
* Change name of GSD_RRFSAK_3km grid to RRFS_AK_3km and modify its write-component grid parameters using values generated by the NCL grid-plotting scripts. * Add RRFS_AK_13km grid.
jwolff-ncar
reviewed
Oct 8, 2020
Contributor
jwolff-ncar
left a comment
There was a problem hiding this comment.
I did not test (since I run on hera and it has already been tested there) but looking through the files everything seems in order. (Does that count as an approval?!)
Collaborator
Author
Sounds good to me, but you still gotta push some buttons to approve! There should be a green "Add your review" button up top. |
jwolff-ncar
approved these changes
Oct 8, 2020
Contributor
jwolff-ncar
left a comment
There was a problem hiding this comment.
Given the thumbs up from Gerard that my visual review was good enough I am approving this PR.
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
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:
This PR renames and modifies several existing grids and creates some new ones. Details below.
Note that the 3 grids required for the ufs-srweather-app release are created in this PR: RRFS_CONUS_25km, RRFS_CONUS_13km, RRFS_CONUS_3km. These pass their tests (2 tests per grid) except for RRFS_CONUS_13km, which fails one of its two tests in the make_ics task. This may need further attention, possibly after the PR is merged.
CONUS and SUBCONUS grids of ESGgrid type:
CONUS grids of GFDLgrid type:
Alaska grids:
TESTS CONDUCTED:
Ran the 14 new WE2E tests added in this PR. Test results are as follows:
Note that: