Skip to content

Commit d3dc4ce

Browse files
committed
Clarify timeout comment and camera log message
Tweak wording for clarity in two places: update STOP_WORKER_TIMEOUT comment to better describe waiting in the STOPPING state before scheduling background reaping, and reflow the MultiCameraController critical log message into a single, spaced string to avoid awkward concatenation. These are documentation/logging refinements only and do not change runtime behavior. Files modified: - dlclivegui/services/dlc_processor.py: adjust timeout comment - dlclivegui/services/multi_camera_controller.py: reformat and clarify critical log message
1 parent 9c0443d commit d3dc4ce

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

dlclivegui/services/dlc_processor.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
from dlclivegui.temp import Engine # type: ignore # TODO use main package enum when released
2222

2323
logger = logging.getLogger(__name__)
24-
STOP_WORKER_TIMEOUT = 10.0 # seconds to wait for worker thread to stop before STOPPING state and reaping
24+
STOP_WORKER_TIMEOUT = 10.0 # # seconds to wait in STOPPING state before scheduling background reaping
2525

2626
try: # pragma: no cover - optional dependency
2727
from dlclive import (

dlclivegui/services/multi_camera_controller.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -271,9 +271,9 @@ def stop(self, wait: bool = True) -> None:
271271
self._expected_cameras = 0
272272

273273
LOGGER.critical(
274-
"Force stop failed for camera threads: %s."
275-
"Sending all_stopped to leave GUI in running state,"
276-
"but background camera threads may still be active and interfere with future camera starts."
274+
"Force stop failed for camera threads: %s. "
275+
"Sending all_stopped to leave GUI in running state, "
276+
"but background camera threads may still be active and interfere with future camera starts. "
277277
"Application restart may be required.",
278278
still_running,
279279
)

0 commit comments

Comments
 (0)