dominicusmento / CSharpTradFriLibrary

This is a .NET Standard (2.0) library to communicate with the IKEA Home (Tradfri) ZigBee-based Gateway.
GNU General Public License v3.0
37 stars 20 forks source link

[request] implement wrapper for setting CIE Yxy colors #25

Closed juliusdeblaaij closed 4 years ago

juliusdeblaaij commented 5 years ago

I have made use of the preset Tradfri colors in the library. However when I wanted to create custom colors by setting the value in coap 5709 and 5710 there really weren't any function available. Perhaps someone could take a crack at implementing it, as I find great use in the library! Some material to explain what CIE colors are: Info on the colors: https://github.com/ffleurey/ThingML-Tradfri

If anyone needs additional info, I can hopefully supply it.

"implementation requirements"

dominicusmento commented 4 years ago

I've implemented the wrapper using the exact same algorithm for conversion as they've used in linked application. I need someone with color bulbs to test it. The Variable Temperature Bulbs which I have always sets some warm color no matter the color chosen.

Tradfri UI which supports this is here: feature/#25-color-wrapper

domjancik commented 4 years ago

I have tested and fixed the library code that deals with setting X and Y values in the bulbs. See the pull request

dominicusmento commented 4 years ago

This one is now available. Thanks @dominikjancik for testing and fixes.