[go: up one dir, main page]

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

Option for strictly use of xy color #1024

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

Paalap
Copy link
Contributor
@Paalap Paalap commented Jun 27, 2024

I've experienced an issue with some third party devices on the hue bridge, as they would not work with rgb or hsv. A manual entry of strictusexy = true in the light's protocol config allows you to fix this issue.

I've experienced an issue with some third party devices on the hue bridge, as they would not work with rgb or hsv. A manual entry of strictusexy = true in the light's protocol config allows you to fix this issue.
@hendriksen-mark
Copy link
Member

from functions.colors import convert_rgb_xy, hsv_to_rgb is missing in the import section.
where does light.protocol_cfg["strictusexy"] come from? its not in the light_types.py or is it added manually to the lights config file?

@Paalap
Copy link
Contributor Author
Paalap commented Jun 30, 2024

@hendriksen-mark : my bad - now it is in;
The entry in the protocol cfg is assumed to be done manually (for this was/is ok, as I had the issue only with 2 devices)

@hendriksen-mark
Copy link
Member

Its also better to include it in the discovery with False so people dont have to guess where to put it.

@hendriksen-mark
Copy link
Member

What is the make and model of the lights you need this for?

@Paalap
Copy link
Contributor Author
Paalap commented Jul 1, 2024

Tuya RGBW Controller or to be detailed: TSO504B / TZ3210_elzv6aia

And yes running on z2m the issue wasn't there but in my particular case I want to run it on the hue bridge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants