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.

Sign up for GitHub

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

Jump to bottom

Issue reported by Algeria: wis2box only read first message from ingested BUFR file #331

Closed
maaikelimper opened this issue Nov 17, 2022 · 1 comment
Closed

Issue reported by Algeria: wis2box only read first message from ingested BUFR file #331

maaikelimper opened this issue Nov 17, 2022 · 1 comment
Assignees
@david-i-berry @webb-ben
Labels
bug Something isn't working

Comments

@maaikelimper
Copy link
Collaborator

"Recently, we noticed that not all of the station data was displayed on the WIS2BOX map. Upon further research, we learned that the system was only reading the first message from our BUFR files."

To reproduce issue, examble of bufr containing 5 messages in this zip:
msg00001637.zip
@maaikelimper maaikelimper added the bug Something isn't working label Nov 17, 2022
@david-i-berry
Copy link
Member

Issue in bufr2geojson, fixed by PR16 (wmo-im/bufr2geojson#16)

This is also likely present in the BUFR passthrough.

@tomkralidis tomkralidis changed the title Issue reported by Algera: wis2box only read first message from ingested BUFR file Issue reported by Algeria: wis2box only read first message from ingested BUFR file Nov 20, 2022
webb-ben added a commit that referenced this issue Nov 22, 2022
@webb-ben
Split BUFR handling into separate functions
cf71032 
- Create separate functions to process individual messages (#331) and subsets.
- Throw away BUFR subsets missing data
- Write BUFR directly to bytestring
webb-ben added a commit that referenced this issue Nov 22, 2022
@webb-ben
Split BUFR handling into separate functions
f5f5b76 
- Create separate functions to process individual messages (#331) and subsets.
- Throw away BUFR subsets missing data
- Write BUFR directly to bytestring
@webb-ben webb-ben mentioned this issue Nov 22, 2022
Merged
tomkralidis added a commit that referenced this issue Nov 25, 2022
@tomkralidis
bump bufr2geojson dependency (#331)
33c23fe
@tomkralidis tomkralidis mentioned this issue Nov 25, 2022
Merged
tomkralidis added a commit that referenced this issue Nov 27, 2022
@tomkralidis
bump bufr2geojson dependency (#331) (#334)
fad769f
webb-ben added a commit that referenced this issue Nov 27, 2022
@webb-ben
Split BUFR handling into separate functions
5a8ba69 
- Create separate functions to process individual messages (#331) and subsets.
- Throw away BUFR subsets missing data
- Write BUFR directly to bytestring
tomkralidis pushed a commit that referenced this issue Nov 28, 2022
@webb-ben
Multi BUFR (#332)
c24e6a3 
* Split BUFR handling into separate functions

- Create separate functions to process individual messages (#331) and subsets.
- Throw away BUFR subsets missing data
- Write BUFR directly to bytestring

* Combine messages

cat algeria test BUFR files together into one file of ten messages

* Update bufr4 docstrings

- Update bufr4 docstrings
- Use bufr2geojson get_tsi() for traditional station identifier
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@david-i-berry david-i-berry

@webb-ben webb-ben

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@maaikelimper @david-i-berry @webb-ben