Initialize video attrs and switch to logging

In deeplabcut/utils/video_processor.py initialize commonly used instance attributes (FPS, vid, svid, sh, sw) in the constructor to avoid missing-attribute errors. Replace a stray print() in the exception handler with a module logger and add logging import/logger setup. Also add a brief note about FPS handling to clarify behavior when overriding video FPS.
MMathisLab · Jun 25, 2026 · Jun 5, 2026 · Jun 5, 2026 · Jun 9, 2026 · Jun 9, 2026
commit 31f89858c60e2964dd613045ba8d87ea157ed8d4
diff --git a/deeplabcut/utils/video_processor.py b/deeplabcut/utils/video_processor.py
@@ -21,11 +21,14 @@
 i.e. 'XVID'
 """
 
+import logging
 from abc import ABC, abstractmethod
 
 import cv2
 import numpy as np
 
+logger = logging.getLogger(__name__)
+
 
 class VideoProcessor(ABC):
     """Base class for a video processing unit, implementation is required for video
@@ -37,6 +40,11 @@ class VideoProcessor(ABC):
     def __init__(self, fname="", sname="", nframes=-1, fps=None, codec="X264", sh="", sw=""):
         self._fname = None
         self._sname = None
+        self.FPS = None
+        self.vid = None
+        self.svid = None
+        self.sh = 0
+        self.sw = 0
         ###
         self.fname = fname
         self.sname = sname
@@ -63,9 +71,11 @@ def __init__(self, fname="", sname="", nframes=-1, fps=None, codec="X264", sh=""
                 self.svid = self.create_video()
 
         except Exception as ex:
-            print("Error: %s", ex)
+            logger.error("VideoProcessor initialization failed: %s", ex)
 
         if fps is not None:  # Overwrite the video's FPS
+            # NOTE @C-Achard 2026-06-09 improving checks here might break old API
+            # same for raising on missing FPS
             self.FPS = fps
 
     def load_frame(self):