Koenkk / zigbee2mqtt

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

Aqara RTCGQ11LM (lumi.sensor_motion.aq2) reports Illuminance in Lux twice #24699

Open RubenKelevra opened 4 days ago

RubenKelevra commented 4 days ago

What happened?

I've noticed that the Aqara RTCGQ11LM (self reporting as vendor: LUMI, product: lumi.sensor_motion.aq2) is reporting Lux twice.

I think this stems from their other product Xiaomi GZCGQ01LM (self reporting as vendor: LUMI, product: lumi.sen_ill.mgl01) which is exposing an internal measurment under illuminance, which is there correctly identified as "Raw measured illuminance" via the description.

The Aqara RTCGQ11LM however shows the same lux measurement twice. So I think we can skip one of them?

Screenshot_20241109_194627 Screenshot_20241109_194645 Screenshot_20241109_194708 Screenshot_20241109_194722

What did you expect to happen?

No response

How to reproduce it (minimal and precise)

No response

Zigbee2MQTT version

1.41.0

Adapter firmware version

20221226

Adapter

Slaesh's CC2652 RB stick

Setup

Add-On on Home Assistant OS on RasPi 4B

Debug log

No response

Koenkk commented 3 days ago

illuminance_lux is deprecated indeed, I will remove it with Zigbee2MQTT 2.0.0 (https://github.com/Koenkk/zigbee2mqtt/discussions/24198)

RubenKelevra commented 3 days ago

@Koenkk thanks. Please note that illuminance_lux is however the useful value for devices like previously mentioned Xiaomi GZCGQ01LM:

Screenshot_20241110_185344 Screenshot_20241110_185359

So I guess your plan is to move illuminance_lux to illuminance then for those devices, and add a field like illuminance_raw or something for the raw value?

Koenkk commented 3 days ago

So I guess your plan is to move illuminance_lux to illuminance then for those devices, and add a field like illuminance_raw or something for the raw value?

Just illuminance (which is the value in lux), no raw value (that's legacy)

RubenKelevra commented 3 days ago

Ah, alright. Yeah, most people won't ever use it anyway.