-
Notifications
You must be signed in to change notification settings - Fork 36
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 Speed issue with zhimi.fan.za5 #61
Comments
I just upgraded my fan to the za5, and I'm seeing the same issue. The issue originates from In the entity attributes, Level 4 is also missing from the speed list
Unfortunately it seems the issue doesn't lie with our code but rather upstream. We'll have to raise an MR with them. |
We can quickly allow iteration through 3 speeds instead of 4. Or option "use_standard_speeds: true" can be used, then it will iterate through low/medium/high options. |
Oh wow you're fast. @insajd thanks for your quick work as always. Let's release a beta, while I'll look into how I can solve that with a PR upstream. To confirm it's a toggle-able option right? |
First option can be used already, for second option we can release beta, which will influence only za5 fan. I'll go with it |
Roger that. Let's roll with that fix for za5 temporarily until support is added. I just poked upstream's upstream. Looks like the issue has been open for a while. ZA5 is using the new miotspec which is a whole new bag of worms and needs significant work. Likely we won't expect the fix to be here within the next month or so. |
@Spegeli Just so you know, there are some ongoing issues with upstream:
|
@insajd It's working as expected, just tested with the ZA5. I'm able to cycle through all 3 speeds. There are some weirdness that I observed however, due to the fan being not fully supported by
For example, when speed is at
It appears that the Nonetheless, I don't think we can or should implement any further hacks for ZA5. Thanks again! |
@Spegeli Do update to |
Works thx ;-) Now its time to wait until the Miio Fan Adapter Dev adds the rest of the fan futures ^^ |
@ikaruswill you can try getting this version, which ought to add support for the fan. rytilahti/python-miio#1087 and get ready the support of the fan card. |
Nice spot. Will check that out. |
@ikaruswill @insajd |
Closing as za5 is now supported with native integration on 2.0.0 |
I own the Pedelstan Fan 3 and got a issue with the Fan Speed:
model: zhimi.fan.za5
integration: xiaomi_miio_fan
The Error:
"Fehler beim Aufrufen des Diensts fan/set_speed. The speed Level 4 is not a valid speed."
So Speed 3 is the Limit and because of this Error i can not switch back to Speed 1.
Thats the Attributes:
speed_list:
preset_modes:
oscillating: false
speed: Level 3
percentage: 100
percentage_step: 33.333333333333336
preset_mode: Level 3
model: zhimi.fan.za5
The text was updated successfully, but these errors were encountered: