-
-
Notifications
You must be signed in to change notification settings - Fork 33.2k
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
Comfoconnect integration is using a deprecated library to talk to the Zehnder #141684
Comments
Hey there @michaelarnauts, mind taking a look at this issue as it has been labeled with an integration ( Code owner commandsCode owners of
(message by CodeOwnersMention) comfoconnect documentation |
Ah, @michaelarnauts, I didn't notice that you owned the Home Assistant integration. I feel certain you're quite familiar with the situation with the two libraries considering you wrote both of them 🙂 |
Indeed. There is also a HACS integration I wrote to replace the one in home assistant, but I don't have the energy and time to handle all the steps that are necessary to replace the integration in core. If someone is interested in helping with this, please do so :) |
Ah, I see: https://github.com/michaelarnauts/home-assistant-comfoconnect. You would just want to replace what's in the repo with that (as-is, to the extent that's possible)? I've never worked with Home Assistant before, but that's doesn't sound terribly difficult. The config migration seems like it's got the most challenging bits. What other issues do you foresee? |
I'm not sure what's best to do here. I think it's best to just replace it, but that requires tests, and I don't have those. It will also be a breaking change, and I don't think the entities are migrated correctly, so you'll loose history of your sensors. |
The problem
pycomfoconnect has been deprecated since 2022. The replacement is aiocomfoconnect, from the same author.
I wasn't able to find a bug tracking the upgrade -- is this change of interest to the integration maintainers?
The reason I noticed this I wanted to control non-fan aspects of the Zehnder from automations. This is technically possible via pycomfoconnect today, but is far nicer with the updated library. For instance, I wanted to control the bypass state. I found someone else who wanted to do the same thing (michaelarnauts/comfoconnect#30), and the solution is much cleaner in aiocomfoconnect -- you just call
set_bypass
.What version of Home Assistant Core has the issue?
core-2025.3.4
What was the last working version of Home Assistant Core?
No response
What type of installation are you running?
Home Assistant OS
Integration causing the issue
comfoconnect
Link to integration documentation on our website
https://www.home-assistant.io/integrations/comfoconnect/
Diagnostics information
No response
Example YAML snippet
Anything in the logs that might be useful for us?
Additional information
No response
The text was updated successfully, but these errors were encountered: