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

Add packingType for template 5.200 (Run length packing with level values) #74

Closed
wants to merge 1 commit into from

Conversation

noritada
Copy link

Hello,

Thank you for adding support for template 5.200. I used HEAD in the repository and noticed that packingType was left as unknown in grib_ls, so I added it.

I named RLE's packingType after simple packing and complex packing, but if there is a naming convention and my naming is inappropriate, I would appreciate it if you could let me know.

% ./install/bin/grib_ls Z__C_RJTD_20160822020000_NOWC_GPV_Ggis10km_Pphw10_FH0000-0100_grib2.bin
Z__C_RJTD_20160822020000_NOWC_GPV_Ggis10km_Pphw10_FH0000-0100_grib2.bin
edition      centre       date         dataType     gridType     stepRange    typeOfLevel  level        shortName    packingType
2            rjtd         20160822     af           regular_ll   0            surface      0            unknown      unknown
2            rjtd         20160822     af           regular_ll   10           surface      0            unknown      unknown
2            rjtd         20160822     af           regular_ll   20           surface      0            unknown      unknown
2            rjtd         20160822     af           regular_ll   30           surface      0            unknown      unknown
2            rjtd         20160822     af           regular_ll   40           surface      0            unknown      unknown
2            rjtd         20160822     af           regular_ll   50           surface      0            unknown      unknown
2            rjtd         20160822     af           regular_ll   1            surface      0            unknown      unknown
7 of 7 messages in Z__C_RJTD_20160822020000_NOWC_GPV_Ggis10km_Pphw10_FH0000-0100_grib2.bin

7 of 7 total messages in 1 files

Note: templates used in the data above is 3.0, 4.0, and 5.200 (so local section 4 templates are not used).
You can download it from JMA's GPV sample data page; the link text is "竜巻発生確度ナウキャスト" and the file name is tornado.zip.
(I have read a comment in #72 and think you are looking for such a data.)

Many thanks!

@shahramn
Copy link
Collaborator

Thank you. I'm afraid I got lost on that Japanese page! could not find tornado.zip
Can you send me the exact URL which will allow me to download it?

@shahramn
Copy link
Collaborator

Dear @noritada
Can you please sign the CLA (Contributor License Agreement). That is a required step before I can continue.
Many thanks

@noritada
Copy link
Author

I hope I can be of help to you. Here is the exact URL of tornado.zip: https://www.data.jma.go.jp/developer/gpv_sample/tornado.zip

By the way, I think CLA check URL would be http:https://claassistant.ecmwf.int/ecmwf/eccodes?pullRequest=74, but my web browsers reports me connection reset (ERR_CONNECTION_RESET) to that URL now. I think it may be a server-side problem. I am waiting to recover from the problem...

@shahramn
Copy link
Collaborator

shahramn commented Feb 13, 2023

I hope I can be of help to you. Here is the exact URL of tornado.zip: https://www.data.jma.go.jp/developer/gpv_sample/tornado.zip

By the way, I think CLA check URL would be http:https://claassistant.ecmwf.int/ecmwf/eccodes?pullRequest=74, but my web browsers reports me connection reset (ERR_CONNECTION_RESET) to that URL now. I think it may be a server-side problem. I am waiting to recover from the problem...

Many thanks. I have downloaded the file now.

Regarding the CLA signing:
Please check this page which may be of some help:
https://confluence.ecmwf.int/display/SUP/ECMWF+software+on+GitHub

@FussyDuck
Copy link

FussyDuck commented Feb 13, 2023

CLA assistant check
All committers have signed the CLA.

@noritada
Copy link
Author

@shahramn Thank you very much for waiting.
Now I am able to access to the CLA app and have signed the agreement!

@shahramn shahramn closed this Feb 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants