fix: handle the ffprobe stdout/stderr as unicode

Author: ptpt

mapillary_tools/ffprobe.py

@@ -49,15 +49,17 @@ def __init__(self, video_file):
             self.audio = []
             datalines = []
 
-            for a in iter(p.stdout.readline, b""):
+            for b in iter(p.stdout.readline, b""):
+                a = b.decode("utf-8")
                 if re.match("\[STREAM\]", a):
                     datalines = []
                 elif re.match("\[\/STREAM\]", a):
                     self.streams.append(FFStream(datalines))
                     datalines = []
                 else:
                     datalines.append(a)
-            for a in iter(p.stderr.readline, b""):
+            for b in iter(p.stderr.readline, b""):
+                a = b.decode("utf-8")
                 if re.match("\[STREAM\]", a):
                     datalines = []
                 elif re.match("\[\/STREAM\]", a):

mapillary_tools/process_video.py

@@ -170,13 +170,15 @@ def get_video_duration(video_file):
     """Get video duration in seconds"""
     probe = FFProbe(video_file)
     if not probe.video:
-        print(f'No video found in {video_file}')
+        print(f"No video found in {video_file}")
         return None
     duration = probe.video[0].duration
     try:
         return float(duration)
     except (TypeError, ValueError) as e:
-        print(f"could not parse {duration} as duration from video {video_file} due to {e}")
+        print(
+            f"could not parse {duration} as duration from video {video_file} due to {e}"
+        )
         return None