syssi / xiaomi_fan

Xiaomi Mi Smart Fan integration for Home Assistant
Apache License 2.0
428 stars 118 forks source link

dmaker.fan.1c issues #117

Closed sokopower closed 3 years ago

sokopower commented 3 years ago

You close all issues about su dmaker.fan.1c but not work it for me.

verification said:

_Invalid config for [fan.xiaomi_miiofan]: value must be one of ['dmaker.fan.p10', 'dmaker.fan.p11', 'dmaker.fan.p5', 'dmaker.fan.p9', 'leshow.fan.ss4', 'zhimi.fan.sa1', 'zhimi.fan.v2', 'zhimi.fan.v3', 'zhimi.fan.za1', 'zhimi.fan.za3', 'zhimi.fan.za4'] for dictionary value @ data['model']. Got 'dmaker.fan.1c'. (See ?, line ?).

I have 2 units of Mi Smart Stading Fan model JLLDS01XY Could you help me about dmaker.fan.1c, please?

Thank you very much! 🙏🏼 Regards

syssi commented 3 years ago

The device is supported of the development branch. A Home Assistant release (2021.5.1) is required before I can release this custom component. I hope the bugfix release of HA will happen this weekend.

sokopower commented 3 years ago

Thank you so much for your answer. I appreciate very much your work, I think is very helpfull.

asifma commented 3 years ago

core-2021.5.1 was just released. Tried this custom component using the dev branch. But get this platform error:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config.py", line 853, in async_process_component_config
    platform = p_integration.get_platform(domain)
  File "/usr/src/homeassistant/homeassistant/loader.py", line 493, in get_platform
    cache[full_name] = self._import_platform(platform_name)
  File "/usr/src/homeassistant/homeassistant/loader.py", line 498, in _import_platform
    return importlib.import_module(f"{self.pkg_path}.{platform_name}")
  File "/usr/local/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 783, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/config/custom_components/xiaomi_miio_fan/fan.py", line 12, in <module>
    from miio import (  # pylint: disable=import-error
ImportError: cannot import name 'FanC1' from 'miio' (/usr/local/lib/python3.8/site-packages/miio/__init__.py)
syssi commented 3 years ago

I've pushed a fix. Please update the custom component and try again!

syssi commented 3 years ago

If you see additional issues please report it here. As soon the device support looks good I will prepare new release of this custom component (installable via HACS).

asifma commented 3 years ago

Still getting the following error:

Logger: homeassistant.helpers.service
Source: helpers/service.py:128
First occurred: 10:28:22 PM (3 occurrences)
Last logged: 10:28:25 PM

Unable to find referenced entities fan.xiaomi_smart_fan

And also:

Logger: homeassistant.components.fan
Source: /usr/local/lib/python3.8/asyncio/events.py:81
Integration: Fan (documentation, issues)
First occurred: 10:31:44 PM (1 occurrences)
Last logged: 10:31:44 PM

Setup of fan platform xiaomi_miio_fan is taking over 10 seconds.
syssi commented 3 years ago

Are you sure about your token and IP address?

syssi commented 3 years ago

Please check the developer tools (dev-state). Do you see a fan.* entity here? This custom component registers a single fan entity. Don't expect a device or integration showing up.

JeeKaheL commented 3 years ago

Mine are working perfectly with the previous yaml configuration ant tokens, those that worked with the previous fork and before Home Assistant breaking change. Now it is well back and official to this add-on. Great thanks !

sokopower commented 3 years ago

Works for me too !! Thanks

asifma commented 3 years ago

Sorry, my fault. I think I restored the fan 2 weeks ago, thus a new token was generated. And my config had an old one. Works perfectly now, thanks! :)

marxkemp commented 3 years ago

Yes, it's working here to. Thanks!

syssi commented 3 years ago

Cool! Let's close this issue. Feel free to create a new one if something isn't working as expected.