Skip to content
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

Updating the RRFS GRIB2 variable handle for "MSTAV" #225

Merged
merged 1 commit into from
Sep 18, 2023

Conversation

jaymes-kenyon
Copy link
Collaborator

A previous commit to the UPP repository (NOAA-EMC/UPP@83e59b5) resulted in a change to the RRFS GRIB2 variable handle for soil moisture availability (MSTAV). As a result of this change, Python graphics are failing in RRFS-B. Running an "ncl_filedump" on a recent RRFS-B GRIB2 file indicates that the new variable handle is "MSTAV_P0_L106_GLC0". This update to pygraf has been successfully used in a RRFS CONUS 3-km retro on Hera.

Copy link
Collaborator

@cshartsough cshartsough left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for putting this together. I should still add something down the road to exit more gracefully when the needed fields for the firewx or other calcs aren't available.

@jaymes-kenyon
Copy link
Collaborator Author

jaymes-kenyon commented Sep 18, 2023

Thanks, @cshartsough. BTW, I do not have write access here; can someone click the "merge" button? Then, I'll make a separate PR in ufs-srweather-app to update the pygraf hash.

@cshartsough cshartsough merged commit e5bf0d5 into NOAA-GSL:main Sep 18, 2023
2 checks passed
@jaymes-kenyon jaymes-kenyon deleted the update_grib2_variable_name branch September 18, 2023 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants