From 9c77f5f5a973a71572da7b55a84a4792d266e174 Mon Sep 17 00:00:00 2001 From: Bryan York Date: Tue, 21 Nov 2017 10:48:36 -0800 Subject: [PATCH] Fix unit conversion for Sensibo A/C units (#10692) * Fix unit conversion for Sensibo A/C units When the Sensibo component was released, there was a provision to not convert the temperature units unless "nativeTemperatureUnit" was returned with the API. I'm not sure if the API changed on Sensibo's side, but I do not get this key passed back with API requests. This causes my current temperature to be returned in CELSIUS instead of FAHRENHEIT. Removing this fixes it, and I can confirm the units are shown properly now. * Update adding comment showing temperature is always in C --- homeassistant/components/climate/sensibo.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/homeassistant/components/climate/sensibo.py b/homeassistant/components/climate/sensibo.py index c55b4c9ce0d..9111e7821a6 100644 --- a/homeassistant/components/climate/sensibo.py +++ b/homeassistant/components/climate/sensibo.py @@ -133,10 +133,8 @@ class SensiboClimate(ClimateDevice): @property def current_temperature(self): """Return the current temperature.""" - # This field is not affected by temperature_unit. - # It is always in C / nativeTemperatureUnit - if 'nativeTemperatureUnit' not in self._ac_states: - return self._measurements['temperature'] + # This field is not affected by temperatureUnit. + # It is always in C return convert_temperature( self._measurements['temperature'], TEMP_CELSIUS,