mpv-player / mpv

🎥 Command line video player
https://mpv.io
Other
28.66k stars 2.93k forks source link

options: increase the default --image-display-duration #15291

Closed guidocella closed 1 week ago

guidocella commented 1 week ago

1 second is too short to view images, so increase the default duration to 5 seconds.

This is the value we had a consensus on in IRC and is the default duration e.g. in sxiv.

github-actions[bot] commented 1 week ago

Download the artifacts for this pull request:

Windows * [mpv-i686-w64-mingw32](https://nightly.link/mpv-player/mpv/actions/artifacts/2168432500.zip) * [mpv-x86_64-w64-mingw32](https://nightly.link/mpv-player/mpv/actions/artifacts/2168433205.zip) * [mpv-x86_64-windows-msvc](https://nightly.link/mpv-player/mpv/actions/artifacts/2168439902.zip)
macOS * [mpv-macos-13-intel](https://nightly.link/mpv-player/mpv/actions/artifacts/2168431105.zip) * [mpv-macos-14-arm](https://nightly.link/mpv-player/mpv/actions/artifacts/2168430360.zip) * [mpv-macos-15-arm](https://nightly.link/mpv-player/mpv/actions/artifacts/2168430545.zip)
Dudemanguy commented 1 week ago

Maybe increase --mf-fps too?

guidocella commented 1 week ago

I think mf:// is mostly useful to animate related images, like frames extracted from a video, so maybe it's fine to leave it faster? (Also it would have to be decreased to 0.2 rather than increased.)

Dudemanguy commented 1 week ago

Ah it can be skipped then. I wasn't sure if the same reasoning applied or not.