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

Alternative for pywapi #163

Closed
cmaxl opened this issue Jul 9, 2020 · 8 comments
Closed

Alternative for pywapi #163

cmaxl opened this issue Jul 9, 2020 · 8 comments

Comments

@cmaxl
Copy link
Contributor

cmaxl commented Jul 9, 2020

pywapi is not under active development (latest version from 2014) and both APIs are apparently discontinued.
I suggest using openweathermap API. Their free plan with 60 calls/minute and a total of a million per month should be sufficient. Keep in mind that the weather.com API discontinued in January 2020 and nobody ever since created an Issue here...
Any other suggestions besides openweathermap?

@FrankX0
Copy link
Contributor

FrankX0 commented Jul 10, 2020

Apparently the weather forecast is not the main usage of the clock 😃
I would be in favour of going to openweahermap. Seems to have all we need.
And then I also want to suggest to use this for sunrise/set, moon-phases, etc. So also replacing astral.
This resolves issue #162 and only requires one location in the config file.

Of course this requires quite some changes.
@bk1285 what's your opinion?

@cmaxl
Copy link
Contributor Author

cmaxl commented Jul 10, 2020

I also had something similar in mind.
How about we save the location to the env vars based on the public IP? Only if there is no user specified location (City/LatLon). Making this the default method at the very first execution of the wordclock is a nice improvement.

@FrankX0
Copy link
Contributor

FrankX0 commented Jul 10, 2020

Getting the location from the IP could be nice touch, but for me not so important.
Users need to add their API key for openweathermap to the config anyway. A small step is then to also specify the GPS location.

@bk1285
Copy link
Owner

bk1285 commented Dec 19, 2020

I'll take a closer look to openweahermap, but that all sounds very promising!

Best,
Bernd

@FabianReister
Copy link

@bk1285 see https://github.com/FabianReister/rpi_wordclock/tree/feature/omw_weather_forecast :)

This is based on @phenze 's master-python3 branch. Therefore, it won't be possible to directly open a PR for this repository (but you could cherry pick the changes).

@bk1285
Copy link
Owner

bk1285 commented Jul 24, 2021

@R0lleston
Copy link

Seems to be one of the last plugin issues to sort....would be great to have weather working

@FrankX0
Copy link
Contributor

FrankX0 commented Nov 20, 2021

Just added support for OpenWeatherMap to the develop branch.
Just in time for the merge?

@bk1285 bk1285 closed this as completed in e9c5284 Nov 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants