-
-
Notifications
You must be signed in to change notification settings - Fork 30.3k
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
HA 0.62 Docker edition. Problem with philips xiaomi light. #11948
Comments
The component light.xiaomi_miio requires python-miio 0.3.4 and construct 2.8.22 which is part of HA 0.62. This line
was fixed (parameters swapped) in python-miio 0.3.4. It looks like you aren't using a recent and required python-miio version. |
I have the same issue with 0.62 from here The recent pyhton-miio should be installed within the image right? Unfortunately, I'am not able to find any Dockerfile for this official image. |
Right. This is the code of the new python-miiio (0.3.4) version: https://github.com/rytilahti/python-miio/blob/master/miio/protocol.py#L179 The mentioned one
is the previous order (python-miio <= 0.3.3). |
But it seems that the docker image does not use the latest version of pythin-miio, since I still have this error in the log and the vacuum cleaner does not show up in Home Assistant anymore. |
My Xiaomi vacuum is missing too after the upgrade to 0.62. |
Could you provide a list of all components you are using actively / mention in your configurations.yaml? I assume a conflict between the component dependencies. |
Hassbian with Xiaomi Wifi Socket. homeassistant (0.62.0) + python-miio (0.3.4) log,
|
@tanus10 Please report the version of the "construct" package also. This is the wanted combination: HA 0.62 + python-miio 0.3.4 + construct 2.8.22. |
Okay. I was able to reproduce the issue. You are using construct 2.9.23 released two hours ago. :-( Sad world. |
I've fixed the issue at the current master of python-miio: https://github.com/rytilahti/python-miio |
HA 0.62 + python-miio 0.3.4 + construct 2.9.23 + patched device.py are working nicely. My wifi socket always have been producing time-out error at first attempt when HA starts but it get it right after. I confirm the current master miio version works as the previous working version. Now it's unbroken. Thank you syssi. |
Is it easy to fix on my hassio or do we have to wait for a new release? |
Only 4 lines of code in device.py. You have to look for the change, since the the line numbers are not in sync. |
hassio is a docker container and changes at a docker container are lost on every reboot. I don't know a easy and persistent way to hotfix this issue at hassio. Any hassio/docker gurus around? |
I have the same problem with my Xiaomi vacuum cleaner. |
can confirm that applying the patch manually from rytilahti/python-miio@17c7a26 to /srv/homeassistant/lib/python3.5/site-packages/miio/device.py fixes the issue |
Update to HAS 62.1 from https://hub.docker.com/r/homeassistant/home-assistant/ python-miio (0.3.4) + construct (2.9.24) - doesn`t work python-miio (0.3.4) + construct (2.8.22) - doesn`t work |
With the combination python-miio 0.3.4 + construct 2.9.27, I got following error:
|
The patch was integrated in hass 61.2, still doesn't work (as zamzas pointed out). |
Following this too, now that I finally manage to get my vacuum token... |
I use HA as an official docker container on x64 architecture. |
HA 62.1 - philips xiaomi light does not work. |
The fix will be part of HA 63.0! Please check the release notes carefully. |
Individual components update must coincide with HA major/minor update, even if individual components have already updated for sometime. Which may take weeks. Only workaround is manual modification of the python files, but with hassio/docker it seems rather unconvenient or near-impossible. With Hassbian and other distributions it’s not as tiresome. |
I'm not sure this will fix the issue, I applied the change from 137933a and made sure I had the latest construct. By start I still got an error for the vacuum component (I only have a vacuum from xiaomi)
|
@RiRomain This line isn't part of python-miio 0.3.5 anymore: It's obviously you aren't using python-miio 0.3.5. Do you know the command line parameter "--skip-pip"? If you start homeassistant (bin/hass) it will retrieve all requirements. The requirements are pinned to a specific version number (cp. https://github.com/home-assistant/home-assistant/blob/dev/requirements_all.txt). If you update python-miio manually a startup of homeassistant (<=0.62.1) will downgrade the requirement again! Please update homeassistant to the "dev" version. It will fix the issue properly. |
Have a philips xiaomi light.
I upgrade homeassistant to 0.62 version.
My log:
This device properly work on HA 0.59
The text was updated successfully, but these errors were encountered: