WKHarmon / led-sectional

MIT License
34 stars 18 forks source link

Last airport ignored #30

Open ayryq opened 19 hours ago

ayryq commented 19 hours ago

I'm using the jcl-rv12is fork and currently have reverted to the led_sectional.ino file as published, except I changed pin number to 14. I'm not seeing the LED updated for the final airport (K11R) It does appear in the url string:

Starting connection to server...
Connected ...
GET /api/data/metar?format=xml&hoursBeforeNow=3&mostRecentForEachStation=true&ids=KGLS,KT41,KEFD,KHOU,KLVJ,KAXH,KSGR,KLBX,KBYY,KARM,KELA,KTME,KDWH,KIAH,KT78,K6R3,KCXO,KUTS,KCFD,KCLL,K60R,K11R HTTP/1.1
Host: aviationweather.gov
User-Agent: LED Map Client
Connection: close

But does not appear in the results list

[...]
KCLL: VFR 8G20kts LED 22 WX: KCLL 261653Z 04008G20KT 10SM CLR 29/16 A2992 RMK AO2 SLP127 T02890156
... found very high winds or gusts!
K60R:  0G0kts LED 23 WX: K60R 261735Z AUTO 35011KT 8SM SCT050 29/17 A2987 RMK AO2 SLP031 T02940171 $
Refreshing LEDs.

I came at this issue from using my own airports, and a set of 50, and the last one was never included—but I'm seeing the same issue with the pre-configured set.

I'm (currently) running FastLED 3.3.3 and esp8266 3.1.2.

ayryq commented 18 hours ago

As a workaround it's simple to add a dummy airport to the end and increase NUM_AIRPORTS by one. In addition to the dummy airport I'm adding at the beginning since my first LED doesn't behave.

ayryq commented 16 hours ago

Also, and this might require some more testing, I'm seeing weird results from the last displayed airport (i.e. the second-to-last programmed airport, since I added a dummy at the very end).

CYTZ:  0G0kts LED 49 WX: CYTZ 262000Z AUTO 20009KT 9SM CLR 22/16 A3001 RMK SLP162 DENSITY ALT 1100FT

There's wind data, 20009kt, so why is it reporting 0G0kts, and no flight category? I've done some swapping around and it's always LED 49 (number 50 of 51)