mmakaay / esphome-xiaomi_bslamp2

ESPHome integration for the Xiaomi Mijia Bedside Lamp v2.
Other
213 stars 51 forks source link

MCU temperature reading support? #112

Closed szymucha94 closed 1 year ago

szymucha94 commented 1 year ago

Is it safe to use this configuration: https://github.com/esphome/esphome/pull/4330 to send MCU temp to HA from these lamps?

mmakaay commented 1 year ago

I gave it a go for you, and it seems to work: image

After flashing, my dev lamp did not correctly reboot and I had to disconnect and reconnect the power. I've never seen that happen and likely it has nothing to do with this new module. But in case the same thing happens with you: the power cycle does help ander after restarting, the temperature was reported. Subsequent flashes didn't show this behaviour, so might have been anything really 😄

szymucha94 commented 1 year ago

Both of my lamps had the same issue with failed reboot. But it happened after keeping original, default configuration and just upgrading to 2023.3.0 Thank you, I will update my configuration then :)

szymucha94 commented 1 year ago

I had troubles building this, but seems like cleaning build files fixed the issue. I get the readings now :) Thanks!

mmakaay commented 1 year ago

Great! And interesting you had the same failed reboot. Maybe something that got introduced with 2023.3.0. I'll experiment a bit with that by doing some downgrade / upgrade cycles. Not important for this issue though. Enjoy the temperature readings!