Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix!: Remove illuminance_lux #8304

Merged
merged 1 commit into from
Nov 11, 2024
Merged

Conversation

Koenkk
Copy link
Owner

@Koenkk Koenkk commented Nov 11, 2024

  • Moved illuminance_lux to illuminance, this breaks in two ways: illuminance_lux has been removed and the value of illuminance changes. This affects the following devices: 83633205, MOSZB-140, MOSZB-153, EFEKTA_iAQ3, EFEKTA_PWS_Max, EFEKTA_PWS_MaxPro, EFEKTA_eON29wz, EFEKTA_eFlower_Pro, EFEKTA_iAQ, E2134, POK004, POK011, ROB_200-070-0, HK-SENSOR-4IN1-A, AU-A1ZBPIRS, RADON TriTech ZB, TS0601_illuminance_temperature_humidity_sensor_1, TS0202_3, S-LUX-ZB, TS0222_light, THE01860A, TS0222, LCZ030, MIR-HE200-TY, XFY-CGQ-ZIGB, TS0601_smart_human_presence_sensor_1, Y1_IN, ZY-M100-S_1, ZY-M100-S_2, WZ-M100, ZG-205ZL, ZG-204ZM, YXZBRB58, ZY-M100-24G, ZY-M100-24GV2, ZY-M100-24GV3, MTG075-ZB-RL, RT_ZCZ03Z, ZY-M100-S_3, ZG-205Z, 8750001213, HSE2927E, ZB003-X, 07502L, JS-MC-SENSOR-ZB, KK-ES-J01W, KMPCIL_RES005, RTCGQ11LM, RTCGQ12LM, b-parasite, 7A-SS-ZABC-H0, 5AA-SS-ZA-H0, RTCGQ14LM, RTCGQ15LM, ZNCLBL01LM, GZCGQ01LM, GZCGQ11LM, 07047L, 9290012607, 9290019758, 9290030675, 9290030674, 545D6306, CCT595011, ZSS-ZK-THL, 4512770, TERNCY-PP01, MBD-S, CTM_MBD_Dim, 3RSNL02043Z, WB-MSW-ZIGBEE v.3, MS100, USM-300ZB, PIR313-E

@Koenkk Koenkk merged commit 1d57246 into feat/21.0.0 Nov 11, 2024
3 checks passed
@Koenkk Koenkk deleted the fix/remove-illuminance-lux branch November 11, 2024 20:04
Koenkk added a commit that referenced this pull request Dec 1, 2024
@CollaVinilica
Copy link

Why would you something like this? WHY? It broke every single sensor with a lux sensor I have. Now I have to waste an afternoon changing back all my entities IDs. At least you could have made it so I wouldn't rename (and disable!) pre-existing entities, c'mon!

@nivek1612
Copy link

The worst thing for me is node-red does not appear to see any illuminance sensor now. So I will be rolling back.

@hardwareadictos
Copy link

hardwareadictos commented Jan 4, 2025

Same here!!! My HA instance doesn't see any illuminance_lux or illuminance sensor (that last should appear at least as this merge states). Rolling back...

imagen

Z2MQTT detects the sensors but HA does not:

imagen

Maybe i'm missing something, but if not, that's going to break instances with light sensors.

@CollaVinilica
Copy link

CollaVinilica commented Jan 4, 2025

Same here!!! My HA instance doesn't see any illuminance_lux or illuminance sensor (that last should appear at least as this merge states). Rolling back...

Try and take a look under the device info, in my case all the lux sensors associated with Aqara devices where disabled.. I had to manually enable them back and then rename their entity IDs to get my automations to work again.

@Schermbecker
Copy link

Schermbecker commented Jan 4, 2025

Try and take a look under the device info, in my case all the lux sensors associated with Aqara devices where disabled.. I had to manually enable them back and then rename their entity IDs to get my automations to work again.

That's the solution, thank you so much! 👍I also wondered, why all my illuminance entities were gone.
The affected entities also have been renamed from sensor._illuminance_lux to sensor._illuminance.

@nivek1612
Copy link

Try and take a look under the device info, in my case all the lux sensors associated with Aqara devices where disabled.. I had to manually enable them back and then rename their entity IDs to get my automations to work again.

That was it thanks.

@hardwareadictos
Copy link

Same here!!! My HA instance doesn't see any illuminance_lux or illuminance sensor (that last should appear at least as this merge states). Rolling back...

Try and take a look under the device info, in my case all the lux sensors associated with Aqara devices where disabled.. I had to manually enable them back and then rename their entity IDs to get my automations to work again.

That did the trick. Thanks!

@redelschaap
Copy link
Contributor

@Koenkk I get that this change was made. However, this does not only break integrations because of the changed name, it also breaks integrations beause the values of the two entities are way too different. In my case, I have to change more than 20 integrations because of the different values. This means that I also have to discover the correct threshold values again.

I have some Aqara GZCGQ11LM sensors. The previous illuminance values were much more detailed than the lux values. For some reason, it also seemed to be more reliable/consistent. See the screenshot below.

Could we at least get the raw value back, perhaps as a new entity? Or is there any way to do this ourselves?

image

@Koenkk
Copy link
Owner Author

Koenkk commented Jan 5, 2025

Let's discuss the illuminance_raw in Koenkk/zigbee2mqtt#25379 (please upvote the OP)

@m0wlheld
Copy link
Contributor

m0wlheld commented Jan 6, 2025

@Koenkk I get that this change was made. However, this does not only break integrations because of the changed name, it also breaks integrations beause the values of the two entities are way too different. In my case, I have to change more than 20 integrations because of the different values. This means that I also have to discover the correct threshold values again.

I have some Aqara GZCGQ11LM sensors. The previous illuminance values were much more detailed than the lux values. For some reason, it also seemed to be more reliable/consistent. See the screenshot below.

Could we at least get the raw value back, perhaps as a new entity? Or is there any way to do this ourselves?

image

Why are the values not identical? From a semantics perspective both report illuminance in Lux/lx, but the lower graph shows values about 1/10 of what to expect during a clear winter day in mid-europe?

@redelschaap
Copy link
Contributor

I don't get why the values are the same either. But that's why I want to be able to use the values from the top graph, they are much more realistic (and useful) to me.

@Viald
Copy link

Viald commented Jan 7, 2025

Same issue here. When the day breaks my external sensor goes from 4769 lux to 3011 lux, and from 3011 lux to 1 lux in 10 minutes. Before, each step was around 80 lux.
Before dusk was between 120 lux and 20 lux. And night was around 20 lux and lower. It's now impossible to detect this steps.
I don't really understand what conversion is applied by z2m 2.0

@DirtyHerby
Copy link

Same here!!! My HA instance doesn't see any illuminance_lux or illuminance sensor (that last should appear at least as this merge states). Rolling back...

Try and take a look under the device info, in my case all the lux sensors associated with Aqara devices where disabled.. I had to manually enable them back and then rename their entity IDs to get my automations to work again.

Where do I find this settings to enable it manually? I use Hue motion sensors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants