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

Multiple Heating Devices via Home Assistant integration #323

Open
CFenner opened this issue Jul 4, 2023 · 8 comments
Open

Multiple Heating Devices via Home Assistant integration #323

CFenner opened this issue Jul 4, 2023 · 8 comments

Comments

@CFenner
Copy link
Member

CFenner commented Jul 4, 2023

I use the ViCare integration to Home Assistant and I think underneath it's using this library. Now I added a second heating device to my account and it's not showing up. Is this use-case in planning or is there anything I can do to help to support this?

@CFenner
Copy link
Member Author

CFenner commented Oct 15, 2023

Meanwhile I found the place in the integration where the devices are loaded and prepared a PR home-assistant/core#96044 to implement this.

@tuxiano
Copy link

tuxiano commented Jan 3, 2024

Hi, will your PR also help to show radiators and room sensors in HA?

@CFenner
Copy link
Member Author

CFenner commented Jan 3, 2024

With the code from the PR, all devices will be iterated. If any sensor is created depends of the sensor definitions that are already maintained in the integration. I don't have such devices, you're welcome to test it.

@Bascht74
Copy link

Bascht74 commented Jan 7, 2024

@CFenner Hi Christopher, I was looking forward to that. But the Radiators a not populated with the change.

The feature was added in the HACS integration:
oischinger/ha_vicare@65fd645

There is a smal hint that might be relevant:

RadiatorActuator have no circuits but also create a climate entity.

Could that be the cause why the radiator devices won't be ppulated with the offifial HA Integration?
I tried it with the custom one and they are there...

Thx for your work anyway!

@CFenner
Copy link
Member Author

CFenner commented Jan 7, 2024

Yes, currently climate entities are only created for circuits.

@Bascht74
Copy link

Bascht74 commented Jan 7, 2024

Good to know!

I hope that somebody can add that definition to the official repro as well…
I don’t know if this might be a rather big change or small addition.

In my second life I will become a python programmer so that I could add stuff myself…

@danielhad
Copy link

If this CR would be productiv, that would be great. Have a heatpump and an oilburner. Now i buyed a second vitoconnect (for the oilburner), but cant add this values to home assistant :-(

@tuxiano
Copy link

tuxiano commented Jan 24, 2024

@CFenner Hi Christopher, would you mind to answer me some question related to the PyVicare development e.g. on discord?

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

4 participants