Skip to content
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 fix for Air Purifier Pro #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

A320Peter
Copy link

@A320Peter A320Peter commented Jan 14, 2019

This is a bug fix for Air Purifier Pro

Adjusting the FAN SPEED in MANUAL mode in the Home App, the FAN SPEED slider has values from 0-100. This plugin translates the slider setting to a number between 0 and 10, so sending out this parsed value to the air purifier. I assume the reason is that other Xiaomi Air Purifiers using fan speed settings from 0 - 10. However, Air Purifier Pro supports 17 different values for the FAN SPEED (in Mi App the minimum setting is 7-12m2 and the maximum is 35-60m2). All together there are 17 different values available. Because the plugin limit is 10, so the maximum FAN SPEED setting stops around ~60%. When the plugin checks for the actual fan speed to adjust the slider position, the same incorrect calculation results in setting the slider to the lower range instead of the actual speed.

I used a different formula to translate the slider setting to a value between 1 and 17.

This is a bug fix for Air Purifier Pro

Adjusting the FAN SPEED in MANUAL mode in the Home App, the FAN SPEED slider has values from 0-100. This plugin translates the slider setting to a number between 0 and 10, so sending out this parsed value to the air purifier. I assume the reason is that other Xiaomi Air Purifiers using fan speed settings from 0 - 10. However, Air Purifier Pro supports 17 different values for the FAN SPEED (in Mi App the minimum setting is 7~12m2 and the maximum is 35~60m2). All together there are 17 different values available. Because the plugin limit is 10, so the maximum FAN SPEED setting stops around ~60%. When the plugin checks for the actual fan speed to adjust the slider position, the same incorrect calculation results in setting the slider to the lower range instead of the actual speed.

I used a different formula to translate the slider setting to a value between 1 and 17.
@MickL
Copy link

MickL commented Jun 13, 2019

Hello @YinHangCode could you please review and merge this PR?
If you dont have time please give the rights to someone else. I might fix some issues or improve features but i need the rights to do so or at least you have to merge pull requests. If you dont i can not put time into this or we have to fork maybe but i would like to keep this project alive instead!

@dgproductions88
Copy link

Completely agree with @MickL, pls, @MickL why not you create new plugin to have this "awesome features" ? ... seems author will not work anymore in this plugin....
Really appreciate it!

Cheers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants