search

kamaradclimber / heishamon-homeassistant

An integration for heatpumps handled by heishamon
Apache License 2.0
44 stars 24 forks source link

Integration/Energy Sensors suddenly gone #222

Closed b3nn0 closed 3 months ago

b3nn0 commented 3 months ago

Versions

Describe the bug Hi, I just noticed that all the total-energy sensors have suddenly become unavailable, and HA's statistics repair tool says their unit changed from kWh to ''. During startup, I can see this message in the log, which seems to be the reason for this. I think this was caused by the HA 2024.07 update.

2024-07-05 08:25:36.191 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up aquarea platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 364, in _async_setup_platform
    await asyncio.shield(awaitable)
  File "/config/custom_components/aquarea/sensor.py", line 188, in async_setup_entry
    integration_sensors.append(EnergyIntegrationEntity(
                               ^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: IntegrationSensor.__init__() missing 1 required keyword-only argument: 'max_sub_interval'

Screenshots Sorry, renamed the entities a bit, it's just the ones created by the integration. image

See also: https://github.com/nathanmarlor/foxess_modbus/discussions/635 https://github.com/home-assistant/core/pull/110685

kamaradclimber commented 3 months ago

Good catch, thanks for the report. It affects only HA 2024.7 and I've fixed this in 1.9.2

wltng commented 3 months ago

Thanks for the fix. Although I am running HA 2024.7.0, trying to update results in an error stating my HA version is incompatible.

b3nn0 commented 3 months ago

Looks like a bit of an unfortunate string comparison mismatch..

Du benutzt die Home Assistant-Version '2024.7.0', für dieses Repository muss jedoch die Mindestversion '2024.7' installiert sein.