home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
69.87k stars 28.97k forks source link

Not all lights with label Slapen are turned off #116502

Open pjannink opened 2 months ago

pjannink commented 2 months ago

The problem

I edited a script replacing appr 14 devices with a label 'Slapen'. Of course all devices involved are labeled with 'Slapen'. Strange enough, only 3 lights do turn off, all others remain on. Seems that using the label is not stable/reliable yet

What version of Home Assistant Core has the issue?

2024.4.4

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Core

Link to integration documentation on our website

No response

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

mchangsp commented 2 months ago

Can you publish your script here? And what protocol is each device using? It could be that Home Assistant is executing the command correctly, but that the device does not respond.

pjannink commented 2 months ago

Here you are:

slapen: alias: Slapen sequence:

I disabled the original service light turn off with all Hue lights and Hue power switches.. I kept enabled the shorter light turn off service based on label slapen. See also screenprint:

image

Also tested with 2024.5, still not working via the labeling.

mchangsp commented 2 months ago

Before there were labels I switched off a mix of Zigbee and Philips Hue lights off (maybe 4-6). Many times some lights stayed on. It became more reliable if I paused after switching off several lights. It could be that all lights are switched off shortly after each other and that the Zigbee stack cannot cope with that many commands in a short time span?

mchangsp commented 2 months ago

As a workaround use label_devices(label_name_or_id) to get all devices with your label of choice.

https://www.home-assistant.io/docs/configuration/templating/#labels

Then in a loop turn one device off and then wait 1 second. Curious to see what happens

pjannink commented 2 months ago

I tested with all devices on and the (disabled) service works fine, so no overload for Hue. Weird that the service using a label only turns off 2 or 3 lights. I wonder why using a label seems to have a different execution of the service than the (I assume) one by one processing of a list of devices. I think the HA team has to look at this or should advise not to use labels in Automations of Scripts.

mchangsp commented 2 months ago

OK Thanks for trying!