-
Notifications
You must be signed in to change notification settings - Fork 15
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
Add support for Xiaomi air purifier Pro H #3
Comments
Ok, so lets try the following:
Now, the device should work (If its really like the v7). Please report. |
Hi, In iobroker objects I got some datapoint below the device but without any data. And not the structure as the 3H have. Greetings, Jürgen |
|
You wrote in your first post: I would bet it is more or less the same as zhimi.airpurifier.v7 which is the air purifier Pro (without H)
|
Sorry for the wrong pointing to the airpurifier pro. That was before I tested it with FHEM and found it working as air purifier 3H. But it's cosmetic, the script is just working perfect. Many thanks 👍 👍 👍 Greetings, Jürgen |
Just saw, thar I get some error messages every now and the, so I#ll reopen the issue: `
Greetings, Jürgen |
I thought i've captured this, its quasi normal, sometimes node-mihome or the cloud sends a empty datapacket, this causes the error. I'll fix it until tomorrow i think. Look for a new Versionnumber. |
Thanks. I'll then close this issue here as it seems it has nothing to do with the errors as they come from node-mihome. Greetings, Jürgen |
Hi, I updated to 0.2.4 as you requested yesterday. All datapoint (including filter-used-time, filter-remaining is completely gone) now seem to have valid data except alarm/alarm at the new Pro H. The 3H show 'false', the Pro H just an empty field. `
` Greetings, Jürgen |
I've answered you at the forum 2hour ago. Please enable logging in the skript, and send me a complete startlog inkl. first refresh after 10seks. |
I stopped the script, enabled logging and then started. This is the log: Anything suspicious? For me it seems to be a problem with the Pro H, as ist allways occurs after this line: script.js.common.Xiaomi_Devices: Reaching RefreshDps at Mi Air Purifier ProH Another question. I did a "upgrade self" in iobroker as I thought, there was a new version of JS-Adaper (it wasn't as I am on stable and new version is only in latest, I read wrong in forum), then restarted hole iobroker and after that, I had to copy a new version of file 'zhimi.airpurifier.vb2' into '/opt/iobroker/node_modules/iobroker.javascript/node_modules/node-mihome/lib/devices/' as ist was gone. Is it normal, that the file gets deleted when restarting hole iobroker or doing a "upgrade self"? Greetings, Jürgen |
"upgrade self" upgrades also the node-mihome to the actual version, and there the new file is missing. Iobroker restrart is no problem.
|
Here is the complete log for those two timesteps: And this the same for a 3H: |
Ok, thats the proof, there is no alarm property at the pro (dont ask why, i dont know). So, thats the reason for the error. I have removed this property from the skript, pls download again (no new version). |
Hi, thanks. New version seems to work better. One funny thing: With the newest version I get no errors, but I've played around a bit in Xiaomi Home App. If I check or uncheck "Benachrichtigungston" for the two 3H, the alarm datapoint in iobroker changes as expected. If I do this with the Pro H I get a warning in the log with newest script:
Greetings, Jürgen |
You can remove the alarm line (its line 21 i think) also in zhimi.airpurifier.vb2.js, then every connection to alarm is cut. I'll do this also for the pullrequest to node-mihome. |
Yes, thanks, I removed the line with alarm and also the lines with functions setBuzzer and getBuzzer. No mor warnings or errors in log if I change somethin in the app. 👍 Thanks for pullrequest. I expect, that this will then solve the problem of the deleted file after iobroker upgrade, as it will be in the "code" in the future?! |
Hi,
I have two 3H which work just fine. Now I bought a air purifier Pro H (larger version, HEPA-Filter) an all I get in iobroker via your script is the info line with all data. Model is zhimi.airpurifier.vb2 . I also get the token, so communication seems fine, just no other data like 'air purifier', 'environment', 'filter' and so on.
Is there a possibility to add this new device? Should I add an issue at https://github.com/maxinminax/node-mihome as your script is using node-mihome and probably the device needs to be added there? I would bet it is more or less the same as zhimi.airpurifier.v7 which is the air purifier Pro (without H)
Greetings, Jürgen
The text was updated successfully, but these errors were encountered: