ad-automoli icon indicating copy to clipboard operation
ad-automoli copied to clipboard

Undocumented motion sensor name format expection

Open bluehallu opened this issue 3 years ago • 2 comments

😩 Describe the issue/bug My sensor was called binary_sensor.ikea_motion_sensor (that's what the zigbee MQTT entity is called) so I decided to specify it instead of relying on autodetection. This didn't work with automoli complaining in logs that no light/sensors were given. Changing the sensor entity name to binary_sensor.motion_sensor_ikea fixed the problem

😒 Expected behavior Name conventions aren't enforced when sensors specified directly or the documentation clearly states the naming conventions are required even the user doesn't want to rely on autodiscovery

bluehallu avatar Sep 16 '21 00:09 bluehallu

The naming of the lights is nothing AutoMoLi cares about. AutoMoLi has no fixed format or something like that, it just relies on the entities available in Home Assistant

if you have the entity "binary_sensor.ikea_motion_sensor" in home assistant, it should work in AutoMoLi too. So maybe you have confused something?

benleb avatar Sep 17 '21 06:09 benleb

The code declares MOTION = "binary_sensor.motion_sensor_ which prevents the name I had from working. Renaming it as I described fixed the issue right away

bluehallu avatar Sep 24 '21 22:09 bluehallu

The Readme now states that you need to follow the following scheme to work properly

sensor.illumination_room
binary_sensor.motion_sensor_room
binary_sensor.motion_sensor_room_something
light.room

evildad avatar Apr 25 '23 11:04 evildad