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

Inky Frame word clock example - swapping out NTP for Open Meteo API #747

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

martin-hamilton
Copy link

This PR swaps out the NTP call for the word clock app and replaces it with the Badger 2040W weather app code which calls the Open Meteo API, then it tries to set the RTC based on the results of the Open Meteo API call. With a following wind we should end up with the correct local time for the lat/long supplied, rather than UTC. Just a quick hack so not expecting you to commit to the repo, but might be useful somewhere!

…tead as this uses Open Meteo, which returns local time accounting for time zones, daylight savings time etc
@Gadgetoid
Copy link
Member

Thank you, currently blocked by a couple of linting issues- the CI logs have gone, but:

flake8 micropython/examples/inky_frame/inkylauncher/word_clock.py  --ignore E501
micropython/examples/inky_frame/inkylauncher/word_clock.py:43:1: W293 blank line contains whitespace
micropython/examples/inky_frame/inkylauncher/word_clock.py:61:5: F841 local variable 'e' is assigned to but never used

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