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

Normalize BUFR4 notes #74

Closed
amilan17 opened this issue May 18, 2021 · 8 comments
Closed

Normalize BUFR4 notes #74

amilan17 opened this issue May 18, 2021 · 8 comments
Assignees

Comments

@amilan17
Copy link
Member

amilan17 commented May 18, 2021

Branch

https://github.com/wmo-im/BUFR4/tree/notes

5e5caad...notes

Purpose

To normalize all notes in published PDF and in GitHub repositories.

@amilan17
Copy link
Member Author

amilan17 commented May 18, 2021

  • migrate all existing notes in tables to a notes.csv file
  • Determine how to normalize notes denoted by a stars and numbers
  • add noteID column to CSVs

@amilan17
Copy link
Member Author

amilan17 commented Aug 4, 2021

https://wmoomm.sharepoint.com/:b:/s/wmocpdb/EVR7epZEpLxHsN8P8pSorLoBLx9dFTznT15zsWc2YAssfQ?e=zwcoJ9

BUFRCREX_CodeFlag

  • discuss: replace starred acronyms with full text + acronym, instead of using note
  • DECISION: create an acronym table, remove notes with acronym meanings

BUFRCREX_TableB

  • ~ 0 08 045 is missing. Referenced in Class 08, note 7: "When descriptor 0 08 043 is used to specify particulate matter (PM) under a given size threshold, descriptor 0 08 045 may also be used to further specify a subset of the PM population on the basis of ion composition."~ see Table B 0 08 045 is missing #88

BUFR_TableC

BUFR_TableD

  • Review relevancy of note. I don't see any references to dB in category 16 at I.2 — BUFR Table D/16 — 18
    p. 634

(5) Decibel (dB) is a logarithmic measure of the relative power, or of the relative values of two flux densities, especially of sound intensities and radio and radar power densities. In radar meteorology, the logarithmic scale (dBZ) is used for measuring radar reflectivity factor (obtained from the American Meteorological Society Glossary of Meteorology).

CREX_TableD/CREX_TableC

  • not adding notes from CREX tables

@amilan17 amilan17 self-assigned this Aug 4, 2021
@amilan17 amilan17 added this to the FT2022-1 milestone Aug 4, 2021
@amilan17

This comment has been minimized.

@amilan17

This comment has been minimized.

@amilan17

This comment has been minimized.

@amilan17
Copy link
Member Author

amilan17 commented Dec 8, 2021

@SimonElliottEUM @richardweedon Will you have time to look at this soon? I'd like to finalize this for FT.

@SimonElliottEUM
Copy link
Contributor

@amilan17 @richardweedon The updated not to 0-02-022 looks fine to me. It can be used like that for that for this FT

@amilan17
Copy link
Member Author

unresolved questions moved to specific issues

@amilan17 amilan17 removed this from the FT2022-1 milestone Dec 20, 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

3 participants