From e21a08da44ed58c7d2722e4c871fb2a72664319d Mon Sep 17 00:00:00 2001 From: Aufano Date: Thu, 6 Apr 2017 19:06:04 +0200 Subject: [PATCH 1/2] Fix current_temperature is rounded --- homeassistant/components/climate/__init__.py | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/homeassistant/components/climate/__init__.py b/homeassistant/components/climate/__init__.py index bd2e38433d6ba..1cd2e7fb9642c 100644 --- a/homeassistant/components/climate/__init__.py +++ b/homeassistant/components/climate/__init__.py @@ -692,18 +692,16 @@ def max_humidity(self): def _convert_for_display(self, temp): """Convert temperature into preferred units for display purposes.""" - if (temp is None or not isinstance(temp, Number) or - self.temperature_unit == self.unit_of_measurement): + if (temp is None or not isinstance(temp, Number)): return temp - - value = convert_temperature(temp, self.temperature_unit, - self.unit_of_measurement) - + if(self.temperature_unit != self.unit_of_measurement): + temp = convert_temperature(temp, self.temperature_unit, + self.unit_of_measurement) # Round in the units appropriate if self.precision == PRECISION_HALVES: - return round(value * 2) / 2.0 + return round(temp * 2) / 2.0 elif self.precision == PRECISION_TENTHS: - return round(value, 1) + return round(temp, 1) else: # PRECISION_WHOLE as a fall back - return round(value) + return round(temp) From 986c0ef84eeaaa7026400be12876167619c2c622 Mon Sep 17 00:00:00 2001 From: Alberto Ufano Date: Thu, 6 Apr 2017 21:50:15 +0200 Subject: [PATCH 2/2] fix Unnecessary parens after 'if' --- homeassistant/components/climate/__init__.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/homeassistant/components/climate/__init__.py b/homeassistant/components/climate/__init__.py index 1cd2e7fb9642c..2a0bd2eb5c19f 100644 --- a/homeassistant/components/climate/__init__.py +++ b/homeassistant/components/climate/__init__.py @@ -692,9 +692,9 @@ def max_humidity(self): def _convert_for_display(self, temp): """Convert temperature into preferred units for display purposes.""" - if (temp is None or not isinstance(temp, Number)): + if temp is None or not isinstance(temp, Number): return temp - if(self.temperature_unit != self.unit_of_measurement): + if self.temperature_unit != self.unit_of_measurement: temp = convert_temperature(temp, self.temperature_unit, self.unit_of_measurement) # Round in the units appropriate