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

Missing grib2 definitions #2

Closed
pgchekill opened this issue Aug 19, 2021 · 3 comments
Closed

Missing grib2 definitions #2

pgchekill opened this issue Aug 19, 2021 · 3 comments

Comments

@pgchekill
Copy link

pgchekill commented Aug 19, 2021

Hi,

While trying to parse data from the ICON-EU model, I saw that the parameter that was of interest to me was missing from the code tables. The result is that the parameter field of the definition section is set to null.

The parameter in question is 52 from table 4.2 or Total Precipitation Rate (TPRATE). (ref: [https://www.nco.ncep.noaa.gov/pmb/docs/grib2/grib2_doc/grib2_table4-2-0-1.shtml])

Would be ok to add some missing Grib2 definitions ?

Best.

@nmangue
Copy link
Owner

nmangue commented Aug 22, 2021 via email

@pgchekill
Copy link
Author

Hi,

As suggested, I've submitted a PR for the modification of parameter definitions. I only completed category 0 and 1 of table 4.2.

And you're right, in my code I was able to to determine the parameter by the index instead of the parameter object. But I thought it would be nice to have the "correct" definition. Your library is pretty neat to use 👍

Best regards

@nmangue
Copy link
Owner

nmangue commented Aug 26, 2021

Thank you very much for your comment and your contribution !

@nmangue nmangue closed this as completed Aug 26, 2021
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

No branches or pull requests

2 participants