michaelarnauts / comfoconnect

Python Zehnder ComfoConnect LAN C library to interface with ComfoAir Q350/450/600 units.
Other
93 stars 34 forks source link

BUG ?: Exhaust Supply switched #34

Open TimoFriedri opened 3 years ago

TimoFriedri commented 3 years ago

I think there might be some mixup with the exhaust and supply values.

  1. I have checked with the official Zehnder app, used its Unbalance setting and an anemometer that the values in the Zehnder app are actually the correct ones.

  2. I use pycomfoconnect indirectly in Home Assistant, but I think, the mixup happens in the pycomfoconnect library, since HA imports all the static parameters.

  3. exhaust_airflow and supply_airflow values are switched in HA compared to the values in the Zehnder app!

Can anyone confirm this?

hme0354 commented 3 years ago

i think so too

TimoFriedri commented 3 years ago

I think that

airflow fan duty fan speed

are mixed up. Humidity and Temperature seem fine for exhaust and supply.

klaudiusz223 commented 3 years ago

Hi, I use my own program ComfoAirQ-Homie with OH and everything looks correct.

But i think that some const definitions could by improved. Maybe someone uses them opposite way For example these https://github.com/michaelarnauts/comfoconnect/blob/433bd3ef309c6c4f70e170eda8b3fedbae851d50/pycomfoconnect/const.py#L8-L9

could be defined similar to my definition

CMD_TEMPORARY_STOP_SUPPLY_FAN       = b'\x84\x15\x07\x01\x00\x00\x00\x00\x10\x0e\x00\x00\x01'
CMD_START_SUPPLY_FAN                = b'\x85\x15\x07\x01'

CMD_TEMPORARY_STOP_EXHAUST_FAN      = b'\x84\x15\x06\x01\x00\x00\x00\x00\x10\x0e\x00\x00\x01'
CMD_START_EXHAUST_FAN               = b'\x85\x15\x06\x01'

https://github.com/klaudiusz223/ComfoAirQ-Homie/blob/09ec1e01059361797c5f9a3d88889137d0b817a3/comfoairq_homie/comfoairq/comfoairq_const.py#L1-L5

klaudiusz223 commented 3 years ago

One thing to check. What is the orientation of Your units? You Can check it in Zehnder app. My unit has "right" orientation. Maybe that's the problem? Maybe constants should be defined as right and left fan and depends on the orientation, fan should be assign to supply or exhaust?

TimoFriedri commented 3 years ago

My unit is right as shown in the Android app. And massflow/fan values are mixed up in home assistant.

Firmware is 1.7.0

On Mon, 11 Jan 2021, 13:47 klaudiusz223, notifications@github.com wrote:

One thing to check. What is the orientation of Your units? You Can check it in Zehnder app. My unit has "right" orientation. Maybe that's the problem? Maybe constants should be defined as right and left fan and depends on the orientation, fan should be assign to supply or exhaust?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/michaelarnauts/comfoconnect/issues/34#issuecomment-757929980, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEUCOZLAZEPCYQDJK7A2HJTSZLXNTANCNFSM4UDOPBRA .

michaelarnauts commented 3 years ago

I have a left orientation, my values are correct. So some logic should be written to read the orientation and return other values based on this.

zathras777 commented 3 years ago

Assuming that the app somehow knows the orientation and adjust it's output, guess it's there somewhere? My unit is RH and shows the same issues in HA (just raised an issue there) but looking at register 33 I get a value of 0 whereas you show 1? Can anyone else with a RH machine verify what they get?

BrechtMo commented 3 years ago

I noticed this yesterday as well (it's very obvious when Frost Protection is active). Today I updated my HA docker container to Home Assistant 2021.2.1 and it seems to be fixed.