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

Update L_RGBController1.lua #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Bekkie
Copy link

@Bekkie Bekkie commented Oct 12, 2015

Add support for the MySensors RGB contoller.

See also this forum: http://forum.mysensors.org/topic/2160/my-mysensors-rgbw-plug-in-has-an-init-problem

@vosmont
Copy link
Owner

vosmont commented Jan 28, 2016

Hello Bekkie,
I'm about to release a new version of RGBController (2.0) with a new main RGB controller in order to spare resources on the Vera.
I was checking if your modification was working with this new architecture... there's no more code about RGBController in the MySensors sources ? Have you found a new way to do what you wanted ?

@Bekkie
Copy link
Author

Bekkie commented Jan 28, 2016

The original issue is still there. The issue that master/slave controlled plugins are initialized twice by Vera. But also my workaround still works, meaning manually create a RGB plugin and configure the ID in the RGB plug in.
BTW: the pull request is still open did you pull my code in version 2.0?

@vosmont
Copy link
Owner

vosmont commented Jan 28, 2016

Yes I have pulled your code in the next version.
I have some tests to do before, because adding a main controller have some impacts on existing systems.

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.

None yet

2 participants