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

fix construct==2.8.17 Cause the plug-in failure #11089

Closed
wants to merge 11 commits into from
Closed

fix construct==2.8.17 Cause the plug-in failure #11089

wants to merge 11 commits into from

Conversation

roiff
Copy link
Contributor

@roiff roiff commented Dec 11, 2017

fix construct==2.8.17 Cause the plug-in failure

Description:

Related issue : fixes #11048 (comment)

At the same time update in python-miio
rytilahti/python-miio#140

See which side to submit first
See which side first submitted to solve the problem

fix construct==2.8.17 Cause the plug-in failure
@@ -31,7 +31,7 @@
vol.Optional(CONF_NAME, default=DEFAULT_NAME): cv.string,
})

REQUIREMENTS = ['python-miio==0.3.2']
REQUIREMENTS = ['python-miio==0.3.2','construct==2.8.16']

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing whitespace after ','

@rytilahti
Copy link
Member

This is not going to be fixed by pinning the version on upstream; if pinning for the specific version in homeassistant is fine, then feel free to do that. I would prefer to have a clean solution (either by fixing python-miio to work with the new construct version, or by fixing construct if the problem lies there).

@rytilahti
Copy link
Member

I just tested the newer version out, and made mirobo to print out the version number. Seems to work fine here.

$ mirobo
construct version: 2.8.17
State: Charging
Battery: 100 %
Fanspeed: 87 %

@roiff
Copy link
Contributor Author

roiff commented Dec 12, 2017

It doesn't matter, I just provide a way to the fastest possible repair, because time is limited, can't further wrong, just as a reminder, after all, a lot of people already can't use, thank you for your effort

@roiff roiff closed this Dec 12, 2017
@home-assistant home-assistant locked and limited conversation to collaborators Mar 30, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Xiaomi Vacuum Robot Unavailable
4 participants