Open louisvarley opened 1 year ago
We sure would object to a PR making the OWM HTTP timeout - that's what the oddly named lostTest
really is - available as a user setting with a sensible default.
We sure would object
Oh bummer, only just noticed. That should of course be "would NOT object".
;) I did wonder. I was like "oh... ok"
I did make the change to hard code it on my weather station but Ill see what I can do.
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg
From: Marcel Stör @.> Sent: Thursday, January 19, 2023 7:56:25 PM To: ThingPulse/esp8266-weather-station @.> Cc: SEALTeamRicks @.>; Author @.> Subject: Re: [ThingPulse/esp8266-weather-station] [HTTP] lost in client with a timeout (Issue #209)
We sure would object
Oh bummer, only just noticed. That should of course be "would NOT object".
— Reply to this email directly, view it on GitHubhttps://github.com/ThingPulse/esp8266-weather-station/issues/209#issuecomment-1397523771, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHHYVZ3B3PIAKEZTBD62W7TWTGL6TANCNFSM6AAAAAAS2ORTRI. You are receiving this because you authored the thread.Message ID: @.***>
Expected behavior
Downloading of Forecast Data
Actual behavior
I've been using this library in a project for some time, unsure if due to a change at OpenWeather but suddenly i get [HTTP] lost in client with a timeout and box restarts,
Test code
My Get Data Code here, current weather is fine, but Forecast will always fail, i'm only doing a max of 3 days
Changing this in OpenWeatherMapForecast.cpp and it runs as expected again
I know that the speed it will be downloading the JSON response would be due to speed of WiFi / Internet, i have Ubiquti APs, Fibre and ESP8266 is in the same room as an AP. Started very suddenly about a week ago
Maybe make it so we can change this value outside of the library?
Weather Station version
Master
Hardware
Custom Hardware, a Clone ESP8266, with a matrix display