search

gekkekoe / esphome-ecodan-hp

ESPHome Ecodan heatpump
GNU General Public License v3.0
67 stars 26 forks source link

Incorrect minimum set point for ATW HP #42

Open bwduncan opened 3 months ago

bwduncan commented 3 months ago

Hey!

My ATW ASHP has a minimum set point temp of 20°C (it can't do cooling at all). There doesn't seem to be any way to configure the esphome to use this instead of the default of 5°C. Rather than maintain my own fork, is there a nice way to implement this? I'm happy to make a PR, but it seems kind of silly to just set a yaml option for something the HP already knows (but I guess it's probably not available over the serial link).

Thanks! Bruce

gekkekoe commented 3 months ago

I'm aware. I tried to solve it via https://github.com/gekkekoe/esphome-ecodan-hp/blob/a7ff6e1036787e50f3615563546ebd65640280ce/components/ecodan/climate.cpp#L187

This does work in esphome, but HA does not change the range when you switch. The best way would be to make HA accept the new bounds on mode switch, but I could not find a way to do it. more info: https://community.home-assistant.io/t/generic-thermostat-heating-range-and-cooling-range/703169

for your case we might need a cooling capability config flag, and then disable all the cooling options if not set, though the first option I described is much easier if it works out.