jghaanstra / com.xiaomi-miio

Homey app to control Mi Home devices that implement the miIO protocol and Xiaomi gateway child devices
GNU General Public License v3.0
456 stars 85 forks source link

Xiaomi smart purifier elite doesnt work #234

Closed thomaswitt closed 1 year ago

thomaswitt commented 1 year ago

I am looking for support for the https://www.mi.com/global/product/xiaomi-smart-air-purifier-elite/

jghaanstra commented 1 year ago

Read this and supply the required information or a device to test with.

https://github.com/jghaanstra/com.xiaomi-miio/wiki/Adding-new-devices

jghaanstra commented 1 year ago

Please test release 3.2.15 which is pending approval by Athom in the app store.

https://homey.app/a/com.xiaomi-miio/test/

https://github.com/jghaanstra/com.xiaomi-miio/wiki/Supported-Devices

jghaanstra commented 1 year ago

Closing this due to lack of response.

thomaswitt commented 1 year ago

@jghaanstra – Sorry for the delay and thank you for your effort! I set it up with 3.2.16, the setup process actually worked - but device modes (on/off/auto) are not recognized and I don't get any readings like PM2.5. Only reading which works is "Air Cleared" (m3). Temperature shows 1° and Humidity a wrong reading (25% where as it is 48%). Setting night and favorite mode works as well, auto does not.

jghaanstra commented 1 year ago

Please send a diagnostic report from the app settings.

thomaswitt commented 1 year ago

@jghaanstra Done – 3f34e560-34aa-4e5a-b84d-d470dcad796d

jghaanstra commented 1 year ago

The next release will contain a possible fix.

jghaanstra commented 1 year ago

Please test release 3.2.17 which might contain a fix. The release is pending approval by Athom in the app store but you can test it through the test channel

https://homey.app/a/com.xiaomi-miio/test/

thomaswitt commented 1 year ago

Just tested it. Better, but not perfect yet.

What's broken:

What's missing:

I attached some screenshots of the app to demonstrate which settings I mean … I hope that helps…

IMG_4634 IMG_4635 IMG_4636

jghaanstra commented 1 year ago

About the mode. There could be differences in what the title says and what mode is actually switched. This driver supports a lot of airpurifier and Xiaomi doesnt really follow a standard. It's impossible to implement every variation. That said, the documentation here implies there are still 4 modes ranging from 0 to 3. That does not explain why some of the modes do nothing so that documentation may be incorrect. Not much I can do about. If you want this fixed you will need to supply the correct miot details of this device (or donate the actual device) yourself.

The screen brightness is available as a on-off setting under the device settings.

If you can confirm that manual fan speed corresponds to value siid 9-1 (called moto-speed-rpm) here I might consider implementing it.

thomaswitt commented 1 year ago

Link is broken - would you be so kind to correct it?

jghaanstra commented 1 year ago

The link is the same is the first one, just look at parameter siid 9 piid 1.

thomaswitt commented 1 year ago

Hi,

thnks for sharking the spec. I think I see the problem.

In the spec, the Mode is PIID 4 (0 - Auto // 1 - Sleep // 2 - Favorite // 3 - Manual), but in the code it is mapped to PIID 5 (Line 243. You just forgot to change the set_properties accordingly.

I made a pull request to correct this.

jghaanstra commented 1 year ago

Thanx, I'll change that locally instead of merging the PR. It will be included in the next release. If that will fix the mode issue remains to be seen then.

jghaanstra commented 1 year ago

This fix is included in release 3.2.18 which is currently pending Athom's approval in the app store. You can test it by installing the test version.

https://homey.app/a/com.xiaomi-miio/test/

thomaswitt commented 1 year ago

Tested and it works. 🚀Thanks a lot for your time and your work. 🙏🏻

Regarding my other question, I'd like to change the display brightness via a "Then..." card (currently only possible via preferences) - should I open another issue for that?

jghaanstra commented 1 year ago

Regarding my other question, I'd like to change the display brightness via a "Then..." card (currently only possible via preferences) - should I open another issue for that?

Please do.