BenPru / luxtronik

Luxtronik integration for Home Assistant
MIT License
67 stars 23 forks source link

Cooling thermostat needs different HVAC_MODE_MAPPING #147

Closed rhammen closed 8 months ago

rhammen commented 9 months ago

Cooling thermostat needs something like this: HVAC_COOL_MODE_MAPPING: dict[str, str] = { LuxMode.off.value: HVACMode.OFF.value, LuxMode.automatic.value: HVACMode.COOL.value, }

Main issue is that for the Cooling Thermostat, the LuxMode.automatic.value needs to be mapped to HVACMode.COOL.value instead of HVACMode.HEAT.value. Currently, when you set Mod e=Cool, you get an error because HVACMode.COOL.value is not defined as a value in HVAC_MODE_MAPPING.

BenPru commented 9 months ago

Thanks for your input. Should be fixed with 2023.10.11-Beta.

rhammen commented 8 months ago

Thanks, confirmed it works OK with 2023.10.12 Beta