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

Can set group brightness but not colour temperature #447

Open
CliffS opened this issue Dec 2, 2020 · 9 comments
Open

Can set group brightness but not colour temperature #447

CliffS opened this issue Dec 2, 2020 · 9 comments

Comments

@CliffS
Copy link

CliffS commented Dec 2, 2020

It seems that there is now no way to set the colour temperature on a group. In order to emulate, for example, the old FOCUS scene, I can set group.setBrightness(100); or group.operateGroup({dimmer: 100}); but there doesn't seems to be a way to set colorTemperature on the group, only on each light individually.

Is this a defect in the gateway or should code be added to node-tradfri-client to allow for it?

@AlCalzone
Copy link
Owner

I've double-checked in the IKEA app's code. Groups don't have the necessary property to set the color temperature.
AFAIK it should still be possible through scenes though?

@CliffS
Copy link
Author

CliffS commented Dec 2, 2020

it should still be possible through scenes

It means setting up a global scene for each room for each of the old scenes. Which wouldn't be so bad if the new scenes actually worked properly. I'm wondering whether it would be possible (and quick enough) to create a temporary global scene on the fly, call it and then delete it.

I do hate upgrades that remove functionality.

@AlCalzone
Copy link
Owner

I do hate upgrades that remove functionality.

I must admit, I haven't really worked with scenes (or moods for that matter). Were the moods group specific?

I'm wondering whether it would be possible (and quick enough) to create a temporary global scene on the fly, call it and then delete it.

Not sure, I'd need to look at the API in depth. Depending on your group size, it might be quicker to iterate trough all devices and set the color temperature separately.

@CliffS
Copy link
Author

CliffS commented Dec 2, 2020

Depending on your group size, it might be quicker to iterate trough all devices and set the color temperature separately.

With 14 'white spectrum' bulbs in the group, it takes around 9 seconds. But I've just realised that there's no create scene function in node-tradfri-client so that will have to do. Thanks for the prompt response.

@AlCalzone
Copy link
Owner

But I've just realised that there's no create scene function in node-tradfri-client so that will have to do.

It should be possible to create one but I'm not sure if and when I'll get to that. Lots to be done in other projects currently...

@CliffS
Copy link
Author

CliffS commented Dec 2, 2020

Lots to be done in other projects currently

As always. I do appreciate all the hard work you put into this low-level stuff. Feel free to close this issue if you wish or leave it for reference, as you prefer.

@CliffS
Copy link
Author

CliffS commented Dec 31, 2020

Depending on your group size, it might be quicker to iterate trough all devices and set the color temperature separately.

@AlCalzone This doesn't work, in practice for a large group. Some bulbs get set correctly, some don't. I think perhaps the promise resolves too early when setting the colour.

Another issue since the change to the controller is that setting the brightness via the group seems to work rarely. After a reboot of the controller it works once or twice then stops working. I'm pretty sure this must be a controller bug.

@AlCalzone
Copy link
Owner

I think perhaps the promise resolves too early when setting the colour.

We can only work off what the gateway tells us. When it acknowledges the command, there's no way to know that it didn't work (I fear)

@emmettprexus
Copy link

Some bulbs get set correctly, some don't. I think perhaps the promise resolves too early when setting the colour.

I've just had the same issue. Had to implement a delay between updating the individual bulbs (100ms) and it worked flawlessly. I'm iterating through 25 bulbs and it looks a little funky, but it works. I think if I added a short transition period it would look better, just haven't gotten around to do that yet.

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

No branches or pull requests

2 participants