akx / hass-ruuvitag

Hass.io addon for Ruuvitag Weather Stations
MIT License
24 stars 7 forks source link

"Incorrect" unit of measurement in Home Assistant #1

Closed Glenf closed 5 years ago

Glenf commented 5 years ago

Home Assistant wants to have unit_of_measurement as "°C". hass-ruuvitag provides it as "C". It does work but Homekit integration doesn't show correct temperature with "C" as unit of measurement.

akx commented 5 years ago

Looks like another case of the Hass docs being vähän sinnepäin.

https://developers.home-assistant.io/docs/en/entity_sensor.html#available-device-classes says:

If specifying a device class, your sensor entity will need to also return the correct unit of measurement. [...] temperature: C/F

Glenf commented 5 years ago

Thanks, this works correctly on Homekit now. Created a PR to fix the docs on Hass end for this.

https://github.com/home-assistant/developers.home-assistant/pull/160