search

jdhartley / terminal-for-fitbit

Terminal is a Fitbit clock face styled to match a terminal session in a Unix environment
https://terminal.watch/
The Unlicense
134 stars 18 forks source link

feature request - add weather as screen data #24

Open muymanwell opened 3 years ago

muymanwell commented 3 years ago

it would be wonderful if weather could be added as an option (temp/current weather type) for the face

jdhartley commented 3 years ago

Hi @muymanwell, thanks for the suggestion! I'd love to bring this feature the Terminal in a future update.

jensenak commented 3 years ago

Hi, @jdhartley, are you still maintaining this project? I'd be happy to open a PR to add a weather dataline if it's something you'd be willing to publish.

jdhartley commented 3 years ago

hi @jensenak, yes I am still here to publish updates. I have not worked on this project much since the latest fitbit SDK fracture, but am still wearing this clock face every day!

I believe that the weather data can be pulled straight from the Fitbit SDK now*, so this change is hopefully fairly easy to add in.

if you want to open a PR adding this in to the sdk5 branch, I would love to test and get a build going! https://dev.fitbit.com/build/reference/companion-api/weather/

*note that this would mean this feature is only available on Sense and Versa 3 devices. as of now, i am not too interested in updating the sdk4 branch with HTTP requests

jensenak commented 3 years ago

Sounds like a plan. I have a Versa 3 and that companion api looks pretty easy to work with. I'll check out the sdk5 branch. Might be a couple days on the PR, since I've got other stuff in progress as well.

jensenak commented 3 years ago

Forgot to ask, does the name WTHR sound okay for the data line? Or maybe TEMP?

jdhartley commented 3 years ago

If we are just showing the temperature, lets go with TEMP, e.g.

[TEMP] 70º

Though as I am typing, I doubt think we have a degrees symbol in the fitfont folders right now. We could generate those (I think the readme talks about how), but if we want to just put 6 Celsius or 42 Fahrenheit that is ok with me. If you open a PR I can find some time to help generate those glyphs and maybe it can be an option for Full Unit / Degrees Symbol.

[TEMP] 31 Celsius
[TEMP] -3 Fahrenheit

For WTHR I would love to find a way to get WeatherCondition in there too like 70º Rainy, but looking at the enum list, oof... If you have thoughts on how to fit PartlySunnyWithThunderstormsDay in a dataline, very open to ideas 😳