Skip to content
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

Added weather mode #21

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Added weather mode #21

wants to merge 14 commits into from

Conversation

cosmos1978
Copy link

you need to configure your openweathermap api key, city name and country code in the main ino file.
Perhaps the saving can be done better and more inline but the other mqtt component.
Also the json output is very large and because I use HTTPclient.getString it will kill the memory if I make a larger request.
so what can be improved is the json parser. Perhaps it was better to use jsonserializer. second I need to find a way to parse and store large json objects

cosmos1978 added 5 commits May 7, 2020 20:43
live weather effect based on openweathermap api
bug fix in mqtt effects list
reduced the number of forecast options
added whlsers config for mqtt via webserver
@vortigont
Copy link

You can use '/data/2.5/weather?' API call, it gives only current weather, but it is less than 500 bytes bytes long for one city.

Using arduinojson to parse fields from json api response.
show a weather report on the weather page.
Show scrolling temp on lamp while in weather mode
@cosmos1978
Copy link
Author

added weather report on webpage.
added scrolling temperature text on weather effects. ( number is rounded to closest decimal value )
using arduinojson now for all non string values.

changed font colors in weather mode effects
shortened descriptions in tab pages
bugfix to weather plugin.
all functions work as expected now
placed a comment wrong.
Fixed now
lowered memory footprint significantly by using arduinojson streaming and filtering.
Full 4 days forecast in a much smaller footprint.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants