search

dlarrick / hass-kumo

Home Assistant module interfacing with Mitsubishi mini-split units
MIT License
97 stars 21 forks source link

Set mode if needed when setting temp set point #44

Closed tmchow closed 3 years ago

tmchow commented 3 years ago

I believe this is the fix for #43. I set the mode explicitly as part of setting the temp set points.

dlarrick commented 3 years ago

Sorry for the delay. Will look this weekend.

dlarrick commented 3 years ago

I did a survey of the climate.py files that ship with Home Assistant and the majority do not set the mode in response to set_temperature. esphome, fritzbox, intensishome, nuheat, ozw, smartthings, venstar, and zha do so; the other 68 'climate.py' implementations do not. In my own automation I always call climate.set_hvac_mode prior to calling climate.set_temperature. So I would say the Scheduler component had better call climate.set_hvac_mode if it wants to change the mode.

dlarrick commented 3 years ago

I did this in a different way. Closing.