-
Notifications
You must be signed in to change notification settings - Fork 118
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 device model zhimi.fan.sa1 #15
Comments
Did you block the internet access of the fan? |
network is ok. maybe it is new xiaomi fan model. https://item.mi.com/1182100014.html _LOGGER.info("%s %s %s detected", log out:
|
This model is new to me. Do you like to provide some support to get your fan integrated? |
I am having the same issue with my Xiaomi fan. I just tried to connect it to your custom component. I would be happy to support in extending this component for the new model. Which Information do you need? |
@TribuneX Could you provide the response/output of:
|
I changed the command to the IP of my fan and the correct token, but I got this: Sending cmd get_prop with params ['led', 'bat_state', 'temp_dec', 'humidity', 'angle', 'speed', 'poweroff_time', 'power', 'ac_power', 'battery', 'angle_enable', 'speed_level', 'natural_level', 'child_lock', 'buzzer', 'led_b', 'use_time', 'bat_charge', 'button_pressed']
ERROR:miio.device:Got error when receiving: timed out
Error: No response from the device I am executing this from my homeassistant docker container with |
Above params have values, others except 'button_pressed' return word, 'None'. |
@syssi Can we further support you on this issue? |
@kikamoo There is no property called |
Some progress: rytilahti/python-miio#354 ;-) |
@kikamoo Could you provide some values of get_prop ['buzzer']? Your example contains a |
@syssi There's no hardware for temperature, that's what I heard. |
Could you try to toggle the buzzer by |
sending [1] or [2] turns the Notification sound ON, |
Is the value 1 another audio volume as 2? :-) |
I could not figure it out. |
Alright! :-) |
When will zhimi.fan.sa1 model support be available? |
@jjongyeonglee Do you like to provide some support? There is some testing needed of the underlying library. Python skills required. |
halo sir ,i have this zhimi.fan.sa1 too.I really hope u support this fan in Hass,.You've worked hard and im so sorry i dont know the phthon. Maybe i can help u test the code in the fan. |
https://github.com/YinHangCode/homebridge-mi-fan can support zhimi.fan.sa1 in hb, I don't know if that does you any good.Very hope ha will be able to support SA1 as soon as possible! |
@syssi I can help with the testing, what do I need to do? |
Please install this development version of python-miio:
A new device (fansa1) will be available via
Please execute every command once and provide some feedback. Does it work properly? Do you receive an error? What's the output of the "status" method? Thanks a lot! As fat as the device is properly tested it can be merged and the Home Assistant counterpart can be implemented. |
Commands that work: Commands that failed: |
Also failed: |
Didn't try "raw_command", don't know what are valid command. |
|
OK, "info" command is working too. |
@syssi Is it possible to check in the current code for support of fansa1 in this repo? I really only need to turn the fan on and off for automation. |
@reymtv Please open a new issue for this model! |
@pustefix8965 Could you provide the output of:
Both commands should return the same values. You've reported |
-d status |
-d raw_command |
Please try to reduce the number of requested properties. Can you provide a working set of properties? |
|
Just 'led' alone is also returning ['null']. I can get angle, return [90] and also child_lock return ['off']. But if I get both angle and child_lock, I only get return of [90]. |
Please remove "led" and try the other :-) |
OK, only "led" is not working, the rest are fine. As mentioned before, I am only able to get one property at a time. |
@pustefix8965 Which model do you own? Could you provide the output of "info"? The firmware version would be interesting. |
|
Are you guys managing to advance on this? Just got this fan and would be happy to help with testing! |
I will update the implementation tonight and will be happy about some more testing afterwards. |
I've pushed some fixes: I've removed the "led" property. If the |
Just tested, |
The |
New error for status: |
@pustefix8965 Fixed! |
I am still having the same error about |
I found and fixed another issue. Try again! |
New error: |
Okay. Another one. I will fix it tonight. |
Fixed. |
One more fix and we are good, line 271 fan.py: |
My output after the fix: |
I will update the Home Assistant component now. |
@syssi Please also update your link for retrieving the access token. The current link is for getting the Xiaomi Gateway key, the correct link should be https://www.home-assistant.io/components/vacuum.xiaomi_miio/#retrieving-the-access-token |
I've updated the custom component. Please update the custom component and make sure HA is using the latest revision of python-miio (./hass --skip-pip). Does it work out of the box? |
Update for fan.xiaomi_smart_fan fails
下午9:48 custom_components/fan/xiaomi_miio_fan.py (ERROR)
unable to parse json '{"result":[null,null,120,468,0,"on","on",null,"on",100,74,"off",2get_down],"id":515}': Expecting ',' delimiter: line 1 column 66 (char 65)
下午9:48 /srv/homeassistant/lib64/python3.6/site-packages/miio/protocol.py (ERROR)
if not set model, log is:
The text was updated successfully, but these errors were encountered: