search

syssi / philipslight

Xiaomi Philips Lights integration for Home Assistant
Apache License 2.0
70 stars 19 forks source link

Custom component doesn't start properly #34

Closed djmalin closed 2 years ago

djmalin commented 4 years ago

Hello @syssi i have Home Assistant 0.106.0b5 and this lamp Xiaomi Philips Smart LED Ball, so far everything works on version 105 so now the lamp stopped working and such an error appears

WARNING (MainThread) [homeassistant.loader] You are using a custom integration for xiaomi_miio_philipslight which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you do experience issues with Home Assistant. 2020-02-26 07:27:11 ERROR (MainThread) [homeassistant.config] Platform error: light Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/config.py", line 752, in async_process_component_config platform = p_integration.get_platform(domain) File "/usr/src/homeassistant/homeassistant/loader.py", line 277, in get_platform f"{self.pkg_path}.{platform_name}" File "/usr/local/lib/python3.7/importlib/init.py", line 127, in import_module return _bootstrap._gcd_import(name[level:], package, level) File "", line 1006, in _gcd_import File "", line 983, in _find_and_load File "", line 967, in _find_and_load_unlocked File "", line 677, in _load_unlocked File "", line 728, in exec_module File "", line 219, in _call_with_frames_removed File "/config/custom_components/xiaomi_miio_philipslight/light.py", line 11, in from homeassistant.components.light import ( ImportError: cannot import name 'ATTR_ENTITY_ID' from 'homeassistant.components.light' (/usr/src/homeassistant/homeassistant/components/light/init.py)

syssi commented 4 years ago

This custom component needs some changes to be 0.106 compatible. Are you able to use the official component (xiaomi_miio). What's missing there?

djmalin commented 4 years ago

ok, it works on the official version, I did not know that there is official support :), everything is needed

kooyeed commented 3 years ago

I have the same problem, I have Home Assistant 0.116.0b4.

TomDeSchutter commented 3 years ago

I just tried implementing this myself to control a new bulb; looks like I'm encountering the same problem. Running HA 0.117.0. I'm clueless.

Configuration invalid

Platform error light.xiaomi_miio_philipslight - cannot import name 'ATTR_ENTITY_ID' from 'homeassistant.components.light' (/usr/src/homeassistant/homeassistant/components/light/init.py)

syssi commented 3 years ago

I will care about. :-)

syssi commented 3 years ago

Fixed.

TomDeSchutter commented 3 years ago

As much as I admire the speed of your response, I'm afraid not.

Logger: homeassistant.components.light Source: custom_components/xiaomi_miio_philipslight/light.py:175 Integration: Light (documentation, issues) First occurred: 23:13:53 (1 occurrences) Last logged: 23:13:53

Error while setting up xiaomi_miio_philipslight platform for light Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 197, in _async_setup_platform await asyncio.shield(task) File "/config/custom_components/xiaomi_miio_philipslight/light.py", line 175, in async_setup_platform light = PhilipsBulb(host, token) NameError: name 'PhilipsBulb' is not defined

syssi commented 3 years ago

@TomDeSchutter Which Home Assistant version do you use? Please try the most recent version (0.117.1).

syssi commented 3 years ago

New HA versions respects the manifest.json and retrieves all requirements. In this case python-miio. This library provides the missing class. It looks like your HA instance doesn't provide this dependency.