mmp / vice

Virtual air traffic control simulator
https://pharr.org/vice
GNU General Public License v3.0
23 stars 33 forks source link

Fix live weather #168

Closed checkandmate1 closed 6 months ago

mmp commented 6 months ago

Not sure if it's overkill but using something like https://github.com/urkk/metar for the METAR parsing could be worthwhile.

checkandmate1 commented 6 months ago

What only parses a METAR, it doesn't get you the METAR from an ICAO code correct?

mmp commented 6 months ago

Yes, I believe so.

checkandmate1 commented 6 months ago

The only METAR parsing that is going on is finding the altimeter. (For some reason it's in HPA and not INHG). I don't think there is a reason to parse the already parsed METAR. If I'm overlooking something please let me know!

mmp commented 6 months ago

Ah, got it, sorry. I was thinking the crash was from parsing the wind from the METAR. So nevermind..