hassio-addons / addon-prometheus

Prometheus - Home Assistant Community Add-ons
https://addons.community
MIT License
21 stars 32 forks source link

Temperatures always say "sensor_unit_c", even if in F #4

Closed ahknight closed 4 years ago

ahknight commented 4 years ago

Problem/Motivation

In America, we use Freedom units. /s

Expected behavior

The software should respect my Gawd-given right to use the wrong unit.

Actual behavior

It logged the correct numerical value (say, 90) but logged it under sensor_unit_c instead of sensor_unit_f where I'd expect it.

Steps to reproduce

Use Freedom units!

Proposed changes

(If you have a proposed change, workaround or fix, describe the rationale behind it)

mjrider commented 4 years ago

This would be an issue with the prometheus integration, the exporter, in home-assistant and not the addon,

Of course the correct solution would be using SI units