search

Koenkk / zigbee2mqtt

Zigbee 🐝 to MQTT bridge 🌉, get rid of your proprietary Zigbee bridges 🔨
https://www.zigbee2mqtt.io
GNU General Public License v3.0
12.2k stars 1.68k forks source link

Motion vs Occupancy class on devices #23770

Open McGiverGim opened 2 months ago

McGiverGim commented 2 months ago

What happened?

I've observed that the "icon" in Home Assistant for some motion detection devices (like the RTCGQ11LM from Aqara) is wrong. There are two "similar" classes for this kind of devices, motion and occupancy. For a motion detection device, like Aqara, the correct one must be the first, but it's using the second one. Here you have a sample: image

The first one is the Aqara with "occupancy", and the second one has the "motion". I don't know when it changed but some time ago it was ok.

What did you expect to happen?

To have the correct icon based on the device.

How to reproduce it (minimal and precise)

Simply look at the icon in Home Assistant for this kind of motion devices.

Zigbee2MQTT version

1.39.1

Adapter firmware version

20230507

Adapter

egony ebyte

Setup

Addon Home Assistant on HAOS on Raspberry Pi

Debug log

No response

McGiverGim commented 1 month ago

I've found the culprit of that: https://github.com/Koenkk/zigbee2mqtt/pull/22896

It seems it was an intended change. So I suppose it will not be a fix for it.

AndreKR commented 1 month ago

Just to be sure that you know: When you open the entity settings there is a "Show as" setting that you can switch to "Motion":

image

This will bring back the running man.

McGiverGim commented 1 month ago

Yes, that's the workaround I used, but I continue thinking the correct fix is to change it at the beginning.