fsaris / home-assistant-zonneplan-one

Unofficial Zonneplan integration for Home Assistant
https://github.com/fsaris/home-assistant-zonneplan-one
MIT License
104 stars 17 forks source link

New costs sensors have wrong state_class #90

Closed hmmbob closed 9 months ago

hmmbob commented 9 months ago

Running 0.0.22

Entity sensor.electricity_delivery_costs_today (<class 'custom_components.zonneplan_one.sensor.ZonneplanP1Sensor'>) is using state class 'total_increasing' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/fsaris/home-assistant-zonneplan-one/issues
Entity sensor.electricity_production_costs_today (<class 'custom_components.zonneplan_one.sensor.ZonneplanP1Sensor'>) is using state class 'total_increasing' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/fsaris/home-assistant-zonneplan-one/issues
Entity sensor.gas_delivery_costs_today (<class 'custom_components.zonneplan_one.sensor.ZonneplanP1Sensor'>) is using state class 'total_increasing' which is impossible considering device class ('monetary') it is using; expected None or one of 'total'; Please update your configuration if your entity is manually configured, otherwise create a bug report at https://github.com/fsaris/home-assistant-zonneplan-one/issues
fsaris commented 9 months ago

@hmmbob thanks for the feedback. Did some investigation (found for example https://github.com/home-assistant/core/issues/88457) and need to set it to total and set last_reset on the start of each day.

Looks that also to wrong value provided for unit_of_measurement. It should be EUR instead of .

fsaris commented 9 months ago

Sensors are fixed in https://github.com/fsaris/home-assistant-zonneplan-one/releases/tag/0.0.23

Please make sure to update the existing statistics see release note

Open your Home Assistant instance and show your statistics developer tools.

hmmbob commented 9 months ago

Works, thanks Frans!

MathijsG commented 8 months ago

What to change on my end? I can't seem to change any unit on these sensors?

image image

fsaris commented 8 months ago

Seems you don't have the sensors enabled so no need to update anything