search

ksheumaker / homeassistant-apsystems_ecur

Home Assistant custom component for local querying of APSystems ECU-R Solar System
Apache License 2.0
166 stars 42 forks source link

Change today_energy sensor stateclass back to TOTAL_INCREASING #175

Closed renini closed 1 year ago

renini commented 1 year ago

According to the HA docs and the errors seen in issue #174 :

Similar to total, with the restriction that the state represents a monotonically increasing positive total which periodically restarts counting from 0, e.g. a daily amount of consumed gas, weekly water consumption or lifetime energy consumption.

Seems to be the correct stateclass to use. This commit to change this back from TOTAL to TOTAL_INCREASING.

IvanSanchez commented 1 year ago

I seem to have introduced this in #170. Reading the docs once again, it seems that it should be TOTAL_INCREASING indeed.

HAEdwin commented 1 year ago

No problem, thank you for notifying.