ensure strings are not published to the message bus from ecobee driver

services/core/PlatformDriverAgent/platform_driver/agent.py

@@ -58,7 +58,7 @@
 
 utils.setup_logging()
 _log = logging.getLogger(__name__)
-__version__ = '4.3.4'
+__version__ = '4.3.5'
 
 
 PROMETHEUS_METRICS_FILE = "/opt/packages/prometheus_exporter/scrape_files/scrape_metrics.prom"

services/core/PlatformDriverAgent/platform_driver/interfaces/ecobee.py

@@ -440,6 +440,25 @@ 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
@@ -467,6 +486,7 @@ def _scrape_all(self):
                         result.update(register_data)
                     else:
                         result[register.point_name] = register_data
+        result = self._ensure_no_string(result)
         return result