-
Notifications
You must be signed in to change notification settings - Fork 62
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
More natural color values #33
Comments
+1 of this. Or allow me to drop in a |
I've used this function to convert values to the hex16 value:
However, i'm not sure what ranges hue, brightness etc.. accept. Is that documented anywhere? |
@kevincannon You find the range information in the lifx docs https://github.com/LIFX/lifx-protocol-docs/blob/master/messages/light.md#hsbk |
@MariusRumpf - neat, thanks for the info! BTW - is node-lifx what I should be using with lights with the new Firmware? I noticed some functions broke since I updated. |
node-lifx is still work in progress and missing key features like turning on or off the lights or set the colors. When it is in any working state I will publish it on npm and write docs for it. |
Great, thanks for the info. I was a bit confused. Luckily I still have one LIFX with the old firmware, so I can play with that. Great library. It's been really amazing to use it! Thanks so much. |
I saw that the official API uses int16 numbers for the colours, but it feels unnatural to me.
Would be nice to make abstraction of that at let us use a more natural writing style. something like:
lx.lightsColour(170, '50%', '45%', .6, '1000ms')
or maybe just provide a helper function that will do the math.I think it would make it dramatically easier to use lifxjs for anyone likes me who comes from the frontend side of JS.
For me this is an issue, but please consider it as a suggestion.
The text was updated successfully, but these errors were encountered: