search

raulgbcr / lednetwf_ble

Home Assistant custom integration for LEDnetWF devices
MIT License
24 stars 2 forks source link

Set R-G-B channel order #16

Closed OctarineLobster closed 7 months ago

OctarineLobster commented 7 months ago

Would it be possible to add an option to switch the Blue and Green channels?

Great add on! Thanks.

8none1 commented 7 months ago

I worked out the LED settings protocol today and documented it here: https://github.com/8none1/zengge_lednetwf?tab=readme-ov-file#led-settings The problem comes with how to expose this information in Home Assistant.

At the moment we only use the standard LIGHT UI which doesn't have provision for any kind of drop downs.

I'd like to work out how to add this, but it might take some time.

In the meantime, you should be able to configure your lights using the phone app and that setting /should/ stick.

raulgbcr commented 7 months ago

So I guess there are 2 approachs here. Either we handle this "internally" and swap the channels before sending the HSV value, or somehow send that packet to change the handling on the device itself. Configs could be added in the configuration section for the integration, but not sure if we could send a packet from there to update the setting in the device

8none1 commented 7 months ago

Oh! I'm an idiot. There is a configuration screen... image

We might be able to add it to that?

8none1 commented 7 months ago

Hmm. Problem...

If we accept user input to change the colour ordering via that packet then we MUST also know the number of LEDs in the strip, the number of segments and the LED type because all of those are set in the same packet.

If we guess those then we risk breaking an already working set up.

There must be a way of reading the current configuration from the device.

I'll endevour to take a look tomorrow.

raulgbcr commented 7 months ago

Honestly, if changing it on the original app persist it I wouldn't really risk it here.

8none1 commented 7 months ago

Fair point. +1

8none1 commented 4 months ago

This is now supported in the latest release.