Bug fixes VideoWriter: video names with spaces (#2388)

n-poulsen · web-flow · commit f64ec9a3141a · 2023-09-18T18:37:47.000+02:00
* bugfix: shorten video when name contains space

* fixed rescale and check_integrity
diff --git a/deeplabcut/utils/auxfun_videos.py b/deeplabcut/utils/auxfun_videos.py
@@ -55,7 +55,7 @@ def __len__(self):
 
     def check_integrity(self):
         dest = os.path.join(self.directory, f"{self.name}.log")
-        command = f"ffmpeg -v error -i {self.video_path} -f null - 2>{dest}"
+        command = f'ffmpeg -v error -i "{self.video_path}" -f null - 2>"{dest}"'
         subprocess.call(command, shell=True)
         if os.path.getsize(dest) != 0:
             warnings.warn(f'Video contains errors. See "{dest}" for a detailed report.')
@@ -267,8 +267,8 @@ def validate_timestamp(stamp):
 
         output_path = self.make_output_path(suffix, dest_folder)
         command = (
-            f"ffmpeg -n -i {self.video_path} -ss {start} -to {end} "
-            f"-c:a copy {output_path}"
+            f'ffmpeg -n -i "{self.video_path}" -ss {start} -to {end} '
+            f'-c:a copy "{output_path}"'
         )
         subprocess.call(command, shell=True)
         return output_path
@@ -333,8 +333,8 @@ def rescale(
     ):
         output_path = self.make_output_path(suffix, dest_folder)
         command = (
-            f"ffmpeg -n -i {self.video_path} -filter:v "
-            f'"scale={width}:{height}{{}}" -c:a copy {output_path}'
+            f'ffmpeg -n -i "{self.video_path}" -filter:v '
+            f'"scale={width}:{height}{{}}" -c:a copy "{output_path}"'
         )
         # Rotate, see: https://stackoverflow.com/questions/3937387/rotating-videos-with-ffmpeg
         # interesting option to just update metadata.
