basnijholt / adaptive-lighting

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

Avoid sending turn on command to bulbs already on. #703

Closed IIV-NATHAN-VII closed 1 year ago

IIV-NATHAN-VII commented 1 year ago

Not sure if this is possible but looking at the logs I see adaptive lighting sends the turn on command to my tapo bulbs when they've just been turned on at the power, this causes them to flash the first time as it seems to trigger some sort of visual reset which is quite annoying.

I have skip redundant commands enabled

Home Assistant Adaptive Lighting Issue Template

Bug Reports

If you need help with using or configuring Adaptive Lighting, please open a Q&A discussion thread here instead.

Before submitting a bug report, please follow these troubleshooting steps:

Please confirm that you have completed the following steps:

Required information for bug reports:

Please include the following information in your issue.

Issues missing this information may not be addressed.

  1. Debug logs captured while the issue occurred. See here for instructions on enabling debug logging:
  1. Your Adaptive Lighting configuration:
  1. (If using Zigbee2MQTT), provide your configuration files (remove all personal information before posting):
    • devices.yaml
    • groups.yaml
    • configuration.yaml ⚠️; Warning REMOVE ALL of the PERSONAL INFORMATION BELOW before posting ⚠️;
      • mqtt: server:
      • mqtt: user:
      • mqtt: password:
      • advanced: pan_id:
      • advanced: network_key:
      • anything in log_syslog if you use this
    • Brand and model number of problematic light(s)


4.  Describe the bug and how to reproduce it:

5. Steps to reproduce the behavior:
basnijholt commented 1 year ago

Can you try setting adapt_delay? See the option here: https://github.com/basnijholt/adaptive-lighting#memo-options

IIV-NATHAN-VII commented 1 year ago

I have adapt delay set to 4 seconds

basnijholt commented 1 year ago

I do not understand the problem. To avoid sending turn_on command by AL, you need to disable the switch?

Also check out https://github.com/basnijholt/adaptive-lighting/issues/44

basnijholt commented 1 year ago

I am closing this because the problem you described is discussed in https://github.com/basnijholt/adaptive-lighting/issues/44.

Let me know if you think otherwise 😄

IIV-NATHAN-VII commented 1 year ago

Can't you send an update brightness/colour command without turn on?

basnijholt commented 1 year ago

No, the only way to make the update is by light.turn_on: https://www.home-assistant.io/integrations/light/#service-lightturn_on