Skip to content
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

Color Temperature Range incorrect for Aqara LED Bulb ZNLDP12LM #53022

Closed
mouth4war opened this issue Jul 14, 2021 · 5 comments · Fixed by #54601
Closed

Color Temperature Range incorrect for Aqara LED Bulb ZNLDP12LM #53022

mouth4war opened this issue Jul 14, 2021 · 5 comments · Fixed by #54601
Assignees
Labels
integration: xiaomi_miio waiting-for-upstream We're waiting for a change upstream

Comments

@mouth4war
Copy link

The problem

Xiaomi_Miio integration light as subdevice of gateway (Zigbee).

It currently reports:
min_mireds: 153
max_mireds: 500 (2000 K)

It should report:
min_mireds: 153
max_mireds: 370 (2700K)

What is version of Home Assistant Core has the issue?

core-2021.7.2

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Supervised

Integration causing the issue

xiaomi_miio

Link to integration documentation on our website

https://www.home-assistant.io/integrations/xiaomi_miio/

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@mouth4war
Copy link
Author

Not sure if color temperature is being provided by miio library @starkillerOG

https://github.com/home-assistant/core/blob/dev/homeassistant/components/xiaomi_miio/light.py

@mouth4war
Copy link
Author

Fix? rytilahti/python-miio#1098

@starkillerOG
Copy link
Contributor

@mouth4war thanks!
You are correct, the min/max cct is reported by the python-miio library and your PR is correct and schould fix the issue.
I don't own any xiaomi bulbs so I did not know the actual values.
Thanks for correcting this!

Any other issues with the xiaomi miio bulbs?

@mouth4war
Copy link
Author

Thanks for confirming! They work fine otherwise. Maybe a little slow in updating status.

@rytilahti rytilahti added the waiting-for-upstream We're waiting for a change upstream label Aug 1, 2021
@rytilahti
Copy link
Member

The slowness of status updates is due to device polling. I assume this (also) is waiting for a new python-miio release to get fixed, so I added the appropriate label.

@github-actions github-actions bot locked and limited conversation to collaborators Sep 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
integration: xiaomi_miio waiting-for-upstream We're waiting for a change upstream
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants