Open Meteo Widget - wind speed in m/s

[marcinbjorn]

What is the request?

An option to set the wind speed unit to m/s.

Why is it useful?

Personal preference: Iceland commonly uses m/s for wind speed in weather forecasts, it's also the standard scientific and meteorological unit.

How could it be implemented?

Open-Meteo accepts wind_speed_unit=ms as an API call argument
All available units: kmh, ms, mph, kn

A new option for the widget could be added to config.yaml

open_meteo:
  options:
   # Units: ms, mph, kmh, kn
    wind_speed_unit: ms

In src.core.widgets.services.open_meteo file api.py, the wind speed unit could then be selected from a user-defined configuration value. If no user-defined value is provided, the existing logic could remain as the fallback:

# Line 63: wind_unit = "mph" if units == "imperial" else "kmh"
if user_wind_unit is not None:
    wind_unit = user_wind_unit
else:
    wind_unit = "mph" if units == "imperial" else "kmh"

Similar changes in files: widgets.py lines 445 and open_meteo.py line 808

This is not a full implementation, only an overview. I hardcoded m/s as wind unit, and it works for me.

[marcinbjorn]

I implemented changes by adding a new option to config.yaml:

wind_speed_unit: "ms" # Options: "ms", "kmh", "mph", "kn"

and created a new tab that displays a wind-related graph with gust speed. How can I contribute this to the project?

[amnweb]

https://github.com/amnweb/yasb/wiki/Contributing
But you should wait with a new PR. We won’t merge anything now because we are working to clean up the codebase and implement some new features. After that, PRs will be accepted.

[marcinbjorn]

I'll do that, thanks.