-
-
Notifications
You must be signed in to change notification settings - Fork 28.6k
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
Xiaomi Philips Ambient Light doesn't work #16269
Comments
Please enable the debug log of the component & library by:
Restart HA and reproduce the issue. Please provide the content of your home-assistant.log. Thanks in advance! |
Sure.
IP address of my Philips lamp is Thanks! |
2018-10-07 12:16:32 DEBUG (Thread-15) [miio.philips_bulb] Count (5) of requested properties does not match the count (2) of received values. It looks three parameters are not working: color_temperature=None, scene=None, delay_off_countdown=None |
@miguelgilmartinez The Philips Eyecare Smart Lamp shouldn't respond as a "miio.philips_bulb". This implementation cannot work. |
I have this lamp and the bug is triggered on that file xiaomi_miio.py
…On Mon, Oct 8, 2018 at 9:19 PM Sebastian Muszynski ***@***.***> wrote:
@miguelgilmartinez <https://github.com/miguelgilmartinez> The Philips
Eyecare Smart Lamp shouldn't respond as a "miio.philips_bulb". This
implementation cannot work.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#16269 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEo6PA30phbq2ls2rbdVMNJydRhLOsE5ks5ui6UzgaJpZM4WRvln>
.
|
@miguelgilmartinez Could you provide some logs (debugging enabled)? Thanks in advance! |
Log Details (ERROR)
Tue Oct 09 2018 21:07:25 GMT+0200 (hora de verano de Europa central)
xiaomi_miio: Error on device update!
Traceback (most recent call last):
File "/home/miguelgilmartinez/homeassistant/lib/python3.5/site-packages/homeassistant/helpers/entity_platform.py",
line 251, in _async_add_entity
await entity.async_device_update(warning=False)
File "/home/miguelgilmartinez/homeassistant/lib/python3.5/site-packages/homeassistant/helpers/entity.py",
line 350, in async_device_update
yield from self.async_update()
File "/home/miguelgilmartinez/homeassistant/lib/python3.5/site-packages/homeassistant/components/light/xiaomi_miio.py",
line 487, in async_update
self.max_mireds, self.min_mireds)
File "/home/miguelgilmartinez/homeassistant/lib/python3.5/site-packages/homeassistant/components/light/xiaomi_miio.py",
line 510, in translate
value_scaled = float(value - left_min) / float(left_span)
TypeError: unsupported operand type(s) for -: 'NoneType' and 'int'
El lun., 8 oct. 2018 22:36, Sebastian Muszynski <notifications@github.com>
escribió:
… @miguelgilmartinez <https://github.com/miguelgilmartinez> Could you
provide some logs (debugging enabled)? Thanks in advance!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#16269 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEo6POCaVxwBipe9MqUf-YJdtzvkYRXzks5ui7dogaJpZM4WRvln>
.
|
Could you provide the xiaomi_miio section of your configuration.yaml, too? |
light:
- platform: xiaomi_miio
name: Xiaomi Philips
host: 192.168.31.153
token: sdsdsdsadsdadadasdasdsadsadasdsad
model: philips.light.bulb
…On Tue, Oct 9, 2018 at 10:11 PM Sebastian Muszynski < ***@***.***> wrote:
Could you provide the xiaomi_miio section of your configuration.yaml, too?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#16269 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEo6PPK1XUN1CDopTZiLpANC3wMtN1KAks5ujQL1gaJpZM4WRvln>
.
|
This is the problem. Your device model must be |
Great! A lot of thanks. I took the sample in
https://www.home-assistant.io/components/light.xiaomi_miio/ This new
parameter is described in "model" field, but I'm afraid it's not enough. I
have added a comment
<home-assistant/home-assistant.io#6786> on Github
documentation . Thanks again.
…On Sat, Oct 13, 2018 at 7:48 PM Sebastian Muszynski < ***@***.***> wrote:
This is the problem. Your device model must be philips.light.sread1. The
bulb is another kind of hardware.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#16269 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEo6PGMAJglAJdXWNAIjzViKu6Xf8OCyks5ukidagaJpZM4WRvln>
.
|
Works fine in 0.80.0, thanks for the fix! |
I'm afraid my patch was not committed to main branch. I was wrong with my
configuration. I have submitted an update to documentation to help other
people with that issue
…On Sun, Oct 14, 2018 at 1:05 PM OleksandrBerchenko ***@***.***> wrote:
Works fine in 0.80.0, thanks for the fix!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#16269 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEo6PNHrvLpkY_AMYxYKJilNBl1bScYIks5ukxqTgaJpZM4WRvln>
.
|
@miguelgilmartinez @OleksandrBerchenko is talking about this change: #17259 |
Well, it works, but there are still some issues with updating state: #17475 |
Home Assistant release with the issue:
0.77.0
Last working Home Assistant release (if known):
n/a
Operating environment (Hass.io/Docker/Windows/etc.):
Virtualenv on Raspberry Pi
Component/platform:
https://www.home-assistant.io/components/light.xiaomi_miio/
Description of problem:
I have Philips Eyecare Smart Lamp (see configuration below). HA automatically detects it as two separate lights:
light.philips_2
(main lamp) andlight.philips_ambient_light_2
(weak lighting at the back). Main lamp works as expected, but Ambient Light doesn't:Below is what I see in the log:
The last state change refers the main lamp.
Thanks!
Problem-relevant
configuration.yaml
entries and (fill out even if it seems unimportant):Traceback (if applicable):
n/a
Additional information:
n/a
The text was updated successfully, but these errors were encountered: