DLCPosVideo: Use provided epoch, multithread matplotlib (#1168)

* Use provided epoch

* Save video to temp dir

* Remove open-cv support

* WIP: Multithread, RAM hungry

* Limit number of workers

* Save file images in batches

* Reduce RAM cost, remove cv2 dep

* Update changelog

* Get debug arg from params

* Revert merge error #870, #975

* Adjust for final frame. Resume from existing

* Resume from fail

* except IndexError for final frame

* Delay delete temp files until complete

* Explicit error messages

* Return video object for debugging

.gitignore

@@ -3,6 +3,7 @@ spyglass.code-workspace
 mountainsort4_output/
 .idea/
 mysql_config
+memray*
 
 # Notebooks
 *.ipynb

CHANGELOG.md

@@ -59,7 +59,11 @@ dj.FreeTable(dj.conn(), "common_session.session_group").drop()
     - Fix video directory bug in `DLCPoseEstimationSelection` #1103
     - Restore #973, allow DLC without position tracking #1100
     - Minor fix to `DLCCentroid` make function order #1112, #1148
-    - Pass output path as string to `cv2.VideoWriter` #1150
+    - Video creator tools:
+        - Pass output path as string to `cv2.VideoWriter` #1150
+        - Set `DLCPosVideo` default processor to `matplotlib`, remove support
+            for `open-cv` #1168
+        - `VideoMaker` class to process frames in multithreaded batches #1168
 
 - Spike Sorting
 

src/spyglass/common/common_behav.py

@@ -541,7 +541,7 @@ def _no_transaction_make(self, key):
 
         # Skip populating if no pos interval list names
         if len(pos_intervals) == 0:
-            logger.error(f"NO POS INTERVALS FOR {key}; {no_pop_msg}")
+            logger.error(f"NO POS INTERVALS FOR {key};\n{no_pop_msg}")
             self.insert1(null_key, **insert_opts)
             return