Restart active previews; ignore bad trigger

Allow pending preview restarts to proceed when the preview is ACTIVE (previously only IDLE) and return early to avoid double UI sync in camera_config_dialog.py. Also wrap CameraTriggerSettings loading in a try/except and fall back to a default instance when parsing fails, making trigger_config_dialog.py tolerant of malformed or missing trigger data.

Author: C-Achard

dlclivegui/gui/camera_config/camera_config_dialog.py

@@ -1832,9 +1832,10 @@ def _on_loader_finished(self, e: int) -> None:
         self._preview.restart_scheduled = False
         self._preview.loader = None
 
-        if pending and self._preview.state == PreviewState.IDLE:
+        if pending and self._preview.state in (PreviewState.IDLE, PreviewState.ACTIVE):
             LOGGER.debug("[Loader] finished with pending restart for backend=%s idx=%s", pending.backend, pending.index)
             self._begin_preview_load(pending, reason="pending-restart-after-finish")
+            return  # UI sync is already handled in _begin_preview_load
 
         self._sync_preview_ui()
 

dlclivegui/gui/camera_config/trigger_config_dialog.py

@@ -40,7 +40,10 @@ def __init__(self, cam: CameraSettings, parent: QWidget | None = None):
         self._cam = cam.model_copy(deep=True)
 
         ns = _backend_namespace(self._cam)
-        self._trigger = CameraTriggerSettings.from_any(ns.get("trigger"))
+        try:
+            self._trigger = CameraTriggerSettings.from_any(ns.get("trigger"))
+        except Exception:
+            self._trigger = CameraTriggerSettings()
 
         self._setup_ui()
         self._load_from_trigger(self._trigger)