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

Merge older and newer MRMS GRIB2 table files #2508

Closed
JohnHalleyGotway opened this issue Apr 10, 2023 · 2 comments · Fixed by #2559
Closed

Merge older and newer MRMS GRIB2 table files #2508

JohnHalleyGotway opened this issue Apr 10, 2023 · 2 comments · Fixed by #2559
Assignees
Labels
MET: Configuration priority: medium Medium Priority requestor: DTC/T&E General DTC Testing and Evaluation work required: FOR OFFICIAL RELEASE Required to be completed in the official release for the assigned milestone type: enhancement Improve something that it is currently doing
Milestone

Comments

@JohnHalleyGotway
Copy link
Collaborator

Discussed in https://github.com/dtcenter/METplus/discussions/2111

Originally posted by michelleharrold April 7, 2023
Happy Friday!

I was just trying to run pcp-combine on MRMS precipitation data from 2020 using this command:

pcp_combine -sum 00000000_000000 1 20200811_000000 24 -pcpdir obs -field 'name="GaugeCorrQPE01H"; level="L0";' -name "APCP_24" mrms_24h.nc

However, I got an error message:

ERROR  : 
ERROR  : VarInfoGrib2::set_dict() -> unrecognized GRIB2 field abbreviation 'GaugeCorrQPE01H'
ERROR  : 

After chatting with @JohnHalleyGotway - we discovered that the MRMS MET GRIB tables have been updated to match the updated MRMS data (see MET issue for more details as well as the change log for the grib2_mrms.txt file).

Would it be possible to take the union of the old grib table entries and new grib table entries in order to make the grib2_mrms.txt file backward compatible for older MRMS data?

Thanks!
Michelle

@JohnHalleyGotway JohnHalleyGotway transferred this issue from dtcenter/METplus Apr 10, 2023
@JohnHalleyGotway JohnHalleyGotway moved this from 📋 Backlog to 🔖 Ready in MET-11.1.0 Development Apr 10, 2023
@JohnHalleyGotway JohnHalleyGotway added this to the MET 11.1.0 milestone Apr 10, 2023
@JohnHalleyGotway JohnHalleyGotway added alert: NEED ACCOUNT KEY Need to assign an account key to this issue alert: NEED CYCLE ASSIGNMENT Need to assign to a release development cycle type: enhancement Improve something that it is currently doing priority: medium Medium Priority requestor: DTC/T&E General DTC Testing and Evaluation work MET: Configuration labels Apr 13, 2023
@JohnHalleyGotway JohnHalleyGotway self-assigned this May 11, 2023
@JohnHalleyGotway JohnHalleyGotway added the required: FOR OFFICIAL RELEASE Required to be completed in the official release for the assigned milestone label May 18, 2023
@JohnHalleyGotway JohnHalleyGotway moved this from 🔖 Ready to 🏗 In progress in MET-11.1.0 Development Jun 2, 2023
@JohnHalleyGotway JohnHalleyGotway linked a pull request Jun 7, 2023 that will close this issue
15 tasks
@JohnHalleyGotway JohnHalleyGotway removed alert: NEED ACCOUNT KEY Need to assign an account key to this issue alert: NEED CYCLE ASSIGNMENT Need to assign to a release development cycle labels Jun 7, 2023
@JohnHalleyGotway JohnHalleyGotway moved this from 🏗 In progress to 👀 In review in MET-11.1.0 Development Jun 7, 2023
JohnHalleyGotway added a commit that referenced this issue Jun 9, 2023
@JohnHalleyGotway JohnHalleyGotway moved this from 👀 In review to ✅ Done in MET-11.1.0 Development Jun 9, 2023
@michelleharrold
Copy link

Release Acceptance Testing Summary

Version: METplus-5.1.0-rc1
Date: 07/05/2023
Location: Cheyenne (module use /glade/p/ral/jntp/MET/MET_releases/modulefiles; module load met/11.1.0-rc1)
Status (PASS/FAIL): PASS
Description: Executed pcp-combine command that previously failed and re-ran with METv11.1.0-rec1 to confirm new functionality works.

1 similar comment
@michelleharrold
Copy link

Release Acceptance Testing Summary

Version: METplus-5.1.0-rc1
Date: 07/05/2023
Location: Cheyenne (module use /glade/p/ral/jntp/MET/MET_releases/modulefiles; module load met/11.1.0-rc1)
Status (PASS/FAIL): PASS
Description: Executed pcp-combine command that previously failed and re-ran with METv11.1.0-rec1 to confirm new functionality works.

@github-project-automation github-project-automation bot moved this from ✅ Done to 🏗 In progress in MET-11.1.0 Development Jul 5, 2023
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in MET-11.1.0 Development Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MET: Configuration priority: medium Medium Priority requestor: DTC/T&E General DTC Testing and Evaluation work required: FOR OFFICIAL RELEASE Required to be completed in the official release for the assigned milestone type: enhancement Improve something that it is currently doing
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants