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

Implement infrared color packets to control LIFX+ #40

Closed
MariusRumpf opened this issue Oct 14, 2016 · 5 comments
Closed

Implement infrared color packets to control LIFX+ #40

MariusRumpf opened this issue Oct 14, 2016 · 5 comments

Comments

@MariusRumpf
Copy link
Owner

The new LIFX+ lamp does use a different message to set it's "color" since it uses infrared light. These message have to be implemented to control these. Documentation can be found here.

@ristomatti
Copy link
Collaborator

@MariusRumpf I would like try to help with this and also #41. I have pre-ordered both LIFX Z and LIFX+ but my experience with protocol level communication is limited and I would probably need some support or pointers along the way.

My only experience is from last week, trying to understand the inner workings of node-lifx to improve its performance when sending commands to multiple lights. With trial, error and luck I found two distinct ways to do that. I'm planning on creating separate pull requests on these (it might take a while before I find time to do that though). In case I get that far - would the pull requests be sufficient or should I also create issues to link them to?

@MariusRumpf
Copy link
Owner Author

Yeah, feel free to do so and add pull requests then, no separate issues needed. Maybe I should add a gitter chat so you we can talk when needed.

@ristomatti
Copy link
Collaborator

ristomatti commented Oct 22, 2016

That would be really helpful especially when I get the new devices and need to try and make sense of the protocol specification. But I also have some questions about architectural decisions related to the performance improvements. Would it be ok to contact you through email?

@MariusRumpf
Copy link
Owner Author

MariusRumpf commented Oct 24, 2016

@MariusRumpf
Copy link
Owner Author

Thanks for all the work, this is now part of the 0.8.0 release.

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

No branches or pull requests

2 participants