auto-editor 24.24.1__tar.gz → 24.27.1__tar.gz

{auto_editor-24.24.1/auto_editor.egg-info → auto_editor-24.27.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: auto-editor
-Version: 24.24.1
+Version: 24.27.1
 Summary: Auto-Editor: Effort free video editing!
 Author-email: WyattBlue <wyattblue@auto-editor.com>
 License: Unlicense
@@ -12,7 +12,7 @@ Requires-Python: >=3.10
 Description-Content-Type: text/markdown
 License-File: LICENSE
 Requires-Dist: numpy>=1.22.0
-Requires-Dist: pyav==12.1.0
+Requires-Dist: pyav==12.2.*
 Requires-Dist: ae-ffmpeg==1.2.*
 
 <p align="center"><img src="https://auto-editor.com/img/auto-editor-banner.webp" title="Auto-Editor" width="700"></p>
@@ -105,16 +105,14 @@ Auto-Editor can also export to:
 - Individual media clips with `--export clip-sequence`
 
 ### Naming Timelines
-By default, auto-editor will name the timeline to "Auto-Editor Media Group" if the export supports naming.
+Some editors support naming timelines. By default, auto-editor will use the name "Auto-Editor Media Group". For `premiere` `resolve` and `final-cut-pro` export options, you can change the name with the following syntax.
 
 ```
+# for POSIX shells
 auto-editor example.mp4 --export 'premiere:name="Your name here"'
 
-auto-editor example.mp4 --export 'resolve:name="Your name here"'
-
-auto-editor example.mp4 --export 'final-cut-pro:name="Your name here"'
-
-# No other export options support naming
+# for Powershell
+auto-editor example.mp4 --export 'premiere:name=""Your name here""'
 ```
 
 ### Split by Clip

{auto_editor-24.24.1 → auto_editor-24.27.1}/README.md

@@ -88,16 +88,14 @@ Auto-Editor can also export to:
 - Individual media clips with `--export clip-sequence`
 
 ### Naming Timelines
-By default, auto-editor will name the timeline to "Auto-Editor Media Group" if the export supports naming.
+Some editors support naming timelines. By default, auto-editor will use the name "Auto-Editor Media Group". For `premiere` `resolve` and `final-cut-pro` export options, you can change the name with the following syntax.
 
 ```
+# for POSIX shells
 auto-editor example.mp4 --export 'premiere:name="Your name here"'
 
-auto-editor example.mp4 --export 'resolve:name="Your name here"'
-
-auto-editor example.mp4 --export 'final-cut-pro:name="Your name here"'
-
-# No other export options support naming
+# for Powershell
+auto-editor example.mp4 --export 'premiere:name=""Your name here""'
 ```
 
 ### Split by Clip

auto_editor-24.27.1/auto_editor/__init__.py

@@ -0,0 +1,2 @@
+__version__ = "24.27.1"
+version = "24w27a"

{auto_editor-24.24.1 → auto_editor-24.27.1}/auto_editor/__main__.py

@@ -1,10 +1,13 @@
 #!/usr/bin/env python3
 
 import sys
+from os import environ
 
 import auto_editor
+from auto_editor.edit import edit_media
+from auto_editor.ffwrapper import FFmpeg
 from auto_editor.utils.func import setup_tempdir
-from auto_editor.utils.log import Log, Timer
+from auto_editor.utils.log import Log
 from auto_editor.utils.types import (
     Args,
     bitrate,
@@ -277,11 +280,16 @@ def main() -> None:
             f"auto_editor.subcommands.{sys.argv[1]}", fromlist=["subcommands"]
         )
         obj.main(sys.argv[2:])
-        sys.exit()
+        return
+
+    ff_color = "AV_LOG_FORCE_NOCOLOR"
+    no_color = bool(environ.get("NO_COLOR")) or bool(environ.get(ff_color))
+    log = Log(no_color=no_color)
 
     args = main_options(ArgumentParser("Auto-Editor")).parse_args(
         Args,
         sys.argv[1:],
+        log,
         macros=[
             ({"--frame-margin"}, ["--margin"]),
             ({"--export-to-premiere", "-exp"}, ["--export", "premiere"]),
@@ -296,41 +304,36 @@ def main() -> None:
 
     if args.version:
         print(f"{auto_editor.version} ({auto_editor.__version__})")
-        sys.exit()
-
-    from auto_editor.edit import edit_media
-    from auto_editor.ffwrapper import FFmpeg
-
-    log = Log(args.debug, args.quiet)
-    ffmpeg = FFmpeg(
-        args.ffmpeg_location,
-        args.my_ffmpeg,
-        args.show_ffmpeg_commands,
-        args.show_ffmpeg_output,
-    )
+        return
 
-    if args.debug and args.input == []:
+    if args.debug and not args.input:
         import platform as plat
 
-        print(f"Python Version: {plat.python_version()}")
-        print(f"Platform: {plat.system()} {plat.release()} {plat.machine().lower()}")
-        print(f"FFmpeg Version: {ffmpeg.version}\nFFmpeg Path: {ffmpeg.path}")
-        print(f"Auto-Editor Version: {auto_editor.version}")
-        sys.exit()
+        import av
+
+        print(f"OS: {plat.system()} {plat.release()} {plat.machine().lower()}")
+        print(f"Python: {plat.python_version()}")
+        print(f"PyAV: {av.__version__}")
+        print(f"Auto-Editor: {auto_editor.version}")
+        return
 
-    if args.input == []:
+    if not args.input:
         log.error("You need to give auto-editor an input file.")
 
-    temp = setup_tempdir(args.temp_dir, Log())
-    log = Log(args.debug, args.quiet, temp)
-    log.machine = args.progress == "machine"
+    temp = setup_tempdir(args.temp_dir, log)
+    log = Log(args.debug, args.quiet, temp, args.progress == "machine", no_color)
     log.debug(f"Temp Directory: {temp}")
 
+    ffmpeg = FFmpeg(
+        args.ffmpeg_location,
+        args.my_ffmpeg,
+        args.show_ffmpeg_commands,
+        args.show_ffmpeg_output,
+    )
     paths = valid_input(args.input, ffmpeg, args, log)
-    timer = Timer(args.quiet or log.machine)
 
     try:
-        edit_media(paths, ffmpeg, args, temp, timer, log)
+        edit_media(paths, ffmpeg, args, temp, log)
     except KeyboardInterrupt:
         log.error("Keyboard Interrupt")
     log.cleanup()

{auto_editor-24.24.1 → auto_editor-24.27.1}/auto_editor/analyze.py

@@ -3,6 +3,7 @@ from __future__ import annotations
 import os
 import re
 from dataclasses import dataclass
+from fractions import Fraction
 from typing import TYPE_CHECKING
 
 import numpy as np
@@ -19,12 +20,12 @@ from auto_editor.lib.contracts import (
     orc,
 )
 from auto_editor.lib.data_structs import Sym
-from auto_editor.render.subtitle import SubtitleParser
 from auto_editor.utils.cmdkw import (
     Required,
     pAttr,
     pAttrs,
 )
+from auto_editor.utils.subtitle_tools import convert_ass_to_text
 from auto_editor.wavfile import read
 
 if TYPE_CHECKING:
@@ -307,31 +308,65 @@ class Levels:
         except re.error as e:
             self.log.error(e)
 
-        sub_file = self.ensure.subtitle(self.src, stream)
-        parser = SubtitleParser(self.tb)
+        import av
+        from av.subtitles.subtitle import AssSubtitle, TextSubtitle
 
-        with open(sub_file, encoding="utf-8") as file:
-            parser.parse(file.read(), "webvtt")
+        try:
+            container = av.open(self.src.path, "r")
+            subtitle_stream = container.streams.subtitles[stream]
+            assert isinstance(subtitle_stream.time_base, Fraction)
+        except Exception as e:
+            self.log.error(e)
 
-        # stackoverflow.com/questions/9662346/python-code-to-remove-html-tags-from-a-string
-        def cleanhtml(raw_html: str) -> str:
-            cleanr = re.compile("<.*?>")
-            return re.sub(cleanr, "", raw_html)
+        # Get the length of the subtitle stream.
+        sub_length = 0
+        for packet in container.demux(subtitle_stream):
+            if packet.pts is None or packet.duration is None:
+                continue
+            for subset in packet.decode():
+                # See definition of `AVSubtitle`
+                # in: https://ffmpeg.org/doxygen/trunk/avcodec_8h_source.html
+                start = float(packet.pts * subtitle_stream.time_base)
+                dur = float(packet.duration * subtitle_stream.time_base)
 
-        if not parser.contents:
-            self.log.error("subtitle has no valid entries")
+                end = round((start + dur) * self.tb)
+                sub_length = max(sub_length, end)
 
-        result = np.zeros((parser.contents[-1].end), dtype=np.bool_)
+        result = np.zeros((sub_length), dtype=np.bool_)
+        del sub_length
 
         count = 0
-        for content in parser.contents:
-            if max_count is not None and count >= max_count:
+        early_exit = False
+        container.seek(0)
+        for packet in container.demux(subtitle_stream):
+            if packet.pts is None or packet.duration is None:
+                continue
+            if early_exit:
                 break
-
-            line = cleanhtml(content.after.strip())
-            if line and re.search(pattern, line):
-                result[content.start : content.end] = 1
-                count += 1
+            for subset in packet.decode():
+                if max_count is not None and count >= max_count:
+                    early_exit = True
+                    break
+
+                start = float(packet.pts * subtitle_stream.time_base)
+                dur = float(packet.duration * subtitle_stream.time_base)
+
+                san_start = round(start * self.tb)
+                san_end = round((start + dur) * self.tb)
+
+                for sub in subset:
+                    if isinstance(sub, AssSubtitle):
+                        line = convert_ass_to_text(sub.ass.decode(errors="ignore"))
+                    elif isinstance(sub, TextSubtitle):
+                        line = sub.text.decode(errors="ignore")
+                    else:
+                        continue
+
+                    if line and re.search(pattern, line):
+                        result[san_start:san_end] = 1
+                        count += 1
+
+        container.close()
 
         return result
 

{auto_editor-24.24.1 → auto_editor-24.27.1}/auto_editor/edit.py

@@ -15,7 +15,7 @@ from auto_editor.utils.bar import Bar
 from auto_editor.utils.chunks import Chunk, Chunks
 from auto_editor.utils.cmdkw import ParserError, parse_with_palet, pAttr, pAttrs
 from auto_editor.utils.container import Container, container_constructor
-from auto_editor.utils.log import Log, Timer
+from auto_editor.utils.log import Log
 from auto_editor.utils.types import Args
 
 
@@ -150,7 +150,7 @@ def parse_export(export: str, log: Log) -> dict[str, Any]:
 
 
 def edit_media(
-    paths: list[str], ffmpeg: FFmpeg, args: Args, temp: str, timer: Timer, log: Log
+    paths: list[str], ffmpeg: FFmpeg, args: Args, temp: str, log: Log
 ) -> None:
     bar = Bar(args.progress)
     tl = None
@@ -160,7 +160,7 @@ def edit_media(
         if path_ext == ".xml":
             from auto_editor.formats.fcp7 import fcp7_read_xml
 
-            tl = fcp7_read_xml(paths[0], ffmpeg, log)
+            tl = fcp7_read_xml(paths[0], log)
             assert tl.src is not None
             sources: list[FileInfo] = [tl.src]
             src: FileInfo | None = tl.src
@@ -168,7 +168,7 @@ def edit_media(
         elif path_ext == ".mlt":
             from auto_editor.formats.shotcut import shotcut_read_mlt
 
-            tl = shotcut_read_mlt(paths[0], ffmpeg, log)
+            tl = shotcut_read_mlt(paths[0], log)
             assert tl.src is not None
             sources = [tl.src]
             src = tl.src
@@ -190,7 +190,6 @@ def edit_media(
 
     if export["export"] == "timeline":
         log.quiet = True
-        timer.quiet = True
 
     if not args.preview:
         log.conwrite("Starting")
@@ -210,18 +209,9 @@ def edit_media(
     else:
         samplerate = args.sample_rate
 
-    ensure = Ensure(ffmpeg, samplerate, temp, log)
+    ensure = Ensure(ffmpeg, bar, samplerate, temp, log)
 
     if tl is None:
-        # Extract subtitles in their native format.
-        if src is not None and len(src.subtitles) > 0 and not args.sn:
-            cmd = ["-i", f"{src.path}", "-hide_banner"]
-            for s, sub in enumerate(src.subtitles):
-                cmd.extend(["-map", f"0:s:{s}"])
-            for s, sub in enumerate(src.subtitles):
-                cmd.extend([os.path.join(temp, f"{s}s.{sub.ext}")])
-            ffmpeg.run(cmd)
-
         tl = make_timeline(sources, ensure, args, samplerate, bar, temp, log)
 
     if export["export"] == "timeline":
@@ -283,7 +273,7 @@ def edit_media(
         apply_later = False
 
         if ctr.allow_subtitle and not args.sn:
-            sub_output = make_new_subtitles(tl, ffmpeg, temp, log)
+            sub_output = make_new_subtitles(tl, ensure, temp)
 
         if ctr.allow_audio:
             audio_output = make_new_audio(tl, ensure, args, ffmpeg, bar, temp, log)
@@ -357,7 +347,7 @@ def edit_media(
     else:
         make_media(tl, output)
 
-    timer.stop()
+    log.stop_timer()
 
     if not args.no_open and export["export"] in ("default", "audio", "clip-sequence"):
         if args.player is None:

{auto_editor-24.24.1 → auto_editor-24.27.1}/auto_editor/ffwrapper.py

@@ -11,6 +11,8 @@ from shutil import which
 from subprocess import PIPE, Popen
 from typing import Any
 
+import av
+
 from auto_editor.utils.func import get_stdout
 from auto_editor.utils.log import Log
 
@@ -190,10 +192,12 @@ class FileInfo:
 
 
 def initFileInfo(path: str, log: Log) -> FileInfo:
-    import av
-
     try:
         cont = av.open(path, "r")
+    except av.error.FileNotFoundError:
+        log.error(f"Could not find '{path}'")
+    except av.error.IsADirectoryError:
+        log.error(f"Expected a media file, but got a directory: {path}")
     except av.error.InvalidDataError:
         log.error(f"Invalid data when processing: {path}")
 

{auto_editor-24.24.1 → auto_editor-24.27.1}/auto_editor/formats/fcp7.py

@@ -7,7 +7,7 @@ from math import ceil
 from typing import TYPE_CHECKING
 from xml.etree.ElementTree import Element
 
-from auto_editor.ffwrapper import FFmpeg, FileInfo, initFileInfo
+from auto_editor.ffwrapper import FileInfo, initFileInfo
 from auto_editor.timeline import ASpace, TlAudio, TlVideo, VSpace, v3
 
 from .utils import Validator, show
@@ -177,7 +177,7 @@ def read_filters(clipitem: Element, log: Log) -> float:
     return 1.0
 
 
-def fcp7_read_xml(path: str, ffmpeg: FFmpeg, log: Log) -> v3:
+def fcp7_read_xml(path: str, log: Log) -> v3:
     def xml_bool(val: str) -> bool:
         if val == "TRUE":
             return True
@@ -188,7 +188,7 @@ def fcp7_read_xml(path: str, ffmpeg: FFmpeg, log: Log) -> v3:
     try:
         tree = ET.parse(path)
     except FileNotFoundError:
-        log.nofile(path)
+        log.error(f"Could not find '{path}'")
 
     root = tree.getroot()
 

{auto_editor-24.24.1 → auto_editor-24.27.1}/auto_editor/formats/shotcut.py

@@ -9,7 +9,6 @@ from auto_editor.utils.func import aspect_ratio, to_timecode
 if TYPE_CHECKING:
     from collections.abc import Sequence
 
-    from auto_editor.ffwrapper import FFmpeg
     from auto_editor.timeline import TlAudio, TlVideo
     from auto_editor.utils.log import Log
 
@@ -22,7 +21,7 @@ https://mltframework.org/docs/mltxml/
 """
 
 
-def shotcut_read_mlt(path: str, ffmpeg: FFmpeg, log: Log) -> v3:
+def shotcut_read_mlt(path: str, log: Log) -> v3:
     raise NotImplementedError
 
 

{auto_editor-24.24.1 → auto_editor-24.27.1}/auto_editor/make_layers.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 from fractions import Fraction
+from math import ceil
 from typing import TYPE_CHECKING, NamedTuple
 
 import numpy as np
@@ -37,7 +38,7 @@ def clipify(chunks: Chunks, src: FileInfo, start: int = 0) -> list[Clip]:
     clips: list[Clip] = []
     i = 0
     for chunk in chunks:
-        if chunk[2] != 99999:
+        if chunk[2] > 0 and chunk[2] < 99999.0:
             dur = round((chunk[1] - chunk[0]) / chunk[2])
             if dur == 0:
                 continue
@@ -112,8 +113,13 @@ def run_interpreter_for_edit_option(
 
 def make_sane_timebase(fps: Fraction) -> Fraction:
     tb = round(fps, 2)
+
+    ntsc_60 = Fraction(60_000, 1001)
     ntsc = Fraction(30_000, 1001)
     film_ntsc = Fraction(24_000, 1001)
+
+    if tb == round(ntsc_60, 2):
+        return ntsc_60
     if tb == round(ntsc, 2):
         return ntsc
     if tb == round(film_ntsc, 2):
@@ -226,21 +232,35 @@ def make_timeline(
         chunks.append((src, start, arr_length, speed_map[arr[j]]))
         return chunks
 
+    # Assert timeline is monotonic because non-monotonic timelines are incorrect
+    # here and causes back-seeking (performance issue) in video rendering.
+
+    # We don't properly check monotonicity for multiple sources, so skip those.
+
+    check_monotonic = len(sources) == 1
+    last_i = 0
+
     clips: list[Clip] = []
-    i = 0
     start = 0
+
     for chunk in echunk(speed_index, src_index):
         if chunk[3] != 99999:
-            dur = round((chunk[2] - chunk[1]) / chunk[3])
+            dur = int((chunk[2] - chunk[1]) / chunk[3])
             if dur == 0:
                 continue
 
-            offset = int(chunk[1] / chunk[3])
+            offset = ceil(chunk[1] / chunk[3])
+
+            if check_monotonic:
+                max_end = start + dur - 1
+                this_i = round((offset + max_end - start) * chunk[3])
+                if this_i < last_i:
+                    raise ValueError("not monotonic", sources, this_i, last_i)
+                last_i = this_i
+
+            clips.append(Clip(start, dur, offset, chunk[3], chunk[0]))
 
-            if not (clips and clips[-1].start == round(start)):
-                clips.append(Clip(start, dur, offset, chunk[3], chunk[0]))
             start += dur
-            i += 1
 
     vtl: VSpace = []
     atl: ASpace = []
@@ -276,4 +296,22 @@ def make_timeline(
     else:
         v1_compatiable = None
 
-    return v3(inp, tb, sr, res, args.background, vtl, atl, v1_compatiable)
+    tl = v3(inp, tb, sr, res, args.background, vtl, atl, v1_compatiable)
+
+    # Additional monotonic check, o(n^2) time complexity so disable by default.
+
+    # if len(sources) != 1:
+    #     return tl
+
+    # last_i = 0
+    # for index in range(tl.end):
+    #     for layer in tl.v:
+    #         for lobj in layer:
+    #             if index >= lobj.start and index < (lobj.start + lobj.dur):
+    #                 _i = round((lobj.offset + index - lobj.start) * lobj.speed)
+    #                 if (_i < last_i):
+    #                     print(_i, last_i)
+    #                     raise ValueError("not monotonic")
+    #                 last_i = _i
+
+    return tl

{auto_editor-24.24.1 → auto_editor-24.27.1}/auto_editor/output.py

@@ -4,7 +4,11 @@ import os.path
 from dataclasses import dataclass, field
 from fractions import Fraction
 
+import av
+from av.audio.resampler import AudioResampler
+
 from auto_editor.ffwrapper import FFmpeg, FileInfo
+from auto_editor.utils.bar import Bar
 from auto_editor.utils.container import Container
 from auto_editor.utils.log import Log
 from auto_editor.utils.types import Args
@@ -13,44 +17,76 @@ from auto_editor.utils.types import Args
 @dataclass(slots=True)
 class Ensure:
     _ffmpeg: FFmpeg
+    _bar: Bar
     _sr: int
     temp: str
     log: Log
-    labels: list[tuple[FileInfo, int]] = field(default_factory=list)
-    sub_labels: list[tuple[FileInfo, int]] = field(default_factory=list)
+    _audios: list[tuple[FileInfo, int]] = field(default_factory=list)
+    _subtitles: list[tuple[FileInfo, int, str]] = field(default_factory=list)
 
     def audio(self, src: FileInfo, stream: int) -> str:
         try:
-            label = self.labels.index((src, stream))
+            label = self._audios.index((src, stream))
             first_time = False
         except ValueError:
-            self.labels.append((src, stream))
-            label = len(self.labels) - 1
+            self._audios.append((src, stream))
+            label = len(self._audios) - 1
             first_time = True
 
         out_path = os.path.join(self.temp, f"{label:x}.wav")
 
         if first_time:
-            self.log.conwrite("Extracting audio")
+            sample_rate = self._sr
+            bar = self._bar
+            self.log.debug(f"Making external audio: {out_path}")
+
+            in_container = av.open(src.path, "r")
+            out_container = av.open(
+                out_path, "w", format="wav", options={"rf64": "always"}
+            )
+            astream = in_container.streams.audio[stream]
+
+            if astream.duration is None or astream.time_base is None:
+                dur = 1
+            else:
+                dur = int(astream.duration * astream.time_base)
+
+            bar.start(dur, "Extracting audio")
+
+            # PyAV always uses "stereo" layout, which is what we want.
+            output_astream = out_container.add_stream("pcm_s16le", rate=sample_rate)
+            assert isinstance(output_astream, av.audio.stream.AudioStream)
+
+            resampler = AudioResampler(format="s16", layout="stereo", rate=sample_rate)  # type: ignore
+            for i, frame in enumerate(in_container.decode(astream)):
+                if i % 1500 == 0:
+                    bar.tick(0 if frame.time is None else frame.time)
+
+                for new_frame in resampler.resample(frame):
+                    for packet in output_astream.encode(new_frame):
+                        out_container.mux_one(packet)
+
+            for packet in output_astream.encode():
+                out_container.mux_one(packet)
 
-            cmd = ["-i", f"{src.path}", "-map", f"0:a:{stream}"]
-            cmd += ["-ac", "2", "-ar", f"{self._sr}", "-rf64", "always", out_path]
-            self._ffmpeg.run(cmd)
+            out_container.close()
+            in_container.close()
+            bar.end()
 
         return out_path
 
-    def subtitle(self, src: FileInfo, stream: int) -> str:
+    def subtitle(self, src: FileInfo, stream: int, ext: str) -> str:
         try:
-            label = self.sub_labels.index((src, stream))
+            self._subtitles.index((src, stream, ext))
             first_time = False
         except ValueError:
-            self.sub_labels.append((src, stream))
-            label = len(self.sub_labels) - 1
+            self._subtitles.append((src, stream, ext))
             first_time = True
 
-        out_path = os.path.join(self.temp, f"{label:x}.vtt")
+        out_path = os.path.join(self.temp, f"{stream}s.{ext}")
 
         if first_time:
+            self.log.debug(f"Making external subtitle: {out_path}")
             self.log.conwrite("Extracting subtitle")
             self._ffmpeg.run(["-i", f"{src.path}", "-map", f"0:s:{stream}", out_path])
 

{auto_editor-24.24.1 → auto_editor-24.27.1}/auto_editor/render/subtitle.py

@@ -10,10 +10,9 @@ from auto_editor.utils.func import to_timecode
 if TYPE_CHECKING:
     from fractions import Fraction
 
-    from auto_editor.ffwrapper import FFmpeg
+    from auto_editor.output import Ensure
     from auto_editor.timeline import v3
     from auto_editor.utils.chunks import Chunks
-    from auto_editor.utils.log import Log
 
 
 @dataclass(slots=True)
@@ -122,26 +121,21 @@ class SubtitleParser:
             file.write(self.footer)
 
 
-def make_new_subtitles(tl: v3, ffmpeg: FFmpeg, temp: str, log: Log) -> list[str]:
+def make_new_subtitles(tl: v3, ensure: Ensure, temp: str) -> list[str]:
     if tl.v1 is None:
         return []
 
     new_paths = []
 
     for s, sub in enumerate(tl.v1.source.subtitles):
-        file_path = os.path.join(temp, f"{s}s.{sub.ext}")
         new_path = os.path.join(temp, f"new{s}s.{sub.ext}")
-
         parser = SubtitleParser(tl.tb)
 
-        if sub.codec in parser.supported_codecs:
-            with open(file_path, encoding="utf-8") as file:
-                parser.parse(file.read(), sub.codec)
-        else:
-            convert_path = os.path.join(temp, f"{s}s_convert.vtt")
-            ffmpeg.run(["-i", file_path, convert_path])
-            with open(convert_path, encoding="utf-8") as file:
-                parser.parse(file.read(), "webvtt")
+        ext = sub.ext if sub.codec in parser.supported_codecs else "vtt"
+        file_path = ensure.subtitle(tl.v1.source, s, ext)
+
+        with open(file_path, encoding="utf-8") as file:
+            parser.parse(file.read(), sub.codec)
 
         parser.edit(tl.v1.chunks)
         parser.write(new_path)