table

Author: kellerza

src/ha_addon_sunsynk_multi/sensor_callback.py

@@ -7,7 +7,7 @@
 from typing import Self
 
 from sunsynk import RWSensor, Sensor, ValType
-from sunsynk.utils import pretty_table
+from sunsynk.utils import pretty_table, table_data
 
 from .a_inverter import AInverter
 from .a_sensor import ASensor, SensorOption
@@ -56,8 +56,11 @@ def build_schedules(self, idx: int) -> Self:
 
     def print_schedule(self, title: str, idx: int, sch: dict[int, SensorRun]) -> None:
         """Print the sensor schedule."""
-        data = [[e, ", ".join(s.sensor.id for s in r.sensors)] for e, r in sch.items()]
-        tab = pretty_table(["s", "Sensors"], data)
+        data = [
+            {"s": e, "Sensors": ", ".join(s.sensor.id for s in r.sensors)}
+            for e, r in sch.items()
+        ]
+        tab = pretty_table(*table_data(data))
         inv_ref = ">1" if idx > 1 else "1"
         _LOG.debug("%s (inverter %s)\n%s", title, inv_ref, tab.get_string())
 

src/ha_addon_sunsynk_multi/sensor_options.py

@@ -68,9 +68,6 @@ def init_sensors(self) -> None:
         if not DEFS.all:
             import_definitions()
             if OPT.overrides:
-                _LOG.info(
-                    "Applying sensor overrides from configuration: %s", OPT.overrides
-                )
                 DEFS.override(OPT.overrides)
         self.clear()
 

src/ha_addon_sunsynk_multi/timer_schedule.py

@@ -7,7 +7,7 @@
 from sunsynk.helpers import slug
 from sunsynk.rwsensors import RWSensor
 from sunsynk.sensors import EnumSensor
-from sunsynk.utils import pretty_table
+from sunsynk.utils import pretty_table, table_data
 
 _LOG = logging.getLogger(__name__)
 
@@ -121,19 +121,18 @@ def init_schedules(schedules: list[Schedule]) -> None:
             # _LOG.info("Added    %s", sch)
         SCHEDULES[sch.key] = sch
 
-    hdr = ["Key", "src", "Read", "Report", "Change by", "Change %", "Change any"]
     data = [
-        [
-            schn,
-            source.get(schn, ""),
-            sch.read_every if sch.read_every else "once",
-            sch.report_every if sch.report_every else "",
-            sch.change_by if sch.change_by else "",
-            sch.change_percent if sch.change_percent else "",
-            sch.change_any if sch.change_any else "",
-        ]
+        {
+            "Key": schn,
+            "src": source.get(schn, ""),
+            "Read": sch.read_every if sch.read_every else "once",
+            "Report": sch.report_every if sch.report_every else "",
+            "Change by": sch.change_by if sch.change_by else "",
+            "Change %": sch.change_percent if sch.change_percent else "",
+            "Change any": sch.change_any if sch.change_any else "",
+        }
         for schn, sch in SCHEDULES.items()
     ]
-    tab = pretty_table(hdr, data)
+    tab = pretty_table(*table_data(data))
 
     _LOG.info("Schedules:\n%s", tab.get_string())

src/sunsynk/sensors.py

@@ -14,6 +14,7 @@
     slug,
     unpack_value,
 )
+from sunsynk.utils import pretty_table, table_data
 
 _LOG = logging.getLogger(__name__)
 LOG_TRACE = 15  # logging.addLevelName(LOG_TRACE, "TRACE")
@@ -222,30 +223,39 @@ def _copy(old: Sensor) -> Sensor:
                         setattr(_copy(sen), attrn, news)
             return news
 
+        info: list[dict[str, str]] = []
+
         for key, val in values.items():
             sen_name, _, sen_attr = key.partition(".")
             sen = self.all.get(slug(sen_name))
+            row = {
+                "Sensor": sen.name if sen else sen_name,
+                "Attribute": sen_attr,
+                "Value": str(val),
+                "Message": "?",
+            }
+            info.append(row)
             if sen is None:
-                _LOG.error("Override: Sensor %s not found, skipping", sen_name)
+                row["Message"] = "✕ Sensor not found"
                 continue
+
             if sen_attr == "":
                 if not isinstance(sen, Constant):
-                    _LOG.warning(
-                        "Override for %s is not a Constant sensor, skipping", sen_name
+                    row["Message"] = (
+                        "✕ Empty attributes can only be used with Constants"
                     )
                     continue
                 sen_attr = "value"
 
             if not hasattr(sen, sen_attr):
-                _LOG.error(
-                    "Override: Sensor %s has no attribute %s, skipping",
-                    sen_name,
-                    sen_attr,
-                )
+                row["Message"] = "✕ Attribute not found on target sensor"
                 continue
 
             setattr(_copy(sen), sen_attr, val)
-            _LOG.info("Override: Sensor %s.%s set to %s", sen_name, sen_attr, val)
+            row["Message"] = f"✓ {getattr(sen, sen_attr, '?')} -> {val}"
+
+        tab = pretty_table(*table_data(info))
+        _LOG.info("Applying sensor overrides from configuration\n%s", tab.get_string())
 
 
 @dataclass(slots=True, eq=False)

src/sunsynk/utils.py

@@ -73,7 +73,10 @@ def table_data[T](
 ) -> tuple[list[str], list[list[T | None]]]:
     """Convert a list of dictionaries to a table data format."""
     if headers is None:
-        headers = list({k for v in data for k in v.keys()})
+        hdrs = {}
+        for row in data:
+            hdrs.update(row)
+        headers = list(hdrs)
     return headers, [[v.get(k) for k in headers] for v in data]
 
 

uv.lock

@@ -142,8 +142,14 @@ For information about available sensors, refer to the sensor [definitions](./def
 
   ```log
   [08:19:42] INFO    Importing sensor definitions single-phase (view the source online: https://github.com/kellerza/sunsynk/tree/main/src/sunsynk/definitions/single_phase.py )
-  [08:19:42] INFO    Applying sensor overrides from configuration: {'prog4_power.max': 4990}
-  [08:19:42] INFO    Override: Sensor prog4_power.max set to 4990
+  [08:19:42] INFO    Applying sensor overrides from configuration
+  +-------------+-----------+-------+----------+
+  |    Sensor   | Attribute | Value | Message  |
+  +-------------+-----------+-------+----------+
+  |    Serial   |   trace   |   1   | ✓ 0 -> 1 |
+  | Battery SOC |   trace   |   1   | ✓ 0 -> 1 |
+  | Prog5 power |   trace   |   1   | ✓ 0 -> 1 |
+  +-------------+-----------+-------+----------+
   ```
 
   :::