openviess / PyViCare

Python Library to access Viessmann ViCare API
Apache License 2.0
131 stars 87 forks source link

Datapoint not included in HA #402

Open tpulatha opened 1 day ago

tpulatha commented 1 day ago

Hi,

I have a Vitovalor PT2 Fuelcell/boiler/hot water cylinder combination which I have connected to Home Assistant since quite a while. Unfortunately, It seems to be missing quite some datapoints. For example heating.dhw.sensors.temperature.hotWaterStorage.bottom

When checking the debuglogs, I do see the information

            "deviceId": "0",
            "feature": "heating.dhw.sensors.temperature.hotWaterStorage.bottom",
            "gatewayId": "################",
            "isEnabled": true,
            "isReady": true,
            "properties": {
              "status": {
                "type": "string",
                "value": "connected"
              },
              "value": {
                "type": "number",
                "unit": "celsius",
                "value": 50.6
              }
            },

I already deleted and readded the device in the integration. Any other idea? Of course latest HA version 2024.9.2. Thanks!

PS: I also posted this in your fork, sorry, didn't know which one should be used.

CFenner commented 20 hours ago

Hi @tpulatha

Just checked for your data point here in the library, it was removed recently (see https://github.com/openviess/PyViCare/pull/381, you should also see a hint for that in your diagnostic information) but it's replacement heating.dhw.sensors.temperature.dhwCylinder.bottom is exposed via getHotWaterStorageTemperatureBottom. So no issue from the library pov.

But there is no sensor in the HA integration that uses this value. Therefore I ask you to open an issue in https://github.com/home-assistant/core

PS: Currently the library for is used in the HA integration, but we plan to switch it back soon.

tpulatha commented 20 hours ago

Thanks, yeah sorry I realized that it was deprecated after I posted this. Ok, I'll open an issue. I try to add all missing sensors.

CFenner commented 19 hours ago

We have dhwCylinder, dhwCylinder.topand dhwCylinder.bottom exposed here. So at least dhwCylinder.middle and dhwCylinder.midBottom are missing.