First of all thanks for great products!
I’m in the process of writing an app that uses the switch LED to show status of sensors and I ran into two issues:
Please add saturation as a possible value in startNotification (param 8 or 16). You have it as a setting in the normal led color but not there. I was going to set the LED to notify white when I realized I can’t. Maybe reserve 4 bytes for Hue, Saturation, Brightness, and Effect and another few bytes for auto-off and other fancies? In any case, expose saturation please.
Please normalize the effect list between dimmer and switch. As it is now, fast-blink on switch and chase on dimmer both use 02 as the sub-value. Just use the same list of effects on both devices and ignore any effects (chase for now) on the switch. Alternately, start at 16 or 128 with dimmer (multi-led) effects? In any case, don’t make me add a conditional when I want to blink depending on it being a switch or dimmer.
2nd param on startNotification
Looking at the driver, I don’t really see how the 2nd optional param (1st in stopNotification) does anything at all. What’s the story there?
Also, @nathanfiscus your https://nathanfiscus.github.io/inovelli-notification-calc/ rocks and the hex mode helped me decode what is actually going on.