-
Notifications
You must be signed in to change notification settings - Fork 22
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
Fan control interface does not display options correctly #226
Comments
HVAC operations are sort of split up between climate modes, default climate presets, and custom climate presets. We map them to one of the supported Home Assistant climate modes here and create presets on a thermostat-by-thermostat basis: zha/zha/application/platforms/climate/const.py Lines 150 to 169 in 4a4abd2
I think for this attribute to work, the various modes from the ZCL would need to be turned into custom presets. Does your device actually support all of the ZCL modes? If not, which ones does it support? |
the only thing im unsure about is if the eco mode corresponds to precooling, but i guess it supports all modes of the zigbee standard mentioned in table 6-16 |
so my understanding is that precooling basically saves energy by pre-cooling air in the morning so the building can be cooled more efficiently during hotter parts of the day. in contrast, eco mode just runs the compressor with less power. so technically they are not the same, although both are aimed at reducing energy cost. however, since the zigbee standard does not support an eco mode, i would use the precooling mode as eco mode for my use case. i'm not sure whats the best solution here, but i would at least support emergency heat, fan only, dry and sleep for climate devices |
We can display the modes (the draft PR I'm working on above does this) but it may have to be gated behind a quirk, depending on device support. It's a little frustrating that the spec makes some of these modes all-or-nothing, without allowing any sort of support bitfield. I'll have to look at how other thermostats report their capabilities. |
its funny how the zigbee standard states that not all SystemMode / ControlSequenceOfOperation values have to be supported, yet doesn't offer a way to report which are |
would it be possible to just add the missing modes to the SEQ_OF_OPERATION array for now? |
I'm currently developing a stick that can control midea AC units, and i stumbled upon a weird behavior when exposing a fan control cluster:
6.4.2.2.2 of the zigbee cluster specification states that the fan mode sequence attribute should be set to 0x02 if the fan supports low/med/high/auto modes.
This does seem to be picked up correctly by home assistant / ZHA:
However, in the interface, the only options are 'on' and 'auto', which corresponds to 0x04:
The text was updated successfully, but these errors were encountered: