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

METAR Parsing for Precipitation Accumulation and Remarks Section #1727

Open
DirkMPetersen opened this issue Feb 23, 2021 · 11 comments
Open

METAR Parsing for Precipitation Accumulation and Remarks Section #1727

DirkMPetersen opened this issue Feb 23, 2021 · 11 comments
Labels
Area: IO Pertains to reading data Type: Feature New functionality

Comments

@DirkMPetersen
Copy link

Hello there! I have been using this module to parse METAR data and would like to suggest the addition of parsing precipitation data and and other miscellaneous objects in the remarks section of a METAR into the "metar" part of the module.

Parsing METARs into CSV's helps make working with the data that are contained in them much less intimidating, and fleshing out the parsing capabilities of this module would make this the no-brainer module to use for this purpose.

Here is an example:
5-MIN KOMA 190550Z 16005KT 10SM -FZDZ SCT019 OVC026 M01/M03 A3015 770 88 -900 150/05 RMK AO2 SLP219 FZDZB44 P0000 60000 I1000 I6000 T10111028 10006 21039 400281056 51018

Very useful information located within the "RMK" (or remarks) section include the following items:
FZDZB44 -> Freezing drizzle began at 0544Z
P0000 -> The amount of precipitation the has occurred since the last METAR is considered a trace.
I1000 -> 1-hr Ice accretion of a trace
I6000 -> 6-hr Ice accretion of a trace

Thanks for doing what you guys do! Like I said the amount of documentation with this python module is great, and fleshing this feature would make the the "go-to" METAR parser for python.

@DirkMPetersen DirkMPetersen added the Type: Feature New functionality label Feb 23, 2021
@dopplershift dopplershift added the Area: IO Pertains to reading data label Mar 2, 2021
@dopplershift
Copy link
Member

We definitely need to make sure the full remarks section is available in the decoded output--the fact that it's not there right now is a major oversight--borderline bug.

Decoding some blocks would be great. The only one of those mentioned above that gives me pause is the FZDZB44 because I'm not at all sure how I would represent that decoded into a column in a DataFrame.

@DirkMPetersen
Copy link
Author

Perhaps that particular block could just have a column of its own, with the label of "precipitation onset time" and also be paired with " precipitation end time? It would only need to be filled out when those are included into the remarks section, and could be null when the aren't included.

FZDZE04 FZDZB44 is still very useful information, and just having their own column would be a plus with start time and end time as the names. I know a particular application that this would be great for is using continuous freezing precipitation for ice accretion research.

@dopplershift
Copy link
Member

What would you want that to look like when there are multiple such periods, sometimes with multiple precipitation types?

@DirkMPetersen
Copy link
Author

DirkMPetersen commented Mar 3, 2021 via email

@eliteuser26
Copy link
Contributor

In issue #1306 I indicated that visibility and remarks were not part of the decoded Metar dataframe. Should I work on that issue by submitting a PR or wait for the code to be modified in the parsing section which decodes Metar reports?

@dopplershift
Copy link
Member

@eliteuser26 PRs are always welcome (and I'm trying to prioritize getting the ones we have reviewed). I can say with certainty that addressing this issue/#1306 is not in our immediate plans.

@eliteuser26
Copy link
Contributor

This is good to know. At least I can do some test on my side to see if I can add visibility and remarks in the code. This way I won't need to submit it just yet until it becomes a higher priority.

@mzuranski
Copy link

When it comes to parsing SLP out of remarks, would it be possible to have that populate the air_pressure_at_sea_level field?

Please correct me if I'm wrong on any of this, but right now air_pressure_at_sea_level is derived from the altimeter. The problem with that method is it assumes a standard atmosphere, and I've been noticing that result can vary significantly from SLP in the same observation. Obtaining SLP directly from the METAR would be ideal, but then we'd also need a way to toggle that I expect; we wouldn't want air_pressure_at_sea_level to be populated from a mix of both sources in the same DataFrame.

@mzuranski
Copy link

Similarly to SLP, could values from the T group populate the air_temperature and dew_point_temperature fields if it is available in the remarks?

Having a mix of sources for temp & dewpoint probably isn't as big of a deal as SLP, but the extra precision from the T group would be nice.

@dopplershift
Copy link
Member

@mzuranski I would be fine with both of your suggestions.

@sgdecker
Copy link
Contributor

I second all of @mzuranski 's suggestions, and, once parsing of SLP remarks is included, there should be an option to set air_pressure_at_sea_level to NaN if there is no SLP remark.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: IO Pertains to reading data Type: Feature New functionality
Projects
None yet
Development

No branches or pull requests

5 participants