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

Has anyone tested it with Air Purifier 2s EU version? #258

Closed
dottoreFell opened this issue Oct 16, 2019 · 10 comments
Closed

Has anyone tested it with Air Purifier 2s EU version? #258

dottoreFell opened this issue Oct 16, 2019 · 10 comments

Comments

@dottoreFell
Copy link

dottoreFell commented Oct 16, 2019

Air Purifier 2s isn't on the supported devices list, but it works fine for me. I have CN version, but I want to buy next two units, this time EU version. Will they work with miio so well as CN version?

@kwesolowski
Copy link

kwesolowski commented Oct 22, 2019

Works for me:

Device ID: XXXXX
Model info: zhimi.airpurifier.mc1
Address: 192.168.0.6
Support: At least generic

@dottoreFell
Copy link
Author

dottoreFell commented Oct 27, 2019

I have bought this version:

Device ID: 132218784
Model info: zhimi.airpurifier.mc1
Address: 192.168.1.148
Token: ???
Support: At least generic

I am connecting, but Purifier won't react to commands, capability list is empty:

Connected to MiioDevice {
model=zhimi.airpurifier.mc1,
types=miio,
capabilities=
}

miio.device({ address: '192.168.1.148', token: tokens }) .then(device => { device.temperature() .then(temp => console.log(temp)) }) .catch(err => console.log(err));

TypeError: device.temperature is not a function
at miio.device.then.device (/home/michal/source/home_assistant/purifier2_IO.js:88:12)

Any ideas?

@kwesolowski
Copy link

Looks like missing token (unless you actually replaced real token with ???)

@dottoreFell
Copy link
Author

dottoreFell commented Oct 27, 2019

tokens is variable that contains device token. If token was missing then the part of code within .then block (device.temperature call) wouldn't be executed at all.

In this case I am able to connect but capability list is empty and therefore the rest of commands doesn't work.

EDIT: Working chinese version has firmware 1.4.3_13108, my second eu purifier has 1.4.3_21104

@kwesolowski
Copy link

kwesolowski commented Oct 27, 2019

When you test this do you have merged #220?

@dottoreFell
Copy link
Author

dottoreFell commented Oct 27, 2019

When you test this do you have merged #220?

@kwesolowski
I manually edited models.js and added missing line. It is working fine. I am new to git and programming, so tommorow I will take a closer look at this and try to do merge. Also I will investigate why I haven't found Your solution before and probably learn something new about github.
Thanks a lot!

@kwesolowski
Copy link

@aholstenson Second person happy with mc1. Can you pick few people who were active here to merge at lest such simple PRs? I can help a bit as well... Its too great project to keep private fork :(

@dottoreFell
Copy link
Author

What could we do? Isn't the project maintainer who can merge PRs?

@kwesolowski
Copy link

kwesolowski commented Oct 28, 2019 via email

@kwesolowski
Copy link

kwesolowski commented Oct 28, 2019 via email

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

No branches or pull requests

2 participants