Addressed PR comments

Author: wk9874

simvue/api/objects/grids.py

@@ -79,11 +79,11 @@ def on_reconnect(self, id_mapping: dict[str, str]) -> None:
         id_mapping : dict[str, str]
             mapping from offline identifier to new online identifier.
         """
-        _online_runs = [
+        _online_runs = (
             (id_mapping[run_id], metric_name)
             for run_id, metric_name in self._staging.pop("runs", [])
-        ]
-        super().commit()  # TODO this means the sender iwll commit twice - that ok?
+        )
+        super().commit()
         for run_id, metric_name in _online_runs:
             try:
                 self.attach_metric_for_run(run_id=run_id, metric_name=metric_name)