Fix uninitialized target temperature

dmitry-cherkas / esphome-danfoss-eco

Extend Danfoss Eco eTRV BLE range over WiFi with ESPHome.

MIT License

21 stars 11 forks source link

Fix uninitialized target temperature #20

Closed ckoca closed 4 months ago

ckoca commented 5 months ago

Fixes an uninitialized variable bug that is sometimes causing the target temperature to temporarily have absurd values upon restarting the ESP32 processor. For example:

I believe that the bug is actually in ESPHome implementation and I am also going to provide a fix in https://github.com/esphome/esphome repository. Once that fix lands I will remove this temporary fix here.

ckoca commented 5 months ago

Opened a PR in esphome repository to fix this: esphome/esphome#5795

Wombosvideo commented 5 months ago

Unrelated: How did you get hold of xxtea-iot-crypt?

ckoca commented 5 months ago

@Wombosvideo See #9.

ckoca commented 4 months ago

esphome/esphome#5795 is merged and the fix is released with esphome v2023.12.0. This fix is no longer required here.