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

add support for setting the light color via rgb? #36

Closed
wez opened this issue Jun 1, 2023 · 9 comments
Closed

add support for setting the light color via rgb? #36

wez opened this issue Jun 1, 2023 · 9 comments
Labels
wontfix This will not be worked on

Comments

@wez
Copy link

wez commented Jun 1, 2023

I expose certain devices to HomeKit for ease of use by the rest of my family.

It would be fantastic if this integration could support parsing an rgb color request into the closest matching pre-defined static color effect so that setting the color of the lights is more natural and easy.

@cryptk
Copy link
Owner

cryptk commented Jun 1, 2023

The issue with that is I don't know what the RGB values of the existing colors are, all I have from the Omni API are names of colors and shows. I also wouldn't know where to begin with writing a "best fit" style algorithm to map the colors around like that.

Unfortunately, I think we are going to have to stick with a list of effects

@wez
Copy link
Author

wez commented Jun 1, 2023

Would you be opposed to a PR that takes a stab at implementing this?

@cryptk
Copy link
Owner

cryptk commented Jun 1, 2023

Have at it! PR's are very welcome!

@cryptk
Copy link
Owner

cryptk commented Jun 1, 2023

I do have one reservation about the functionality. I'm concerned that it might lead people to believe that the integration provides functionality beyond what it actually does. If someone sends an RGB value and they get something different than what they request, that may lead them to believe that there is something wrong with the integration. I am a bit worried about getting tickets such as "I set the light to light purple, but I got deep purple instead".

@wez
Copy link
Author

wez commented Jun 1, 2023

Understandable; I think that provided that the reported color value quickly snaps to the approximated value it will feel reasonably clear to folks while they are selecting colors in the color wheel or whatever UI they are using. I've used a couple of devices with similar behavior, and it wasn't especially jarring.

@cryptk
Copy link
Owner

cryptk commented Jun 1, 2023

Sounds good, let me know if you get something working and I'll be happy to test it out!

@irlkitcom
Copy link

This would be great, I use HomeKit Bridge and the effects can't be passed to HomeKit but I believe I could get an RGB switch going!

@cryptk
Copy link
Owner

cryptk commented Jan 6, 2024

@irlkitcom one thing to know, this would not allow you to show any color on the light... If it were implemented (which I have no plans to implement it myself) then all it would be able to do is select a "closest match" color from the ones already available to whatever RGB you asked for.

@cryptk
Copy link
Owner

cryptk commented Jun 11, 2024

The more I think about this, I don't think there is a reasonably way to implement a "Tell the light RGB, but have it auto-select a preset color from the very limited set that kinda matches" without it creating a lot of confusion and issue reports.

Additionally, I want to keep the underlying library, and the integration, to functionality that is actually supported by the controller. If people need a "best match" algorithm of sorts, they should implement that outside of the integration and then have that request the integration to perform an action that the lights are actually capable of.

I'm going to close this as WONTFIX.

@cryptk cryptk closed this as completed Jun 11, 2024
@cryptk cryptk added the wontfix This will not be worked on label Jun 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

3 participants