-
Notifications
You must be signed in to change notification settings - Fork 211
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
OneCall updates #256
Comments
I now get most responses on the serial monitor with the exception of rain. Cleaned up and deleted my api key. Ihaven't tried this on a display, yet |
Well done, all that looks OK. |
It was raining most of the day yesterday. I wasn't getting any reading on the serial monitor. |
For some reason the json library decoder isn’t always converting some variables to type float, so this accounts for the 0 values, it’s almost certainly choosing type integers instead of floats. I’ll look at the json source code to see what’s happening. |
You’ll need to use the directives .as(); to convert some of the json data to the correct variable type. So for the two errant rain and s is types use .as(); or .as(); I see the OneCall does not provide hourly high/low temperatures and only 48-hours of the rest, plus daily values are limited to 7-days, so actually quite a lot of change and a lot has to change. I’m still thinking through how to make the transition as easy as possible. |
I did try to update to the new onecall API, but received all 0's for a reply. I did decipher some of the data coming back, and making changed to common.h for this, but I am not proficient enough to finish. here is what I have done so far:
under
" if (Type == "weather") { "
the roots had to change, some from main to current, or some root had to be added
under the forecast, change root["list"] to root["hourly"]
I am having issues how to deal with forecasts with the new api
common.txt
The text was updated successfully, but these errors were encountered: