BenPru / luxtronik

Luxtronik integration for Home Assistant
MIT License
68 stars 23 forks source link

No Solar Buffer Temperature & Solar Collector #62

Closed marcelhen closed 1 year ago

marcelhen commented 1 year ago

Hi. I currently have the software 2022.11.20 running with me and everything is fine with that.
When I make a newer update, I no longer see the temperatures from the solar collector and from the buffer storage and the status from the solar pump. Do you have an idea why that is?

BenPru commented 1 year ago

LUX_DETECT_SOLAR_SENSOR: Final = "parameters.ID_BSTD_Solar" 

parameters.ID_BSTD_Solar seems to be not set in your config.

BenPru commented 1 year ago

@marcelhen Please try the current release 2023.01.05. I have changed the detection but I think its not the final solution. Please export a diagnostic file after update and provide it as file attachment in this issue: You can export it Settings => Devices & Services => scroll to Luxtronik => ... => Download diagnostics: image

marcelhen commented 1 year ago

Hello. I made the update 2023.01.05. Everything is fine, the system finds all sun sensors!!!
But now I have the problem that is listed under the integration 2 heat pumps with the same serial number.

BenPru commented 1 year ago

But now I have the problem that is listed under the integration 2 heat pumps with the same serial number.

Please provide a diagnostic file.

Kars-de-Jong commented 1 year ago

I had the same issue after upgrading: all devices were duplicated.

I ended up manually editing config/.storage/core.entity_registry and config/.storage/core.device_registry to remove the old entries.

Probably should have just removed/re-added the integration instead.

Oh, I also don't think the integration will work with HA 2023.1 - some constants related to units were removed (you need to use enumerations now).

BenPru commented 1 year ago

Probably should have just removed/re-added the integration instead.

But ist the sensor history lost after that?

Have you 2023.01.05 or 2023.01.05a?

Kars-de-Jong commented 1 year ago

I don't think so because removing an integration doesn't remove anything from the database as far as I know. And because it uses entity id as key, once you re-add the entities and they have the same entity id, they will just start updating the database again too.

I use 2023.01.05.

BenPru commented 1 year ago

I have marked 2023.01.05 as pre-release until I have fixed the double devices.

BenPru commented 1 year ago

I think its corrected with pre-release 2023.01.07. I have added a config migration to V2 which deletes old devices, change the integration and device unique ids to the serial number. Know I knew that the serial number is unique over all luxtronik heatpumps. nnnnnn-nnn First six are the serial number, last 3 are the model id.

marcelhen commented 1 year ago

Hi. Since yesterday, Home Assistant has stopped recording data. The data is only retrieved once after a restart.

148A9B7B-46A5-48E2-B64E-D3CDE786AC99

BenPru commented 1 year ago

Moved to own issue: #65

BenPru commented 1 year ago

Hi. Since yesterday, Home Assistant has stopped recording data. The data is only retrieved once after a restart.

I think its fixed with 2023.01.15

BenPru commented 1 year ago

I think the original issue has been resolved. If not, please re-open again.