Skip to content

Hue Bridge and Off with effect compatibility#84

Open
mbchristoff wants to merge 8 commits intoluar123:masterfrom
mbchristoff:master
Open

Hue Bridge and Off with effect compatibility#84
mbchristoff wants to merge 8 commits intoluar123:masterfrom
mbchristoff:master

Conversation

@mbchristoff
Copy link
Copy Markdown

@mbchristoff mbchristoff commented Feb 22, 2026

After applying #26 to the current master branch I noticed the light would turn on and applying different colors / brightness would work but turning of didn't.

By utilizing this fix I was able to get full compatibility with a Philips Hue bridge.
espressif/esp-zigbee-sdk#519 (comment)

A new example file is added to show how to use this PR to control the onboard LED from Philips Hue.
I scrambled the Trust center key to prevent a takedown.

@luar123
Copy link
Copy Markdown
Owner

luar123 commented Mar 19, 2026

Sorry for the late reply.
So this is basically #26 + the added attributes?
I am not a big fan of adding attributes in the c++ part without user control. For manual setups you can just define them in the yaml and for basic mode it would be easier to add them here:

@mbchristoff
Copy link
Copy Markdown
Author

Sorry for the late reply. So this is basically #26 + the added attributes?

Yes, it's indeed #26 plus the attributes.
I want to give a fully compatible package for people who stumbled on this repo and wanted to use it with Hue.

I am not a big fan of adding attributes in the c++ part without user control. For manual setups you can just define them in the yaml and for basic mode it would be easier to add them here:

The board I used for testing is already installed in a housing and being used.
I could get another one to implement this the correct way if it's something you'd like to add to the repo.

@luar123
Copy link
Copy Markdown
Owner

luar123 commented Mar 21, 2026

No problem, I will have to rewrite the whole light stuff anyway as it is too limited now. I can include the attributes then and will probably merge the changes from #26 when I have time. Until then we can keep this PR open so others can find it more easily.

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.

2 participants