michaelarnauts / home-assistant-comfoconnect

Home Assistant integration for Zehnder ComfoAir trough a Zehnder ComfoConnect LAN C.
Other
26 stars 13 forks source link

FR: Support ComfoCool Q600 #15

Open middelink opened 1 year ago

middelink commented 1 year ago

I own a ComfoCool which I to control or at least make its state visible in Home Assistant.

So far I have found out sensor_802 is the condensor temperature and sensor_85 seems to be the ComfoCool Next Change state (it counts down when I turn the comfocool off for say 1 hour).

sensors_53 and sensor_341 seem related, at least the app changes both it seems.

sensor_53: auto -> -1 off for xx -> 1

sensor _341: auto -> 0 off for xx -> 2

Attached are a decoded pcap and the pcap file for good measure where I play with first turning the compfocool off for 1 hour, the continiously off then back to auto and I change the temperature profile to "cool", which first turns bypass to 100% as it is chilly right now, but about 1 minute later the comfocool kicks in.

Le tme know if you need any other information to integrate the comfocool!

comfoconnect.txt comfoconnect.pcap.zip

gertst commented 1 year ago

This would be a very nice addition, indeed! If you need testers, please let me know! Very pleased with the ComfoCool module, btw. If I could automate it more using HA, that would result in an awesome climate control!

michaelarnauts commented 4 months ago

More information here: https://github.com/michaelarnauts/comfoconnect/issues/27

szibis commented 3 months ago

This may help here https://github.com/michaelarnauts/home-assistant-comfoconnect/pull/51 and https://github.com/michaelarnauts/home-assistant-comfoconnect/pull/52

szibis commented 3 months ago

Still I think thing that I am not fully sure yet how to register comfocool as seperate device but will dig into this somehow.

gertst commented 3 months ago

I don't think it should be a separate device. In fact, the Zehnder ComfoAir app just shows it as a property: In the app, "ComfoCool" is shown under the "Tasks" tab, more or less like the "Boost" task. So there is also a duration feature, but that's not required in HA: A simple switch entity is all we need: more fancy stuff can then be done using HA automations. At least this is my take on it. If you need a tester, I'm happy to help.

szibis commented 3 months ago

In Unit status shows as a part of whole as it is reporting the temp and humidity going through comfocool when working. Still also has separate serial number and own firmware which indicates as separate device from perspective of how comfoair and comfolan c is reported image

szibis commented 3 months ago

I will start full tests once we will have new release of aiocomfoconnect, but all help is appreciated.

michaelarnauts commented 3 months ago

I have merged all your commits and released a new aiocomfoconnect. You can try installing the master branch of this integration in HACS to have these changes in HACS.

@szibis I want to thank you for your invested time in this!

michaelarnauts commented 3 months ago

In Unit status shows as a part of whole as it is reporting the temp and humidity going through comfocool when working. Still also has separate serial number and own firmware which indicates as separate device from perspective of how comfoair and comfolan c is reported image

This is probably another Node. See https://github.com/michaelarnauts/aiocomfoconnect/blob/master/docs/PROTOCOL-RMI.md#nodes.

During connection, the bridge sends a CnNodeNotificationType to indicate what nodes are available.

I assume you can get the version numbers from that Node, It's probably on SubUnti 0x01, property 0x06. This is the firmware version for Node 0x01 (The Unit itself) and 0x30 (The Comfoconnect Bridge).

szibis commented 3 months ago

Yes thanks @michaelarnauts will try to investigate this to register this properly as it is in Bridge. For now adding some improvements to have that sensors/mode in ComfoAir working properly https://github.com/michaelarnauts/home-assistant-comfoconnect/pull/57, but once I add new device will move the sensors to this new device and also mode selector.

szibis commented 3 months ago

Just for now current master, before above PR for small changes reports properly state and condensor temperature

image
szibis commented 3 months ago

Ok and state now reported properly

image