BenPru / luxtronik

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

Many sensors are not available anymore #103

Closed Albert994 closed 10 months ago

Albert994 commented 1 year ago

I updated from 2022.12.22 to current version 2023.03.24. Unfortunately many sensors (~30) are not available anymore e.g.: sensor.luxtronik2_outdoor_temperature_average sensor.luxtronik2_compressor_heating_temperature sensor.luxtronik2_pump_frequency sensor.luxtronik2_compressor_impulses sensor.luxtronik2_domenstic_water_temperature sensor.luxtronik2_heat_source_input_temperature

In Jan, there was a similar issues reported #79, but this is closed as solved.

BenPru commented 10 months ago

Should be fixed with the pre-release 2023.9.10 Thanks four your support.

Albert994 commented 10 months ago

@BenPru: I updated to this version, unfortunately the sensors are still not available.

Albert994 commented 10 months ago

I saw in the log the following error:

Logger: homeassistant.components.sensor
Source: custom_components/luxtronik2/sensor.py:1206
Integration: Sensor (documentation, issues)
First occurred: 19:18:25 (1 occurrences)
Last logged: 19:18:25

Error while setting up luxtronik2 platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 362, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/luxtronik2/sensor.py", line 531, in async_setup_entry
    add_sensor_if_active(luxtronik, entities, "visibilities.ID_Visi_Waermemenge_ZWE", LuxtronikSensor(
  File "/config/custom_components/luxtronik2/sensor.py", line 1206, in add_sensor_if_active
    if luxtronik.get_value(check_key) <= 0:
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: '<=' not supported between instances of 'NoneType' and 'int'

I commented out 2 lines in sensor.py:

def add_sensor_if_active(luxtronik, entities, check_key: str, sensor: LuxtronikSensor):
#    if luxtronik.get_value(check_key) <= 0:
#        sensor.disable_by_default()
    entities += [sensor]

Now most of the sensors are back again. Only 4 are still missing

BenPru commented 10 months ago

Please try the current main branch.

Albert994 commented 10 months ago

I installed the main branch. It's equivalent to the patch I did.

The following sensors are still not available: binary_sensor.luxtronik2_circulation_pump binary_sensor.luxtronik2_circulation_pump_domestic_water sensor.luxtronik2_pump_frequency sensor.luxtronik2_heat_source_output_temperature

I'm getting the following warning in the Protocol, but I don't know if this has something to do with the issue:

Dieser Fehler wurde von einer benutzerdefinierten Integration verursacht

Logger: py.warnings
Source: custom_components/luxtronik2/climate.py:254
Integration: Luxtronik
First occurred: 11. September 2023 um 00:22:24 (4 occurrences)
Last logged: 15:29:29

/config/custom_components/luxtronik2/climate.py:254: RuntimeWarning: coroutine 'LuxtronikThermostat._async_control_heating' was never awaited self._async_control_heating()
miktin commented 10 months ago

Same hear

BenPru commented 10 months ago

@miktin and @Albert994

You use old tags. Please read the Breaking Changes in the changelog: 2023.01.04

This properties have no big impact for most of the users. So thay are disabled by default. You can open your ha heatpump device, scroll down the sensors list, click '+n entities not shown', search and click 'Pump frequency', click settings and enable it.

sensor.luxtronik2_heat_source_output_temperature is also only visible if 291 visibilities.ID_Visi_LIN_UH has a value. You can check this via diagnostic file: image

Please don't create tickets without reading the changelog and please provide a diagnostic file in tickets.

miktin commented 10 months ago

It works many month. After last update from HA have this problems.