Allow control cool modes from HA #11
Conversation
Added doc enable cool modes
There was a problem hiding this comment.
If config.CoolEnabled is set, do you want to add it to the list of supported modes in publish_ha_climate_auto_discover() as well?
If so, you'll probably want to tweak ha_mode_as_string() in ehal_hp.h as well, so that it can report back whether it's trying to heat or cool (as that will be used to indicate whether the heating is on/off or heating in publish_climate_status().
There was a problem hiding this comment.
Ah sorry, correction here, the operation mode (from ha_action_as_string()) is probably more useful, and that only currently supports indicating either "heating" or "idle" for the climate entity.
There was a problem hiding this comment.
Hi. @rbroker. Thanks for your comments. I´ve reviewed and changed what you mentioned. Also, while checking the BartGijsbers repo, I saw that there is a OperationMode status (2) that indicates that it is cooling. I've been testing it this morning and in principle it works well.
|
Thanks again! The updated changes look good to me so I've merged it, hopefully it all looks ok with the climate control in homeassistant! |
This PR allow configure cool modes from HA.
Also, it has been added a new configuration zone to set heat pump parameters. The idea is that through parameters, enable or not support for functionalities that may not have all the most ecodan HVAC like cool mode, dual zones, DHW, ...