-
Notifications
You must be signed in to change notification settings - Fork 16
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
Support for AirPurifier 2H #6
Comments
The 2H should work, but is completely untested. It was also recognized from the script I see, so, lets look where the errors come. |
I have updated to version 0.2.7, however the function with the 2H is not yet given. Here is the data you wanted.
|
You forgot one important answer: Whats your JavaScript Adapter Version? If you are an expirienced user, you can also take a look at #4 |
The JavaScript adapter is on version 4.10.13. At the time of the error I tried to set the object "on" to true. I'll take a look at your link and let you know if anything changes. |
I tried a little more today. In the meantime, values are displayed and I can also switch the AirPurifier on and off. Only the temperature is not displayed correctly. This should be divided by 10, but it is not. Even changing the divisor to 100 did not change anything. I have also added the units for some objects. Maybe you can adjust the objects to the other AirPuriefiers (subfolders) and solve the temperature problem. I will test all other functions tomorrow and give feedback. Attached are my changes. Changes to the ioBroker script:
File: zhimi.airpurifier.mc2.js
|
I have just been able to test everything. The buzzer does not work. Neither the read out, nor the set. I'll have to check again when I have time. Here is the current state.
|
The buzzer works now, too. Only the problem with the temperature remains. This is still displayed incorrectly. At 20.7°C, 207°C is displayed. Any idea how I can change this? Could compensate for this on ioBroker side, but that is not the purpose. |
I have now tried several things to read out the temperature correctly. However, I have the assumption that it is not at node-mihome, but at the script. Maybe your script can't handle comma numbers? The situation looks like this: Enclosed my attempts on sides of node-mihome. Test #1
Test #2
Test #3
|
The Skript dont uses all that get Commands, it uses the raw data defined in this._propertiesToMonitor. The shown value is crazy but correct. temp_dec means 10th of temp and this means its *10. I'll try to write a function to watch for this temp_dec and correct it then, wait for next version. Good work btw.! |
I have also changed something in your script. You use format: "boolean" etc., I would solve it with type: "boolean" etc.. I also added units. Attached is my current version:
|
What is the current status? Is there anything else I can do? |
If still everything works fine, nothing do to for you. I've integrated and modified the skript, and the next days make a pr to node-mihome. |
Everything works for me so far, except for the problem with the temperature (instead of 17.6°C, 176°C is displayed). |
For the temp Problem there is an workaround in the new scriptversion, dont worry. |
I just updated to version 0.2.8. The temperature is now displayed correctly. But I noticed one thing regarding the AirPurifier 2H. There are still changes to be made in the setter section. And the device is now Tested / Working!
|
I own an AirPurifier 2H and run it through the iobroker-miio adapter. I find your script very promising. I also plan to buy more MiHome devices, so I would like to switch to your script.
Unfortunately, the 2H does not yet work with your script and is rightly still on the status "in Progress".
I would therefore be happy to support you and provide you with all the necessary data. I will therefore start with a few logs and screenshots.
The text was updated successfully, but these errors were encountered: