jmcollin78 / versatile_thermostat

A full featured Thermostat for Home Assistant: presets, window, motion, presence and overpowering management
MIT License
328 stars 34 forks source link

climate with window state configured with sensor is *also* using the centrally configured slope detection, which it shouldn't #327

Closed maia closed 10 months ago

maia commented 10 months ago

One of my rooms does not use the temperature slope to detect open windows, as this room has a door sensor. I've configured this climate entity to use the door sensor:

Bildschirmfoto 2024-01-08 um 19 14 36 Bildschirmfoto 2024-01-08 um 19 14 55

Unfortunately the entity seems to also use the temperature slope detection I configured in the central config (which is used by all other rooms), as any time the slope falls drastically, the heating turns off (see upper two history charts in the image below).

So besides the issue that the EMA calculation isn't yet working for me (see graphs below), the bug is that a climate entity configured to use a door sensor should not also use the central config.

Bildschirmfoto 2024-01-08 um 19 19 13
jmcollin78 commented 10 months ago

Idem #324