From 1ca99c4fa4b2fcb3d1580fe36931a83d82518f20 Mon Sep 17 00:00:00 2001
From: treylok <verylogical@gmail.com>
Date: Mon, 21 Dec 2020 13:03:26 -0600
Subject: [PATCH] Add ecobee humidity attributes (#44366)
* Update humidity attributes.
* Update climate.py
* Raise ValueError
* Update conditions for humidity controls to show
Humidity controls only show when the humidifier mode is in "manual" mode.
---
homeassistant/components/ecobee/climate.py | 39 +++++++++++++++++++++-
1 file changed, 38 insertions(+), 1 deletion(-)
diff --git a/homeassistant/components/ecobee/climate.py b/homeassistant/components/ecobee/climate.py
index 94396bbf883..6bb7dc1a870 100644
--- a/homeassistant/components/ecobee/climate.py
+++ b/homeassistant/components/ecobee/climate.py
@@ -65,6 +65,11 @@ AWAY_MODE = "awayMode"
PRESET_HOME = "home"
PRESET_SLEEP = "sleep"
+DEFAULT_MIN_HUMIDITY = 15
+DEFAULT_MAX_HUMIDITY = 50
+HUMIDIFIER_MANUAL_MODE = "manual"
+
+
# Order matters, because for reverse mapping we don't want to map HEAT to AUX
ECOBEE_HVAC_TO_HASS = collections.OrderedDict(
[
@@ -162,7 +167,6 @@ SUPPORT_FLAGS = (
| SUPPORT_AUX_HEAT
| SUPPORT_TARGET_TEMPERATURE_RANGE
| SUPPORT_FAN_MODE
- | SUPPORT_TARGET_HUMIDITY
)
@@ -332,6 +336,8 @@ class Thermostat(ClimateEntity):
@property
def supported_features(self):
"""Return the list of supported features."""
+ if self.has_humidifier_control:
+ return SUPPORT_FLAGS | SUPPORT_TARGET_HUMIDITY
return SUPPORT_FLAGS
@property
@@ -391,6 +397,31 @@ class Thermostat(ClimateEntity):
return self.thermostat["runtime"]["desiredCool"] / 10.0
return None
+ @property
+ def has_humidifier_control(self):
+ """Return true if humidifier connected to thermostat and set to manual/on mode."""
+ return (
+ self.thermostat["settings"]["hasHumidifier"]
+ and self.thermostat["settings"]["humidifierMode"] == HUMIDIFIER_MANUAL_MODE
+ )
+
+ @property
+ def target_humidity(self) -> Optional[int]:
+ """Return the desired humidity set point."""
+ if self.has_humidifier_control:
+ return self.thermostat["runtime"]["desiredHumidity"]
+ return None
+
+ @property
+ def min_humidity(self) -> int:
+ """Return the minimum humidity."""
+ return DEFAULT_MIN_HUMIDITY
+
+ @property
+ def max_humidity(self) -> int:
+ """Return the maximum humidity."""
+ return DEFAULT_MAX_HUMIDITY
+
@property
def target_temperature(self):
"""Return the temperature we try to reach."""
@@ -653,7 +684,13 @@ class Thermostat(ClimateEntity):
def set_humidity(self, humidity):
"""Set the humidity level."""
+ if humidity not in range(0, 101):
+ raise ValueError(
+ f"Invalid set_humidity value (must be in range 0-100): {humidity}"
+ )
+
self.data.ecobee.set_humidity(self.thermostat_index, int(humidity))
+ self.update_without_throttle = True
def set_hvac_mode(self, hvac_mode):
"""Set HVAC mode (auto, auxHeatOnly, cool, heat, off)."""