From 518d4ac4a2d1fa8d93f55f4b59d558e586d31d71 Mon Sep 17 00:00:00 2001 From: Cristian Date: Mon, 5 Aug 2024 12:01:24 -0500 Subject: [PATCH] Revert "ensure strings are not published to the message bus from ecobee driver" This reverts commit 67ea01461ab11b39f65f0bc14d5d41bacb3b5655. --- .../platform_driver/agent.py | 2 +- .../platform_driver/interfaces/ecobee.py | 20 ------------------- 2 files changed, 1 insertion(+), 21 deletions(-) diff --git a/services/core/PlatformDriverAgent/platform_driver/agent.py b/services/core/PlatformDriverAgent/platform_driver/agent.py index f8d98e1c87..91ca805d42 100644 --- a/services/core/PlatformDriverAgent/platform_driver/agent.py +++ b/services/core/PlatformDriverAgent/platform_driver/agent.py @@ -58,7 +58,7 @@ utils.setup_logging() _log = logging.getLogger(__name__) -__version__ = '4.3.5' +__version__ = '4.3.4' PROMETHEUS_METRICS_FILE = "/opt/packages/prometheus_exporter/scrape_files/scrape_metrics.prom" diff --git a/services/core/PlatformDriverAgent/platform_driver/interfaces/ecobee.py b/services/core/PlatformDriverAgent/platform_driver/interfaces/ecobee.py index da9783770b..6b2ceb6901 100644 --- a/services/core/PlatformDriverAgent/platform_driver/interfaces/ecobee.py +++ b/services/core/PlatformDriverAgent/platform_driver/interfaces/ecobee.py @@ -440,25 +440,6 @@ def _set_point(self, point_name, value, **kwargs): if register.readable: return register.get_state(self.thermostat_data) - def _ensure_no_string(self, results): - """ - Ensure that all values in the results dictionary are not strings - :param results: dictionary of results to check - :return: dictionary with all string values converted to floats - """ - new_results = {} - for key, value_list in results.items(): - if not isinstance(value_list, list): - value_list = [value_list] - for value in value_list: - if not isinstance(value, float): - try: - new_results[key] = float(value) - except ValueError: - _log.warning(f"Could not convert value {value} to float for point {key}") - pass - return new_results - def _scrape_all(self): """ Fetch point data for all configured points @@ -486,7 +467,6 @@ def _scrape_all(self): result.update(register_data) else: result[register.point_name] = register_data - result = self._ensure_no_string(result) return result