Expandable-6-Channel-ESP32-Energy-Meter icon indicating copy to clipboard operation
Expandable-6-Channel-ESP32-Energy-Meter copied to clipboard

Update YAMLs to conform to Home Assistant developer guidelines?

Open pyrodex opened this issue 9 months ago • 3 comments

Hello I am seeing the following errors in my HA logs:

2023-09-14 20:04:05.616 WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.house_energy_monitor_02_total_watts (<class 'homeassistant.components.esphome.sensor.EsphomeSensor'>) is using native unit of measurement 'W' which is not a valid unit for the device class ('energy') it is using; expected one of ['MJ', 'GJ', 'kWh', 'Wh', 'MWh']; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/home-assistant/core/issues?q=is%3Aopen+is%3Aissue+label%3A%22integration%3A+esphome%22

Should the ESPHome YAML be adjusted to correct these type of invalid measurement types?

pyrodex avatar Sep 15 '23 00:09 pyrodex

You're right, the device class should be power and not energy. I'm not using those template at the moment. Could you try to change device_class : energy to device_class: power and install the new version to confirm ?

Once done, we could ping @CircuitSetup to modify this or create a PR (dunno how to do this, sorry)

PandemiK911 avatar Oct 10 '23 07:10 PandemiK911