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

Long term statistics for inverter sensors #164

Closed mcflexus closed 1 year ago

mcflexus commented 1 year ago

I see the ecu sensors have long term statistics enabled but the inverter sensors don't. Can you enable them? I think especially for temperature sensor it makes much sense to do so. Thank you for this great integration!

HAEdwin commented 1 year ago

They are long term (at least when I quick tested it) image Or do you mean something else?

mcflexus commented 1 year ago

I mean something else, this is just the history of a few days, long term statistics are stored for ever. See https://data.home-assistant.io/docs/statistics/

HAEdwin commented 1 year ago

I see what you mean, state_class is missing. Are you able to add that to the integration?

mcflexus commented 1 year ago

Don't think that should be a problem. I'll try to find some time this weekend.

mcflexus commented 1 year ago

just made a pull request, tested locally and working as expected. #165

HAEdwin commented 1 year ago

Merged the pull request, thank you!

mcflexus commented 1 year ago

No problem!