openhab-addons icon indicating copy to clipboard operation
openhab-addons copied to clipboard

[ipcamera] Fix Reolink Duo Floodlight whiteLED Channel

Open simmonyau opened this issue 10 months ago • 2 comments

Updated whiteLED channel for reolink ipcamera so that when the light is turned on, it will stay on forever and if the light is turned off, it will stay off forever. This is achieved by setting the floodlight mode to "Always On at Night" when the floodlight is turned on and to Off when the floodlight is turned off. Prior to this change, the mode would be set to Auto when turning the floodlight off and to Off when setting the floodlight ON and then back to Auto if you change the brightness via the dimmer.

This was tested and confirmed on a Reolink Duo Floodlight POE camera and is working well for me. I believe this setting would be more ideal for most users but I could be wrong. I just thought it was odd that the floodlight wouldn't stay on longer than 3 minutes after the brightness was set to 100. The same thing probably could have been achieved if mode is set to 0 for all 3 settings.

Signed-off-by: Simmon Yau [email protected]

simmonyau avatar Apr 19 '24 05:04 simmonyau

I'm unable to asses if these changed modes are good. Code is fine. Need @Skinah to determine if this is ok to merge. In the mean time it would be nice if you can fix the DCO.

lsiepel avatar Apr 28 '24 18:04 lsiepel

I'm unable to asses if these changed modes are good. Code is fine. Need @Skinah to determine if this is ok to merge. In the mean time it would be nice if you can fix the DCO.

Done.

simmonyau avatar May 02 '24 05:05 simmonyau

Gentle ping @Skinah

lsiepel avatar May 10 '24 10:05 lsiepel

LGTM I am not able to test as I don't own any Reolink cameras so thanks for the testing and changes you have made.

Skinah avatar May 11 '24 11:05 Skinah