home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.89k stars 30.95k forks source link

Can't use Xiaomi Miio integration #76255

Closed patricksmms closed 2 years ago

patricksmms commented 2 years ago

The problem

When I try to configure the integration, to control my Xiaomi Smart Standing Fan 2, I get an error 'code': -9999.

Repro steps:

Result:

Expected:

Note: I can control it with the phone without issues, and I can ping it device running home assistant.

What version of Home Assistant Core has the issue?

2022.7.7

What was the last working version of Home Assistant Core?

2022.7.7

What type of installation are you running?

Home Assistant Container

Integration causing the issue

xiaomi_miio

Link to integration documentation on our website

https://www.home-assistant.io/integrations/xiaomi_miio#xiaomi-air-purifier-air-humidifier-and-standing-fan

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

2022-08-04 21:38:41 WARNING (MainThread) [homeassistant.config_entries] Config entry 'Mi Smart Standing Fan 2' for xiaomi_miio integration not ready yet: {'code': -9999, 'message': 'user ack timeout'}; Retrying in background

Additional information

I can see the device on the cli, running miiicli cloud list: == Mi Smart Standing Fan 2 (Device online ) == Model: dmaker.fan.p18 Token: 419514e2f3733e6a844dbeae2958f176 IP: 192.168.11.17 (mac: 54:48:E6:49:88:5E) DID: 250819038 Locale: de

I tried to configure manually, using this data, resulted on the same error.

probot-home-assistant[bot] commented 2 years ago

Hey there @rytilahti, @syssi, @starkillerog, @bieniu, mind taking a look at this issue as it has been labeled with an integration (xiaomi_miio) you are listed as a code owner for? Thanks! (message by CodeOwnersMention)


xiaomi_miio documentation xiaomi_miio source (message by IssueLinks)

patricksmms commented 2 years ago

OK, I just understood that the problem is that this integration doesn't support yet the model I have 'dmaker.fan.p18'.

Would be nice to have it added, instead of having to install https://github.com/syssi/xiaomi_fan

rytilahti commented 2 years ago

Yeah, the current implementation requires adding all supported devices also inside homeassistant. This upstream issue tries to solve that by providing a nicer way to initialize the instances for all known to be supported devices: https://github.com/rytilahti/python-miio/pull/1328

patricksmms commented 2 years ago

@rytilahti why did you close the issue, if it is still a work in progress?

rytilahti commented 2 years ago

@patricksmms ah, sorry about that. So, I closed this because it's a wishlist item which are generally not accepted in the issue tracker. But considering I didn't close the another similar one (#73496), you can follow that to keep up with the status. I modified its title to add mention about p18, so we can track both of them in a single issue.

patricksmms commented 2 years ago

Ok. Thank you!