pail23 / stiebel_eltron_isg_component

Stiebel Eltron ISG integration for Home Assistant
MIT License
43 stars 19 forks source link

Several sensors not working with HA Core 2024.1.2 #168

Closed Arne1346 closed 5 months ago

Arne1346 commented 5 months ago

After upgrading to HA Core 2024.1.2 several sensors no longer provide data - e.g.

Stiebel Eltron ISG Actual Temperature HK 1
Stiebel Eltron ISG Outdoor Temperature
Stiebel Eltron ISG Flow Temperature
Stiebel Eltron ISG Actual Temperature Buffer
Stiebel Eltron ISG Actual Temperature Water

and more.

Others (e.g. Stiebel Eltron ISG Is heating, Stiebel Eltron ISG Heating Curve Rise HK1, Stiebel Eltron ISG Compressor) continued working properly.

After restoring backup and downgraded to HA Core 2023.12.4 all sensors are working again properly

pail23 commented 5 months ago

Hi @Arne1346

Which version of the Stiebel Eltron component do you use? Would you mind upload the diagnostic information? This would help investigating the issue.

werpu commented 5 months ago

I have the same issue, i checked the logs and some internal api seems to have broken the sensor read, I see following errors:

fetching Stiebel Eltron ISG data in 0.012 seconds (success: True) 2024-01-11 08:40:53.533 DEBUG (MainThread) [custom_components.stiebel_eltron_isg] Finished fetching Stiebel Eltron ISG data in 0.011 seconds (success: True) 2024-01-11 08:43:37.835 DEBUG (MainThread) [custom_components.stiebel_eltron_isg] Finished fetching Stiebel Eltron ISG data in 0.011 seconds (success: True) 2024-01-11 08:43:37.848 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up stiebel_eltron_isg platform for sensor Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 360, in _async_setup_platform await asyncio.shield(task) File "/config/custom_components/stiebel_eltron_isg/sensor.py", line 365, in async_setup_entry description.state_class = STATE_CLASS_TOTAL_INCREASING ^^^^^^^^^^^^^^^^^^^^^^^ File "", line 4, in setattr dataclasses.FrozenInstanceError: cannot assign to field 'state_class' 2024-01-11 08:47:56.301 DEBUG (MainThread) [custom_components.stiebel_eltron_isg] Finished fetching Stiebel Eltron ISG data in 0.032 seconds (success: True) 2024-01-11 08:47:56.336 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up stiebel_eltron_isg platform for sensor Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 360, in _async_setup_platform await asyncio.shield(task) File "/config/custom_components/stiebel_eltron_isg/sensor.py", line 365, in async_setup_entry description.state_class = STATE_CLASS_TOTAL_INCREASING ^^^^^^^^^^^^^^^^^^^^^^^ File "", line 4, in setattr dataclasses.FrozenInstanceError: cannot assign to field 'state_class'

Seems like an attribute has changed!

Btw only the sensors are broken atm, the rest works fine, and so far only the Stiebel Eltron Plugin is broken nothing else! Mabe a deprecated attribute?

Also two warnings can be found: Logger: homeassistant.const Source: helpers/deprecation.py:205 First occurred: 10. Januar 2024 um 17:47:51 (2 occurrences) Last logged: 10. Januar 2024 um 17:47:51

DEVICE_CLASS_ENERGY was used from stiebel_eltron_isg, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorDeviceClass.ENERGY instead, please create a bug report at https://github.com/pail23/stiebel_eltron_isg_component/issues ENERGY_KILO_WATT_HOUR was used from stiebel_eltron_isg, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfEnergy.KILO_WATT_HOUR instead, please create a bug report at https://github.com/pail23/stiebel_eltron_isg_component/issues

Version of the plugin, latest version installed via HACS (1.1.2024) I did a remove and reinstall just to make sure the problem persists, it still is there, the main difference after reinstalling is that the sensors are not displayed at all anymore because they are excluded now after failing!

PS: Small question is there a place where I can see the current version of the component, I was searching around in the ui and could not find it!

pail23 commented 5 months ago

In order to see the version, you need to download the diagnosis information:

image

When you open the file and search for stiebel_eltron, you will find something like this:

    "stiebel_eltron_isg": {
      "version": "2024.1.1",
      "requirements": [
        "pymodbus>=3.5.1"
      ]
    },

It would be very helpful if you could upload the file here.

gargar69 commented 5 months ago

Below is the log error from Home Assistant. It's all the sensors. The device class used has been deprecated.

Logger: homeassistant.const Source: helpers/deprecation.py:205 First occurred: 13:40:45 (2 occurrences) Last logged: 13:40:45

DEVICE_CLASS_ENERGY was used from stiebel_eltron_isg, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorDeviceClass.ENERGY instead, please create a bug report at https://github.com/pail23/stiebel_eltron_isg/issues ENERGY_KILO_WATT_HOUR was used from stiebel_eltron_isg, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfEnergy.KILO_WATT_HOUR instead, please create a bug report at https://github.com/pail23/stiebel_eltron_isg/issues

pail23 commented 5 months ago

This error has been already fixed in 2024.1.1. In case you still see this error into the latest version I would appreciate if you could post your diagnosis information here. Thanks.

gargar69 commented 5 months ago

Thanks Paul. Nice job, very prompt. It worked first time. GAR

On Thu, 11 Jan 2024 at 19:06, Paul Frank @.***> wrote:

This error has been already fixed in 2024.1.1. In case you still see this error into the latest version I would appreciate if you could post your diagnosis information here. Thanks.

— Reply to this email directly, view it on GitHub https://github.com/pail23/stiebel_eltron_isg_component/issues/168#issuecomment-1887786131, or unsubscribe https://github.com/notifications/unsubscribe-auth/BAHNRVQTHZIRVP34C7CWUODYOAZV3AVCNFSM6AAAAABBRI3QTWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBXG44DMMJTGE . You are receiving this because you commented.Message ID: @.> [ { @.": "http://schema.org", @.": "EmailMessage", "potentialAction": { @.": "ViewAction", "target": " https://github.com/pail23/stiebel_eltron_isg_component/issues/168#issuecomment-1887786131", "url": " https://github.com/pail23/stiebel_eltron_isg_component/issues/168#issuecomment-1887786131", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { @.***": "Organization", "name": "GitHub", "url": " https://github.com" } } ]

pail23 commented 5 months ago

Fixed in 2024.1.1

werpu commented 5 months ago

Thanks yes I was indeed on an. old version user error, did not know that the HACS integrations need to be updated from the HACS tab! However with the 20234.1.1 version i still cannot see the sensors, they are now simply deactivated by the plugin despite being there and showing values in ISG itself:

image image

(Isttemperatur is the sensor value being deactivated, because it is a readonly value, as it seems)

So I would not really consider this issue to be fixed!