search

rcasula / kostalpiko-homeassistant

19 stars 15 forks source link

Make integration compatible with Home Assistant 2021.8, 2021.9 and the energy dashboard. #2

Closed Lemocin closed 2 years ago

Lemocin commented 2 years ago

This component was not compatible with the up-to-date Home Assistant versions. I added the version in the manifest file which closes #1. I also made the component much faster by only making one http request for all sensors instead of one request per sensor.

rcasula commented 2 years ago

Hi, sorry for the late response! I've just tested your work and seems fine, I will merge this Thanks for your work!

pergolafabio commented 1 year ago

Hi @Lemocin , is it possible to change the code again for last reset ? we have error below

I prefer to use daily energy to update the energy dashboard, cause its with decimals... The total energy doesnt update that frequently, especially on winter period

thx

2023-01-10 09:18:05.021 WARNING (MainThread) [homeassistant.components.sensor] Entity sensor.kostal_piko_daily_energy (<class 'custom_components.kostal.sensor.PikoSensor'>) with state_class total_increasing has set last_reset. Setting last_reset for entities with state_class other than 'total' is not supported. Please update your configuration if state_class is manually configured, otherwise report it to the custom integration author.

Lemocin commented 1 year ago

Hey, sorry I have been busy this week. I no longer have access to a Kostal Pico Inverter because my friend who had it moved away. I might be able to look into it later but you would have to test it yourself.

Lemocin commented 1 year ago

Are there any other log entries @pergolafabio ? Because that one is just a warning

pergolafabio commented 1 year ago

Hi, no problem, already found another component :-)