Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Datapoint not included in HA #402

Closed
tpulatha opened this issue Sep 23, 2024 · 7 comments
Closed

Datapoint not included in HA #402

tpulatha opened this issue Sep 23, 2024 · 7 comments

Comments

@tpulatha
Copy link

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
Copy link
Member

CFenner commented Sep 23, 2024

Hi @tpulatha

Just checked for your data point here in the library, it was removed recently (see #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
Copy link
Author

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
Copy link
Member

CFenner commented Sep 23, 2024

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

@CFenner
Copy link
Member

CFenner commented Sep 25, 2024

@tpulatha can you add a diagnostics file, please?

@tpulatha
Copy link
Author

Added below. Sorry for the delay.

config_entry-vicare-01J7Y6PA4RYJY92JQDB3EER28G.json

@CFenner
Copy link
Member

CFenner commented Jan 23, 2025

@tpulatha this should be in production by now.. can you verify and close the issue?

@tpulatha
Copy link
Author

Confirmed works. I'll close it! Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants