bugfix revert auto mode -> the mode is hardcoded for the thermostat_ha_cmd #183

[kpschaper]

@proddy

Not sure if this is the best solution, always pass AUTO mode and determine inside the function if the mode is actual set to AUTO?

EMS-ESP32/src/devices/thermostat.cpp

Line 483 in 5893487

set_temperature(f, HeatingCircuit::Mode::AUTO, hc_num);

[proddy]

that is what it does now. The thermostat component in HA is very simple and stupid. It doesn't handle all the modes you can have in thermostats (like temp, summer, comfort, eco, holiday, design etc etc) so I just default it to AUTO when a temperature is changed in HA

[kpschaper]

Ah I get it now, it's just the most convenient way to map from HA modes to thermostat specific modes

[proddy]

exactly. Probably the best method is to use whatever the current Thermostat mode is and adjust that setpoint temperature. There's probably a good reason why we don't do it that way. I can't remember. No one has complained so I guess it just works !

What do you want to do with this PR - merge it?

[kpschaper]

I think it has to do with the fact that some thermostats have a mode and mode_type.

Yes, please merge this PR and leave it like this!