Mariano-Github / Edge-Drivers-Beta

Edge Driver Beta Modified for Created
Apache License 2.0
116 stars 118 forks source link

Aqara Temperature Sensor Reads 0 Then Immediately Updates Again to Correct Temp #111

Closed StarScream159 closed 8 months ago

StarScream159 commented 8 months ago

Hi Mariano,

I came across your post here about how you were going to implement a possible fix like Veonua did.

However veonua's driver was showing the problem so I decided to install and try yours to see if you had it fixed.

I just had it flash to zero for an update and then immediately update again to the correct temp..

The problem is this is triggering some automations.

Do you know why it is showing zero and then updating correctly? Did your fix of ignoring zero values if the previous value wasn't around zero not work?

Thanks!

Mariano-Github commented 8 months ago

Hi,

The github code was not updated.

Nobody who asked me told me that it didn't work well, but when I reviewed the code I saw an error and I corrected it. Thanks

Only works for manufacturer== "LUMI" and model== "lumi.weather"

This driver version fix it

─────────────────────────────────────────────────────────
 Name         Zigbee Temp Sensor and Child Thermostat Mc 
 Version      2024-01-04T10:04:05.829046739
─────────────────────────────────────────────────────────
StarScream159 commented 8 months ago

I assume because manufacturer== "LUMI" this will not work for the Aqara/Xiaomi devices? Could you add in support for them too?

Mariano-Github commented 8 months ago

Aqara, xioami are trademarks. What the driver recognizes is the manufacturer and model recorded in its firmware, in this case LUMI

You can view the manufacturer and model code of your device at my.smartthings.com, advanced users.

As far as I know, the problem only affects the Aqara model, which I wrote to you

StarScream159 commented 8 months ago

Oh, I understand about trademarks vs. manufacturer. Thanks for pointing that out. I checked under advanced users and I can confirm it shows as "lumi.weather" for model and manufacture code is "LUMI".

With that it seems like your manufacturer== "LUMI" will work. I see your commit/change and will test it and update this ticket.

Thanks.

StarScream159 commented 8 months ago

I haven't had anymore false 0's since this change. I believe this has been fixed now. Thank you. This can be closed.