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

Conversion of Current Wind Speed from m/s to mph #34

Open
g0ods opened this issue Oct 4, 2020 · 0 comments
Open

Conversion of Current Wind Speed from m/s to mph #34

g0ods opened this issue Oct 4, 2020 · 0 comments

Comments

@g0ods
Copy link

g0ods commented Oct 4, 2020

Hi there,
I'm new to GitHub so am not sure if this is the best way to raise a bug report. In my understanding wind speed from Weatherbit.IO comes as m/s, which to convert to km/h you need to multiply by 3.6, similarly to convert from m/s to mph you need to multiply by 2.24, so in WeatherPiTFT.py the following line, line #731
wind_speed = wind_speed * 3.6 if METRIC else wind_speed
perhaps should be something like
wind_speed = wind_speed * 3.6 if METRIC else wind_speed * 2.2

BTW I love your work on this project,
g0ods.

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

No branches or pull requests

1 participant