Skip to content

docs: -d=n is not working #493

Closed
Closed

Description

@gphg

Description:

As written on docs:

Integer factor to downscale video by before processing. If unset, value is selected based on resolution. Set -d=1 to disable downscaling.

It is not working, but

-d 1

Is working.

Command:

--

Output:

--

Environment:

[PySceneDetect] PySceneDetect 0.6.5.2

System Info
------------------------------------------------------------
OS               Linux-5.15.149-android13-8-00844-gbadc131c2419-aarch64-with-libc
Python           CPython 3.12.9
Architecture     64bit + ELF

Packages
------------------------------------------------------------
av               Not Installed
click            8.1.8
cv2              4.11.0
imageio          2.36.1
imageio_ffmpeg   0.5.1
moviepy          2.1.1
numpy            2.2.3
platformdirs     4.3.6
scenedetect      0.6.5.2
tqdm             4.67.1

Tools
------------------------------------------------------------
ffmpeg           7.1
mkvmerge         v89.0 ('And the Melody Still Lingers On (Night in Tunisia)') 64-bit

Media/Files:

--

Activity

  1. Breakthrough commented on Mar 2, 2025

    @Breakthrough
    Owner

    It looks like the = syntax only works for the long form of these arguments, the short form takes every character afterwards literally (e.g. it should be -d1 or -d 1). We should see if there's a workaround for this, such as removing any leading = character (I'm not sure if this would break anything else though).

    For now updating the docs is probably sufficient, after which we can re-purpose this issue for the long term solution.

  2. Breakthrough commented on Mar 2, 2025

    @Breakthrough
    Owner

    This should be fixed now, let me know if you run into any other instances (I could have missed a few). I couldn't find an obvious way of fixing the short-form version, and confusingly, it works with anything that's not a single character (e.g. -fs=1 works but -d=1 does not).

    For now I think we'll just have to leave this as-is, and avoid recommending the = syntax for anything except long-from args. And for consistency with short ones, just use a space.

  3. added this to the 0.6.6 milestone on Mar 2, 2025
  4. gphg commented on Mar 2, 2025

    @gphg
    Author

    I can't confirm if this happens to other sort-form options.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions