18652090836

Committed 20 Oct 2025 12:32PM UTC coverage: 71.425% (+0.1%) from 71.277%

Build # 18652090836

Build Type

push

github

Committed by

web-flow

Commit Message

feat: add codec suffix option (#128)

Run Details

38 of 45 new or added lines in 7 files covered. (84.44%)

446 existing lines in 7 files now uncovered.

5454 of 7636 relevant lines covered (71.42%)

0.71 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

84.11

/talks_reducer/cli.py

"""Command line interface for the talks reducer package."""

from __future__ import annotations

import argparse
import os
import shutil
import subprocess
import sys
import time
from importlib import import_module
from pathlib import Path
from typing import Callable, Dict, List, Optional, Sequence, Tuple

from . import audio
from .ffmpeg import FFmpegNotFoundError
from .models import ProcessingOptions, default_temp_folder
from .pipeline import speed_up_video
from .progress import TqdmProgressReporter
from .version_utils import resolve_version


def _build_parser() -> argparse.ArgumentParser:
    """Create the argument parser used by the command line interface."""

    parser = argparse.ArgumentParser(
        description="Modifies a video file to play at different speeds when there is sound vs. silence.",
    )

    # Add version argument
    pkg_version = resolve_version()

    parser.add_argument(
        "--version",
        action="version",
        version=f"talks-reducer {pkg_version}",
    )

    parser.add_argument(
        "input_file",
        type=str,
        nargs="+",
        help="The video file(s) you want modified. Can be one or more directories and / or single files.",
    )
    parser.add_argument(
        "-o",
        "--output_file",
        type=str,
        dest="output_file",
        help="The output file. Only usable if a single file is given. If not included, it'll append _ALTERED to the name.",
    )
    parser.add_argument(
        "--temp_folder",
        type=str,
        default=str(default_temp_folder()),
        help="The file path of the temporary working folder.",
    )
    parser.add_argument(
        "-t",
        "--silent_threshold",
        type=float,
        dest="silent_threshold",
        help="The volume amount that frames' audio needs to surpass to be considered sounded. Defaults to 0.05.",
    )
    parser.add_argument(
        "-S",
        "--sounded_speed",
        type=float,
        dest="sounded_speed",
        help="The speed that sounded (spoken) frames should be played at. Defaults to 1.",
    )
    parser.add_argument(
        "-s",
        "--silent_speed",
        type=float,
        dest="silent_speed",
        help="The speed that silent frames should be played at. Defaults to 4.",
    )
    parser.add_argument(
        "-fm",
        "--frame_margin",
        type=float,
        dest="frame_spreadage",
        help="Some silent frames adjacent to sounded frames are included to provide context. Defaults to 2.",
    )
    parser.add_argument(
        "-sr",
        "--sample_rate",
        type=float,
        dest="sample_rate",
        help="Sample rate of the input and output videos. Usually extracted automatically by FFmpeg.",
    )
    parser.add_argument(
        "--keyframe-interval",
        type=float,
        dest="keyframe_interval_seconds",
        help="Override the keyframe spacing in seconds when using --small. Defaults to 30.",
    )
    parser.add_argument(
        "--video-codec",
        choices=["h264", "hevc", "av1"],
        default="hevc",
        help=(
            "Select the video encoder used for the final render (default: hevc — "
            "h.265 for roughly 25% smaller files). Pick h264 (about 10% faster) "
            "when speed matters or av1 (no advantages) for experimental runs."
        ),
    )
    parser.add_argument(
        "--add-codec-suffix",
        dest="add_codec_suffix",
        action="store_true",
        help="Append the selected video codec to the default output filename.",
    )
    parser.add_argument(
        "--prefer-global-ffmpeg",
        action="store_true",
        help="Use an FFmpeg binary from PATH before falling back to the bundled static build.",
    )
    parser.add_argument(
        "--small",
        action="store_true",
        help="Apply small file optimizations: resize video to 720p (or 480p with --480), audio to 128k bitrate, best compression (uses CUDA if available).",
    )
    parser.add_argument(
        "--480",
        dest="small_480",
        action="store_true",
        help="Use with --small to scale video to 480p instead of 720p.",
    )
    parser.add_argument(
        "--url",
        dest="server_url",
        default=None,
        help="Process videos via a Talks Reducer server at the provided base URL (for example, http://localhost:9005).",
    )
    parser.add_argument(
        "--host",
        dest="host",
        default=None,
        help="Shortcut for --url when targeting a Talks Reducer server on port 9005 (for example, localhost).",
    )
    parser.add_argument(
        "--server-stream",
        action="store_true",
        help="Stream remote progress updates when using --url.",
    )
    return parser


def gather_input_files(paths: List[str]) -> List[str]:
    """Expand provided paths into a flat list of files that contain audio streams."""

    files: List[str] = []
    for input_path in paths:
        if os.path.isfile(input_path) and audio.is_valid_input_file(input_path):
            files.append(os.path.abspath(input_path))
        elif os.path.isdir(input_path):
            for file in os.listdir(input_path):
                candidate = os.path.join(input_path, file)
                if audio.is_valid_input_file(candidate):
                    files.append(candidate)
    return files


def _print_total_time(start_time: float) -> None:
    """Print the elapsed processing time since *start_time*."""

    end_time = time.time()
    total_time = end_time - start_time
    hours, remainder = divmod(total_time, 3600)
    minutes, seconds = divmod(remainder, 60)
    print(f"\nTime: {int(hours)}h {int(minutes)}m {seconds:.2f}s")


class CliApplication:
    """Coordinator for CLI processing with dependency injection support."""

    def __init__(
        self,
        *,
        gather_files: Callable[[List[str]], List[str]],
        send_video: Optional[Callable[..., Tuple[Path, str, str]]],
        speed_up: Callable[[ProcessingOptions, object], object],
        reporter_factory: Callable[[], object],
        remote_error_message: Optional[str] = None,
    ) -> None:
        self._gather_files = gather_files
        self._send_video = send_video
        self._speed_up = speed_up
        self._reporter_factory = reporter_factory
        self._remote_error_message = remote_error_message

    def run(self, parsed_args: argparse.Namespace) -> Tuple[int, List[str]]:
        """Execute the CLI pipeline for *parsed_args*."""

        start_time = time.time()
        files = self._gather_files(parsed_args.input_file)

        args: Dict[str, object] = {
            key: value for key, value in vars(parsed_args).items() if value is not None
        }
        del args["input_file"]

        if "host" in args:
            del args["host"]

        if len(files) > 1 and "output_file" in args:
            del args["output_file"]

        if getattr(parsed_args, "small_480", False) and not getattr(
            parsed_args, "small", False
        ):
            print(
                "Warning: --480 has no effect unless --small is also provided.",
                file=sys.stderr,
            )

        error_messages: List[str] = []
        reporter_logs: List[str] = []

        if getattr(parsed_args, "server_url", None):
            remote_success, remote_errors, fallback_logs = self._process_via_server(
                files, parsed_args, start_time
            )
            error_messages.extend(remote_errors)
            reporter_logs.extend(fallback_logs)
            if remote_success:
                return 0, error_messages

        reporter = self._reporter_factory()
        for message in reporter_logs:
            reporter.log(message)

        for index, file in enumerate(files):
            print(
                f"Processing file {index + 1}/{len(files)} '{os.path.basename(file)}'"
            )
            local_options = dict(args)

            option_kwargs: Dict[str, object] = {"input_file": Path(file)}

            if "output_file" in local_options:
                option_kwargs["output_file"] = Path(local_options["output_file"])
            if "temp_folder" in local_options:
                option_kwargs["temp_folder"] = Path(local_options["temp_folder"])
            if "silent_threshold" in local_options:
                option_kwargs["silent_threshold"] = float(
                    local_options["silent_threshold"]
                )
            if "silent_speed" in local_options:
                option_kwargs["silent_speed"] = float(local_options["silent_speed"])
            if "sounded_speed" in local_options:
                option_kwargs["sounded_speed"] = float(local_options["sounded_speed"])
            if "frame_spreadage" in local_options:
                option_kwargs["frame_spreadage"] = int(local_options["frame_spreadage"])
            if "sample_rate" in local_options:
                option_kwargs["sample_rate"] = int(local_options["sample_rate"])
            if "keyframe_interval_seconds" in local_options:
                option_kwargs["keyframe_interval_seconds"] = float(
                    local_options["keyframe_interval_seconds"]
                )
            if "video_codec" in local_options:
                option_kwargs["video_codec"] = str(local_options["video_codec"])
            if local_options.get("add_codec_suffix"):
                option_kwargs["add_codec_suffix"] = True
            if "small" in local_options:
                option_kwargs["small"] = bool(local_options["small"])
            if local_options.get("small_480"):
                option_kwargs["small_target_height"] = 480
            if "prefer_global_ffmpeg" in local_options:
                option_kwargs["prefer_global_ffmpeg"] = bool(
                    local_options["prefer_global_ffmpeg"]
                )
            options = ProcessingOptions(**option_kwargs)

            try:
                result = self._speed_up(options, reporter=reporter)
            except FFmpegNotFoundError as exc:
                message = str(exc)
                return 1, [*error_messages, message]

            reporter.log(f"Completed: {result.output_file}")
            summary_parts: List[str] = []
            time_ratio = getattr(result, "time_ratio", None)
            size_ratio = getattr(result, "size_ratio", None)
            if time_ratio is not None:
                summary_parts.append(f"{time_ratio * 100:.0f}% time")
            if size_ratio is not None:
                summary_parts.append(f"{size_ratio * 100:.0f}% size")
            if summary_parts:
                reporter.log("Result: " + ", ".join(summary_parts))

        _print_total_time(start_time)
        return 0, error_messages

    def _process_via_server(
        self,
        files: Sequence[str],
        parsed_args: argparse.Namespace,
        start_time: float,
    ) -> Tuple[bool, List[str], List[str]]:
        """Upload *files* to the configured server and download the results."""

        if not self._send_video:
            message = self._remote_error_message or "Server processing is unavailable."
            fallback_notice = "Falling back to local processing pipeline."
            return False, [message, fallback_notice], [message, fallback_notice]

        server_url = parsed_args.server_url
        if not server_url:
            message = "Server URL was not provided."
            fallback_notice = "Falling back to local processing pipeline."
            return False, [message, fallback_notice], [message, fallback_notice]

        output_override: Optional[Path] = None
        if parsed_args.output_file and len(files) == 1:
            output_override = Path(parsed_args.output_file).expanduser()
        elif parsed_args.output_file and len(files) > 1:
            print(
                "Warning: --output is ignored when processing multiple files via the server.",
                file=sys.stderr,
            )

        remote_option_values: Dict[str, object] = {}
        if parsed_args.silent_threshold is not None:
            remote_option_values["silent_threshold"] = float(
                parsed_args.silent_threshold
            )
        if parsed_args.silent_speed is not None:
            remote_option_values["silent_speed"] = float(parsed_args.silent_speed)
        if parsed_args.sounded_speed is not None:
            remote_option_values["sounded_speed"] = float(parsed_args.sounded_speed)
        if getattr(parsed_args, "video_codec", None):
            remote_option_values["video_codec"] = str(parsed_args.video_codec)
        if getattr(parsed_args, "add_codec_suffix", False):
            remote_option_values["add_codec_suffix"] = True
        if getattr(parsed_args, "prefer_global_ffmpeg", False):
            remote_option_values["prefer_global_ffmpeg"] = True

        unsupported_options: List[str] = []
        for name in (
            "frame_spreadage",
            "sample_rate",
            "temp_folder",
            "keyframe_interval_seconds",
        ):
            if getattr(parsed_args, name) is not None:
                unsupported_options.append(f"--{name.replace('_', '-')}")

        if unsupported_options:
            print(
                "Warning: the following options are ignored when using --url: "
                + ", ".join(sorted(unsupported_options)),
                file=sys.stderr,
            )

        small_480_mode = bool(getattr(parsed_args, "small_480", False)) and bool(
            getattr(parsed_args, "small", False)
        )
        if small_480_mode:
            remote_option_values["small_480"] = True

        for index, file in enumerate(files, start=1):
            basename = os.path.basename(file)
            print(
                f"Processing file {index}/{len(files)} '{basename}' via server {server_url}"
            )
            printed_log_header = False
            progress_state: dict[str, tuple[Optional[int], Optional[int], str]] = {}
            stream_updates = bool(getattr(parsed_args, "server_stream", False))

            def _stream_server_log(line: str) -> None:
                nonlocal printed_log_header
                if not printed_log_header:
                    print("\nServer log:", flush=True)
                    printed_log_header = True
                print(line, flush=True)

            def _stream_progress(
                desc: str, current: Optional[int], total: Optional[int], unit: str
            ) -> None:
                key = desc or "Processing"
                state = (current, total, unit)
                if progress_state.get(key) == state:
                    return
                progress_state[key] = state

                parts: List[str] = []
                if current is not None and total and total > 0:
                    percent = (current / total) * 100
                    parts.append(f"{current}/{total}")
                    parts.append(f"{percent:.1f}%")
                elif current is not None:
                    parts.append(str(current))
                if unit:
                    parts.append(unit)
                message = " ".join(parts).strip()
                print(f"{key}: {message or 'update'}", flush=True)

            try:
                destination, summary, log_text = self._send_video(
                    input_path=Path(file),
                    output_path=output_override,
                    server_url=server_url,
                    small=bool(parsed_args.small),
                    small_480=small_480_mode,
                    **remote_option_values,
                    log_callback=_stream_server_log,
                    stream_updates=stream_updates,
                    progress_callback=_stream_progress if stream_updates else None,
                )
            except Exception as exc:  # pragma: no cover - network failure safeguard
                message = f"Failed to process {basename} via server: {exc}"
                fallback_notice = "Falling back to local processing pipeline."
                return False, [message, fallback_notice], [message, fallback_notice]

            print(summary)
            print(f"Saved processed video to {destination}")
            if log_text.strip() and not printed_log_header:
                print("\nServer log:\n" + log_text)

        _print_total_time(start_time)
        return True, [], []


def _launch_gui(argv: Sequence[str]) -> bool:
    """Attempt to launch the GUI with the provided arguments."""

    try:
        gui_module = import_module(".gui", __package__)
    except ImportError:
        return False

    gui_main = getattr(gui_module, "main", None)
    if gui_main is None:
        return False

    return bool(gui_main(list(argv)))


def _launch_server(argv: Sequence[str]) -> bool:
    """Attempt to launch the Gradio server with the provided arguments."""

    try:
        server_module = import_module(".server", __package__)
    except ImportError:
        return False

    server_main = getattr(server_module, "main", None)
    if server_main is None:
        return False

    server_main(list(argv))
    return True


def _find_server_tray_binary() -> Optional[Path]:
    """Return the best available path to the server tray executable."""

    binary_name = "talks-reducer-server-tray"
    candidates: List[Path] = []

    which_path = shutil.which(binary_name)
    if which_path:
        candidates.append(Path(which_path))

    try:
        launcher_dir = Path(sys.argv[0]).resolve().parent
    except Exception:
        launcher_dir = None

    potential_names = [binary_name]
    if sys.platform == "win32":
        potential_names = [f"{binary_name}.exe", binary_name]

    if launcher_dir is not None:
        for name in potential_names:
            candidates.append(launcher_dir / name)

    for candidate in candidates:
        if candidate and candidate.exists() and os.access(candidate, os.X_OK):
            return candidate

    return None


def _should_hide_subprocess_console() -> bool:
    """Return ``True` ` when a detached Windows launch should hide the console."""

    if sys.platform != "win32":
        return False

    try:
        import ctypes
    except Exception:  # pragma: no cover - optional runtime dependency
        return False

    try:
        get_console_window = ctypes.windll.kernel32.GetConsoleWindow  # type: ignore[attr-defined]
    except Exception:  # pragma: no cover - platform specific guard
        return False

    try:
        handle = get_console_window()
    except Exception:  # pragma: no cover - defensive fallback
        return False

    return handle == 0


def _launch_server_tray_binary(argv: Sequence[str]) -> bool:
    """Launch the packaged server tray executable when available."""

    command = _find_server_tray_binary()
    if command is None:
        return False

    tray_args = [str(command), *list(argv)]

    run_kwargs: Dict[str, object] = {"check": False}

    if sys.platform == "win32":
        no_window_flag = getattr(subprocess, "CREATE_NO_WINDOW", 0)
        if no_window_flag and _should_hide_subprocess_console():
            run_kwargs["creationflags"] = no_window_flag

    try:
        result = subprocess.run(tray_args, **run_kwargs)
    except OSError:
        return False

    return result.returncode == 0


def _launch_server_tray(argv: Sequence[str]) -> bool:
    """Attempt to launch the server tray helper with the provided arguments."""

    if _launch_server_tray_binary(argv):
        return True

    try:
        tray_module = import_module(".server_tray", __package__)
    except ImportError:
        return False

    tray_main = getattr(tray_module, "main", None)
    if tray_main is None:
        return False

    tray_main(list(argv))
    return True


def main(argv: Optional[Sequence[str]] = None) -> None:
    """Entry point for the command line interface.

    Launch the GUI when run without arguments, otherwise defer to the CLI.
    """

    if argv is None:
        argv_list = sys.argv[1:]
    else:
        argv_list = list(argv)

    if "--server" in argv_list:
        index = argv_list.index("--server")
        tray_args = argv_list[index + 1 :]
        if not _launch_server_tray(tray_args):
            print("Server tray mode is unavailable.", file=sys.stderr)
            sys.exit(1)
        return

    if argv_list and argv_list[0] in {"server", "serve"}:
        if not _launch_server(argv_list[1:]):
            print("Gradio server mode is unavailable.", file=sys.stderr)
            sys.exit(1)
        return

    if not argv_list:
        if _launch_gui(argv_list):
            return

        parser = _build_parser()
        parser.print_help()
        return

    parser = _build_parser()
    parsed_args = parser.parse_args(argv_list)

    host_value = getattr(parsed_args, "host", None)
    if host_value:
        parsed_args.server_url = f"http://{host_value}:9005"

    send_video = None
    remote_error_message: Optional[str] = None
    try:  # pragma: no cover - optional dependency guard
        from . import service_client
    except ImportError as exc:
        remote_error_message = (
            "Server mode requires the gradio_client dependency. " f"({exc})"
        )
    else:
        send_video = service_client.send_video

    application = CliApplication(
        gather_files=gather_input_files,
        send_video=send_video,
        speed_up=speed_up_video,
        reporter_factory=TqdmProgressReporter,
        remote_error_message=remote_error_message,
    )

    exit_code, error_messages = application.run(parsed_args)
    for message in error_messages:
        print(message, file=sys.stderr)
    if exit_code:
        sys.exit(exit_code)


if __name__ == "__main__":
    main()

1	"""Command line interface for the talks reducer package."""
2
3	from __future__ import annotations	1✔
4
5	import argparse	1✔
6	import os	1✔
7	import shutil	1✔
8	import subprocess	1✔
9	import sys	1✔
10	import time	1✔
11	from importlib import import_module	1✔
12	from pathlib import Path	1✔
13	from typing import Callable, Dict, List, Optional, Sequence, Tuple	1✔
14
15	from . import audio	1✔
16	from .ffmpeg import FFmpegNotFoundError	1✔
17	from .models import ProcessingOptions, default_temp_folder	1✔
18	from .pipeline import speed_up_video	1✔
19	from .progress import TqdmProgressReporter	1✔
20	from .version_utils import resolve_version	1✔
21
22
23	def _build_parser() -> argparse.ArgumentParser:	1✔
24	"""Create the argument parser used by the command line interface."""
25
26	parser = argparse.ArgumentParser(	1✔
27	description="Modifies a video file to play at different speeds when there is sound vs. silence.",
28	)
29
30	# Add version argument
31	pkg_version = resolve_version()	1✔
32
33	parser.add_argument(	1✔
34	"--version",
35	action="version",
36	version=f"talks-reducer {pkg_version}",
37	)
38
39	parser.add_argument(	1✔
40	"input_file",
41	type=str,
42	nargs="+",
43	help="The video file(s) you want modified. Can be one or more directories and / or single files.",
44	)
45	parser.add_argument(	1✔
46	"-o",
47	"--output_file",
48	type=str,
49	dest="output_file",
50	help="The output file. Only usable if a single file is given. If not included, it'll append _ALTERED to the name.",
51	)
52	parser.add_argument(	1✔
53	"--temp_folder",
54	type=str,
55	default=str(default_temp_folder()),
56	help="The file path of the temporary working folder.",
57	)
58	parser.add_argument(	1✔
59	"-t",
60	"--silent_threshold",
61	type=float,
62	dest="silent_threshold",
63	help="The volume amount that frames' audio needs to surpass to be considered sounded. Defaults to 0.05.",
64	)
65	parser.add_argument(	1✔
66	"-S",
67	"--sounded_speed",
68	type=float,
69	dest="sounded_speed",
70	help="The speed that sounded (spoken) frames should be played at. Defaults to 1.",
71	)
72	parser.add_argument(	1✔
73	"-s",
74	"--silent_speed",
75	type=float,
76	dest="silent_speed",
77	help="The speed that silent frames should be played at. Defaults to 4.",
78	)
79	parser.add_argument(	1✔
80	"-fm",
81	"--frame_margin",
82	type=float,
83	dest="frame_spreadage",
84	help="Some silent frames adjacent to sounded frames are included to provide context. Defaults to 2.",
85	)
86	parser.add_argument(	1✔
87	"-sr",
88	"--sample_rate",
89	type=float,
90	dest="sample_rate",
91	help="Sample rate of the input and output videos. Usually extracted automatically by FFmpeg.",
92	)
93	parser.add_argument(	1✔
94	"--keyframe-interval",
95	type=float,
96	dest="keyframe_interval_seconds",
97	help="Override the keyframe spacing in seconds when using --small. Defaults to 30.",
98	)
99	parser.add_argument(	1✔
100	"--video-codec",
101	choices=["h264", "hevc", "av1"],
102	default="hevc",
103	help=(
104	"Select the video encoder used for the final render (default: hevc — "
105	"h.265 for roughly 25% smaller files). Pick h264 (about 10% faster) "
106	"when speed matters or av1 (no advantages) for experimental runs."
107	),
108	)
109	parser.add_argument(	1✔
110	"--add-codec-suffix",
111	dest="add_codec_suffix",
112	action="store_true",
113	help="Append the selected video codec to the default output filename.",
114	)
115	parser.add_argument(	1✔
116	"--prefer-global-ffmpeg",
117	action="store_true",
118	help="Use an FFmpeg binary from PATH before falling back to the bundled static build.",
119	)
120	parser.add_argument(	1✔
121	"--small",
122	action="store_true",
123	help="Apply small file optimizations: resize video to 720p (or 480p with --480), audio to 128k bitrate, best compression (uses CUDA if available).",
124	)
125	parser.add_argument(	1✔
126	"--480",
127	dest="small_480",
128	action="store_true",
129	help="Use with --small to scale video to 480p instead of 720p.",
130	)
131	parser.add_argument(	1✔
132	"--url",
133	dest="server_url",
134	default=None,
135	help="Process videos via a Talks Reducer server at the provided base URL (for example, http://localhost:9005).",
136	)
137	parser.add_argument(	1✔
138	"--host",
139	dest="host",
140	default=None,
141	help="Shortcut for --url when targeting a Talks Reducer server on port 9005 (for example, localhost).",
142	)
143	parser.add_argument(	1✔
144	"--server-stream",
145	action="store_true",
146	help="Stream remote progress updates when using --url.",
147	)
148	return parser	1✔
149
150
151	def gather_input_files(paths: List[str]) -> List[str]:	1✔
152	"""Expand provided paths into a flat list of files that contain audio streams."""
153
154	files: List[str] = []	1✔
155	for input_path in paths:	1✔
156	if os.path.isfile(input_path) and audio.is_valid_input_file(input_path):	1✔
157	files.append(os.path.abspath(input_path))	1✔
158	elif os.path.isdir(input_path):	1✔
159	for file in os.listdir(input_path):	1✔
160	candidate = os.path.join(input_path, file)	1✔
161	if audio.is_valid_input_file(candidate):	1✔
162	files.append(candidate)	1✔
163	return files	1✔
164
165
166	def _print_total_time(start_time: float) -> None:	1✔
167	"""Print the elapsed processing time since start_time."""
168
169	end_time = time.time()	1✔
170	total_time = end_time - start_time	1✔
171	hours, remainder = divmod(total_time, 3600)	1✔
172	minutes, seconds = divmod(remainder, 60)	1✔
173	print(f"\nTime: {int(hours)}h {int(minutes)}m {seconds:.2f}s")	1✔
174
175
176	class CliApplication:	1✔
177	"""Coordinator for CLI processing with dependency injection support."""
178
179	def __init__(	1✔
180	self,
181	*,
182	gather_files: Callable[[List[str]], List[str]],
183	send_video: Optional[Callable[..., Tuple[Path, str, str]]],
184	speed_up: Callable[[ProcessingOptions, object], object],
185	reporter_factory: Callable[[], object],
186	remote_error_message: Optional[str] = None,
187	) -> None:
188	self._gather_files = gather_files	1✔
189	self._send_video = send_video	1✔
190	self._speed_up = speed_up	1✔
191	self._reporter_factory = reporter_factory	1✔
192	self._remote_error_message = remote_error_message	1✔
193
194	def run(self, parsed_args: argparse.Namespace) -> Tuple[int, List[str]]:	1✔
195	"""Execute the CLI pipeline for parsed_args."""
196
197	start_time = time.time()	1✔
198	files = self._gather_files(parsed_args.input_file)	1✔
199
200	args: Dict[str, object] = {	1✔
201	key: value for key, value in vars(parsed_args).items() if value is not None
202	}
203	del args["input_file"]	1✔
204
205	if "host" in args:	1✔
206	del args["host"]	×
207
208	if len(files) > 1 and "output_file" in args:	1✔
209	del args["output_file"]	×
210
211	if getattr(parsed_args, "small_480", False) and not getattr(	1✔
212	parsed_args, "small", False
213	):
214	print(	×
215	"Warning: --480 has no effect unless --small is also provided.",
216	file=sys.stderr,
217	)
218
219	error_messages: List[str] = []	1✔
220	reporter_logs: List[str] = []	1✔
221
222	if getattr(parsed_args, "server_url", None):	1✔
223	remote_success, remote_errors, fallback_logs = self._process_via_server(	1✔
224	files, parsed_args, start_time
225	)
226	error_messages.extend(remote_errors)	1✔
227	reporter_logs.extend(fallback_logs)	1✔
228	if remote_success:	1✔
229	return 0, error_messages	1✔
230
231	reporter = self._reporter_factory()	1✔
232	for message in reporter_logs:	1✔
233	reporter.log(message)	1✔
234
235	for index, file in enumerate(files):	1✔
236	print(	1✔
237	f"Processing file {index + 1}/{len(files)} '{os.path.basename(file)}'"
238	)
239	local_options = dict(args)	1✔
240
241	option_kwargs: Dict[str, object] = {"input_file": Path(file)}	1✔
242
243	if "output_file" in local_options:	1✔
244	option_kwargs["output_file"] = Path(local_options["output_file"])	1✔
245	if "temp_folder" in local_options:	1✔
246	option_kwargs["temp_folder"] = Path(local_options["temp_folder"])	1✔
247	if "silent_threshold" in local_options:	1✔
248	option_kwargs["silent_threshold"] = float(	1✔
249	local_options["silent_threshold"]
250	)
251	if "silent_speed" in local_options:	1✔
252	option_kwargs["silent_speed"] = float(local_options["silent_speed"])	1✔
253	if "sounded_speed" in local_options:	1✔
254	option_kwargs["sounded_speed"] = float(local_options["sounded_speed"])	1✔
255	if "frame_spreadage" in local_options:	1✔
256	option_kwargs["frame_spreadage"] = int(local_options["frame_spreadage"])	1✔
257	if "sample_rate" in local_options:	1✔
258	option_kwargs["sample_rate"] = int(local_options["sample_rate"])	1✔
259	if "keyframe_interval_seconds" in local_options:	1✔
260	option_kwargs["keyframe_interval_seconds"] = float(	1✔
261	local_options["keyframe_interval_seconds"]
262	)
263	if "video_codec" in local_options:	1✔
264	option_kwargs["video_codec"] = str(local_options["video_codec"])	1✔
265	if local_options.get("add_codec_suffix"):	1✔
266	option_kwargs["add_codec_suffix"] = True	1✔
267	if "small" in local_options:	1✔
268	option_kwargs["small"] = bool(local_options["small"])	1✔
269	if local_options.get("small_480"):	1✔
270	option_kwargs["small_target_height"] = 480	×
271	if "prefer_global_ffmpeg" in local_options:	1✔
272	option_kwargs["prefer_global_ffmpeg"] = bool(	1✔
273	local_options["prefer_global_ffmpeg"]
274	)
275	options = ProcessingOptions(**option_kwargs)	1✔
276
277	try:	1✔
278	result = self._speed_up(options, reporter=reporter)	1✔
279	except FFmpegNotFoundError as exc:	×
280	message = str(exc)	×
281	return 1, [*error_messages, message]	×
282
283	reporter.log(f"Completed: {result.output_file}")	1✔
284	summary_parts: List[str] = []	1✔
285	time_ratio = getattr(result, "time_ratio", None)	1✔
286	size_ratio = getattr(result, "size_ratio", None)	1✔
287	if time_ratio is not None:	1✔
288	summary_parts.append(f"{time_ratio * 100:.0f}% time")	1✔
289	if size_ratio is not None:	1✔
290	summary_parts.append(f"{size_ratio * 100:.0f}% size")	1✔
291	if summary_parts:	1✔
292	reporter.log("Result: " + ", ".join(summary_parts))	1✔
293
294	_print_total_time(start_time)	1✔
295	return 0, error_messages	1✔
296
297	def _process_via_server(	1✔
298	self,
299	files: Sequence[str],
300	parsed_args: argparse.Namespace,
301	start_time: float,
302	) -> Tuple[bool, List[str], List[str]]:
303	"""Upload files to the configured server and download the results."""
304
305	if not self._send_video:	1✔
306	message = self._remote_error_message or "Server processing is unavailable."	1✔
307	fallback_notice = "Falling back to local processing pipeline."	1✔
308	return False, [message, fallback_notice], [message, fallback_notice]	1✔
309
310	server_url = parsed_args.server_url	1✔
311	if not server_url:	1✔
312	message = "Server URL was not provided."	×
313	fallback_notice = "Falling back to local processing pipeline."	×
314	return False, [message, fallback_notice], [message, fallback_notice]	×
315
316	output_override: Optional[Path] = None	1✔
317	if parsed_args.output_file and len(files) == 1:	1✔
318	output_override = Path(parsed_args.output_file).expanduser()	×
319	elif parsed_args.output_file and len(files) > 1:	1✔
320	print(	1✔
321	"Warning: --output is ignored when processing multiple files via the server.",
322	file=sys.stderr,
323	)
324
325	remote_option_values: Dict[str, object] = {}	1✔
326	if parsed_args.silent_threshold is not None:	1✔
327	remote_option_values["silent_threshold"] = float(	1✔
328	parsed_args.silent_threshold
329	)
330	if parsed_args.silent_speed is not None:	1✔
331	remote_option_values["silent_speed"] = float(parsed_args.silent_speed)	1✔
332	if parsed_args.sounded_speed is not None:	1✔
333	remote_option_values["sounded_speed"] = float(parsed_args.sounded_speed)	1✔
334	if getattr(parsed_args, "video_codec", None):	1✔
335	remote_option_values["video_codec"] = str(parsed_args.video_codec)	1✔
336	if getattr(parsed_args, "add_codec_suffix", False):	1✔
NEW 337	remote_option_values["add_codec_suffix"] = True	×
338	if getattr(parsed_args, "prefer_global_ffmpeg", False):	1✔
339	remote_option_values["prefer_global_ffmpeg"] = True	1✔
340
341	unsupported_options: List[str] = []	1✔
342	for name in (	1✔
343	"frame_spreadage",
344	"sample_rate",
345	"temp_folder",
346	"keyframe_interval_seconds",
347	):
348	if getattr(parsed_args, name) is not None:	1✔
349	unsupported_options.append(f"--{name.replace('_', '-')}")	1✔
350
351	if unsupported_options:	1✔
352	print(	1✔
353	"Warning: the following options are ignored when using --url: "
354	+ ", ".join(sorted(unsupported_options)),
355	file=sys.stderr,
356	)
357
358	small_480_mode = bool(getattr(parsed_args, "small_480", False)) and bool(	1✔
359	getattr(parsed_args, "small", False)
360	)
361	if small_480_mode:	1✔
362	remote_option_values["small_480"] = True	×
363
364	for index, file in enumerate(files, start=1):	1✔
365	basename = os.path.basename(file)	1✔
366	print(	1✔
367	f"Processing file {index}/{len(files)} '{basename}' via server {server_url}"
368	)
369	printed_log_header = False	1✔
370	progress_state: dict[str, tuple[Optional[int], Optional[int], str]] = {}	1✔
371	stream_updates = bool(getattr(parsed_args, "server_stream", False))	1✔
372
373	def _stream_server_log(line: str) -> None:	1✔
374	nonlocal printed_log_header
375	if not printed_log_header:	1✔
376	print("\nServer log:", flush=True)	1✔
377	printed_log_header = True	1✔
378	print(line, flush=True)	1✔
379
380	def _stream_progress(	1✔
381	desc: str, current: Optional[int], total: Optional[int], unit: str
382	) -> None:
383	key = desc or "Processing"	1✔
384	state = (current, total, unit)	1✔
385	if progress_state.get(key) == state:	1✔
386	return	1✔
387	progress_state[key] = state	1✔
388
389	parts: List[str] = []	1✔
390	if current is not None and total and total > 0:	1✔
391	percent = (current / total) * 100	1✔
392	parts.append(f"{current}/{total}")	1✔
393	parts.append(f"{percent:.1f}%")	1✔
394	elif current is not None:	1✔
395	parts.append(str(current))	×
396	if unit:	1✔
397	parts.append(unit)	1✔
398	message = " ".join(parts).strip()	1✔
399	print(f"{key}: {message or 'update'}", flush=True)	1✔
400
401	try:	1✔
402	destination, summary, log_text = self._send_video(	1✔
403	input_path=Path(file),
404	output_path=output_override,
405	server_url=server_url,
406	small=bool(parsed_args.small),
407	small_480=small_480_mode,
408	**remote_option_values,
409	log_callback=_stream_server_log,
410	stream_updates=stream_updates,
411	progress_callback=_stream_progress if stream_updates else None,
412	)
413	except Exception as exc: # pragma: no cover - network failure safeguard
414	message = f"Failed to process {basename} via server: {exc}"
415	fallback_notice = "Falling back to local processing pipeline."
416	return False, [message, fallback_notice], [message, fallback_notice]
417
418	print(summary)	1✔
419	print(f"Saved processed video to {destination}")	1✔
420	if log_text.strip() and not printed_log_header:	1✔
421	print("\nServer log:\n" + log_text)	1✔
422
423	_print_total_time(start_time)	1✔
424	return True, [], []	1✔
425
426
427	def _launch_gui(argv: Sequence[str]) -> bool:	1✔
428	"""Attempt to launch the GUI with the provided arguments."""
429
430	try:	×
431	gui_module = import_module(".gui", __package__)	×
432	except ImportError:	×
433	return False	×
434
435	gui_main = getattr(gui_module, "main", None)	×
436	if gui_main is None:	×
437	return False	×
438
439	return bool(gui_main(list(argv)))	×
440
441
442	def _launch_server(argv: Sequence[str]) -> bool:	1✔
443	"""Attempt to launch the Gradio server with the provided arguments."""
444
445	try:	×
446	server_module = import_module(".server", __package__)	×
447	except ImportError:	×
448	return False	×
449
450	server_main = getattr(server_module, "main", None)	×
451	if server_main is None:	×
452	return False	×
453
454	server_main(list(argv))	×
455	return True	×
456
457
458	def _find_server_tray_binary() -> Optional[Path]:	1✔
459	"""Return the best available path to the server tray executable."""
460
461	binary_name = "talks-reducer-server-tray"	1✔
462	candidates: List[Path] = []	1✔
463
464	which_path = shutil.which(binary_name)	1✔
465	if which_path:	1✔
466	candidates.append(Path(which_path))	1✔
467
468	try:	1✔
469	launcher_dir = Path(sys.argv[0]).resolve().parent	1✔
470	except Exception:	×
471	launcher_dir = None	×
472
473	potential_names = [binary_name]	1✔
474	if sys.platform == "win32":	1✔
475	potential_names = [f"{binary_name}.exe", binary_name]	×
476
477	if launcher_dir is not None:	1✔
478	for name in potential_names:	1✔
479	candidates.append(launcher_dir / name)	1✔
480
481	for candidate in candidates:	1✔
482	if candidate and candidate.exists() and os.access(candidate, os.X_OK):	1✔
483	return candidate	1✔
484
485	return None	×
486
487
488	def _should_hide_subprocess_console() -> bool:	1✔
489	"""Return ``True` ` when a detached Windows launch should hide the console."""
490
491	if sys.platform != "win32":	1✔
492	return False	1✔
493
494	try:	1✔
495	import ctypes	1✔
496	except Exception: # pragma: no cover - optional runtime dependency
497	return False
498
499	try:	1✔
500	get_console_window = ctypes.windll.kernel32.GetConsoleWindow # type: ignore[attr-defined]	1✔
501	except Exception: # pragma: no cover - platform specific guard
502	return False
503
504	try:	1✔
505	handle = get_console_window()	1✔
506	except Exception: # pragma: no cover - defensive fallback
507	return False
508
509	return handle == 0	1✔
510
511
512	def _launch_server_tray_binary(argv: Sequence[str]) -> bool:	1✔
513	"""Launch the packaged server tray executable when available."""
514
515	command = _find_server_tray_binary()	1✔
516	if command is None:	1✔
517	return False	1✔
518
519	tray_args = [str(command), *list(argv)]	1✔
520
521	run_kwargs: Dict[str, object] = {"check": False}	1✔
522
523	if sys.platform == "win32":	1✔
524	no_window_flag = getattr(subprocess, "CREATE_NO_WINDOW", 0)	1✔
525	if no_window_flag and _should_hide_subprocess_console():	1✔
526	run_kwargs["creationflags"] = no_window_flag	1✔
527
528	try:	1✔
529	result = subprocess.run(tray_args, **run_kwargs)	1✔
530	except OSError:	×
531	return False	×
532
533	return result.returncode == 0	1✔
534
535
536	def _launch_server_tray(argv: Sequence[str]) -> bool:	1✔
537	"""Attempt to launch the server tray helper with the provided arguments."""
538
539	if _launch_server_tray_binary(argv):	1✔
540	return True	×
541
542	try:	1✔
543	tray_module = import_module(".server_tray", __package__)	1✔
544	except ImportError:	×
545	return False	×
546
547	tray_main = getattr(tray_module, "main", None)	1✔
548	if tray_main is None:	1✔
549	return False	×
550
551	tray_main(list(argv))	1✔
552	return True	1✔
553
554
555	def main(argv: Optional[Sequence[str]] = None) -> None:	1✔
556	"""Entry point for the command line interface.
557
558	Launch the GUI when run without arguments, otherwise defer to the CLI.
559	"""
560
561	if argv is None:	1✔
562	argv_list = sys.argv[1:]	×
563	else:
564	argv_list = list(argv)	1✔
565
566	if "--server" in argv_list:	1✔
567	index = argv_list.index("--server")	1✔
568	tray_args = argv_list[index + 1 :]	1✔
569	if not _launch_server_tray(tray_args):	1✔
570	print("Server tray mode is unavailable.", file=sys.stderr)	1✔
571	sys.exit(1)	1✔
572	return	1✔
573
574	if argv_list and argv_list[0] in {"server", "serve"}:	1✔
575	if not _launch_server(argv_list[1:]):	1✔
576	print("Gradio server mode is unavailable.", file=sys.stderr)	1✔
577	sys.exit(1)	1✔
578	return	1✔
579
580	if not argv_list:	1✔
581	if _launch_gui(argv_list):	1✔
582	return	1✔
583
584	parser = _build_parser()	×
585	parser.print_help()	×
586	return	×
587
588	parser = _build_parser()	1✔
589	parsed_args = parser.parse_args(argv_list)	1✔
590
591	host_value = getattr(parsed_args, "host", None)	1✔
592	if host_value:	1✔
593	parsed_args.server_url = f"http://{host_value}:9005"	×
594
595	send_video = None	1✔
596	remote_error_message: Optional[str] = None	1✔
597	try: # pragma: no cover - optional dependency guard
598	from . import service_client
599	except ImportError as exc:	×
600	remote_error_message = (	×
601	"Server mode requires the gradio_client dependency. " f"({exc})"
602	)
603	else:
604	send_video = service_client.send_video	1✔
605
606	application = CliApplication(	1✔
607	gather_files=gather_input_files,
608	send_video=send_video,
609	speed_up=speed_up_video,
610	reporter_factory=TqdmProgressReporter,
611	remote_error_message=remote_error_message,
612	)
613
614	exit_code, error_messages = application.run(parsed_args)	1✔
615	for message in error_messages:	1✔
616	print(message, file=sys.stderr)	×
617	if exit_code:	1✔
618	sys.exit(exit_code)	×
619
620
621	if __name__ == "__main__":	1✔
622	main()	×

popstas / talks-reducer / 18652090836

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous