Capability for understanding data in Lambert conformal projection implemented #1319
Conversation
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
This commit includes three calls of map_set for the Lambert conformal projection (PROJ_LC) case in three subroutines in mpas_init_atm_cases.F (blend_bdy_terrain, init_atm_case_gfs and init_atm_case_lbc).
|
@mgduda Could you take a look at this? I'd like to get the RUC soil grid variables done before the next release also but I'd like to have this one merged before implementing that because the HRRR/RAP nesting only works with the Lambert projection implemented. Thank you. |
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.
As a next step towards being able to nest MPAS into RAP and HRRR this pull request includes changes to mpas_init_atm_cases.F. In three subroutines (blend_bdy_terrain, init_atm_case_gfs and init_atm_case_lbc), the case field % iproj == PROJ_LC is now included.
Disclaimer: this is not fully tested yet, since the HRRR/RAP soil grid can not yet be understood by MPAS, which I plan on fixing in a future pull request. However, the changes made in this commit won't break anything.