JohnMarzulli / categorical-sectional

Version of the light-up-map that works with "individually addressable" string lights based on the WS2801.
28 stars 17 forks source link

Found issue (in logs) that caused remarks with partial reports to cau… #14

Closed JohnMarzulli closed 6 years ago

JohnMarzulli commented 6 years ago

Fixes https://github.com/JohnMarzulli/categorical-sectional/issues/13

Reviewing logs found that this METAR caused an exception while being classified for ceiling, and thus was classified as "INVALID".

KVOK 251453Z 34004KT 10SM SCT008 OVC019 21/21 A2988 RMK AO2A SCT V BKN SLP119 53012 The source is that the remarks section had a SCT and BKN token without a value.

To avoid this:

Skip over RMK since they do not represent the current condition in regards to our classification. Wrap the ceiling parsing into a try/except block and skip over in case of an error.