Remove separate multi-cam display signal

Simplify multi-camera frame delivery by removing the throttled `display_ready` path and relying on `frame_ready` for both processing and UI updates. The main window now marks display state dirty when `frame_ready` arrives, and the controller no longer maintains GUI FPS-based display emission logic.

Author: C-Achard

dlclivegui/gui/main_window.py

@@ -855,7 +855,6 @@ def _connect_signals(self) -> None:
 
         # Multi-camera controller signals (used for both single and multi-camera modes)
         self.multi_camera_controller.frame_ready.connect(self._on_multi_frame_processing_ready)
-        self.multi_camera_controller.display_ready.connect(self._on_multi_frame_display_ready)
         self.multi_camera_controller.all_started.connect(self._on_multi_camera_started)
         self.multi_camera_controller.all_stopped.connect(self._on_multi_camera_stopped)
         self.multi_camera_controller.camera_error.connect(self._on_multi_camera_error)
@@ -1702,6 +1701,7 @@ def _on_multi_frame_processing_ready(self, frame_data: MultiFrameData) -> None:
         """
         self._multi_camera_frames = frame_data.frames
         self._multi_camera_display_ids = frame_data.display_ids or {}
+        self._display_dirty = True
         self._try_start_pending_recording()
         src_id = frame_data.source_camera_id
         if src_id:
@@ -1755,15 +1755,6 @@ def _on_multi_frame_processing_ready(self, frame_data: MultiFrameData) -> None:
             self._dlc_timing.note_frame()
             self._dlc_timing.maybe_log()
 
-    def _on_multi_frame_display_ready(self, frame_data: MultiFrameData) -> None:
-        """Throttled UI/display path.
-
-        Called at GUI_MAX_DISPLAY_FPS, not at camera capture FPS for performance reasons.
-        """
-        self._multi_camera_frames = frame_data.frames
-        self._multi_camera_display_ids = frame_data.display_ids or {}
-        self._display_dirty = True
-
     def _on_multi_camera_started(self) -> None:
         """Handle all cameras started event."""
         self.preview_button.setEnabled(False)

dlclivegui/services/multi_camera_controller.py

@@ -4,7 +4,6 @@
 
 import copy
 import logging
-import time
 from dataclasses import dataclass
 from functools import partial
 from threading import Lock
@@ -183,22 +182,6 @@ def set_recording_frame_is_enabled(self, enabled: bool) -> None:
         for worker in list(self._workers.values()):
             worker.set_recording_enabled(enabled)
 
-    def _should_emit_display_ready(self) -> bool:
-        """
-        Return True if enough time has passed since the last display_ready emission, based on GUI_MAX_DISPLAY_FPS.
-        """
-        if self._gui_display_max_fps <= 0:
-            return True
-
-        now = time.perf_counter()
-        min_interval = 1.0 / max(self._gui_display_max_fps, 1e-9)
-
-        if now - self._gui_display_last_emit < min_interval:
-            return False
-
-        self._gui_display_last_emit = now
-        return True
-
     def start(self, camera_settings: list[CameraSettings]) -> None:
         """Start multiple cameras."""
         if self._running or self._stopping:
@@ -496,11 +479,6 @@ def _on_frame_captured(
                 with timing.measure("Multi.emit.frame_ready"):
                     self.frame_ready.emit(frame_data)
 
-                # GUI-only path: throttled display updates
-                if self._should_emit_display_ready():
-                    with timing.measure("Multi.emit.display_ready"):
-                        self.display_ready.emit(frame_data)
-
         timing.note_frame()
         timing.maybe_log()