search

mawinkler / astroweather

Asynchronous Astro Weather Forecast for Home Assistant
GNU General Public License v3.0
79 stars 9 forks source link

unit of wind speed does not update the value when changing #33

Closed el97 closed 1 year ago

el97 commented 1 year ago

The value of wind speed does not change. It looks like the value isn't changing no matter if I choose km/h, mp/h or m/s. I compared with the met.no integration and it looks like astroweather only shows the unit in km/h.

mawinkler commented 1 year ago

It's m/s on my side. Where do you choose/change the unit?

image image

Screenshots are from my current core redesign, work in progress...

el97 commented 1 year ago

Oh, I see what I did wrong here... I changed the unit on the weather entity, not the sensor.astroweather_10m_wind_speed. It would be great if you removed the options you can make on the weather entity to avoid confusion.

mawinkler commented 1 year ago

So, changing the units in a weather entity is something new in home assistant. I don't really know how to deal with this or how to disable this. AstroWeather sets the native units for the sensors and the card checks for the configured unit in the system. As an example: if the system configured length unit is km the wind speed will be m/s. If length is not km I calculate mph from the internally used m/s. This means changing the wind speed unit in the weather entity does not have an effect.

Maybe check the latest release v0.31.0.

mawinkler commented 1 year ago

Closing as this should be fixed with the above version. If it's not resolved, please let me know. 👍