basnijholt / adaptive-lighting

Adaptive Lighting custom component for Home Assistant
https://basnijholt.github.io/adaptive-lighting/
Apache License 2.0
1.76k stars 131 forks source link

Bad behaviour with binding (physical switches) #96

Open jd1900 opened 3 years ago

jd1900 commented 3 years ago

I have most of my switches bound to the lights. And the behaviour (with different brands) is not really good.

If I turn on and then off, the lights turns back on again after 2-3 seconds.

Moreover, the lights just turn on and then start to change the brightness and color temperature, instead of turning on directly changed.

And..., something similar with brightness. Sometimes I dim lights right after turning them on. But Adaptive Lighting set brightness back if I do "too fast" or before Adaptive Lighting "works".

I've tried with and without 'detect_non_ha_changes' option. Also with and without 'only_once' option.

Any ideas of how to fix it?

LEJOUI commented 3 years ago

You could try to decrease the initial_transition to 0 or 1

jd1900 commented 3 years ago

It doesn't improve much.

I don't know if it would be useful, but I can provide a video.

jd1900 commented 3 years ago

It is possible to change colour temperature in some devices without turning them on. Why don't you do that instead of turning on while changing the colour temperature? That way, that would never happen, and would keep state as is.

LEJOUI commented 3 years ago

In the AL switch there’re attributes you could template in the service call to turn the light on.
An example can be found here:

https://github.com/claytonjn/hass-circadian_lighting/wiki/Additional-Functionality#automation

make sure that everything matches the AL switch.

Op 4 mrt. 2021 om 21:44 heeft jd1900 notifications@github.com het volgende geschreven:

 It is possible to change colour temperature in some devices without turning them on. Why don't you do that instead of turning on while changing the colour temperature? That way, that would never happen, and would keep state as is.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

jd1900 commented 3 years ago

In the link you sent me is meant to change the way to turn lights on via HA. However, it is unfeasible when using binding to the lights, as they are directly turned on without HA as an intermediate.

What I mean is that Adaptive Lighting, instead of changing the temperature value using the "turning_on" service, that changed the temperature values not turning the lights on. That way they could change temperature of the lights and when turned on using bound devices there is no delay and noticable change, as they have been already changed while being off.

LEJOUI commented 3 years ago

Oh yeah ofcourse, the binding.. Forgot about that.. Not sure if it's possible what you'd want then if the initial transition isn't working.. I do know that Circadian Lighting used to work like you're describing (Somewhere around release 1.0.9 I thought.). It was constantly updating the temperature colors and brightness in HA. But not sure if that would work in your case. Perhaps someone else has an idea to help you :)

th3w1zard1 commented 1 year ago

@jd1900 Hello! Are you by chance having this issue with the latest adaptive lighting version?

basnijholt commented 1 year ago

I believe this is the same as https://github.com/basnijholt/adaptive-lighting/issues/307

basnijholt commented 1 year ago

The turning on problem should be fixed in v1.19.0.

Turn off detect_non_ha_changes.