[Issue]: nvenc slow/slower preset order #4733

Closed
opened 2026-02-07 01:06:49 +03:00 by OVERLORD · 2 comments
Owner

Originally created by @TheTyrius on GitHub (Mar 25, 2023).

Please describe your bug

While browsing the transcoding code I noticed that for nvenc the "slow" setting uses a higher preset than the "slower" one. This seemed unintuitive to me, shouldn't it be the other way around? See 9f7418cf42/MediaBrowser.Controller/MediaEncoding/EncodingHelper.cs (L1554-L1564)
-> slow uses p6, slower p5, but veryslow is p7

Jellyfin Version

10.8.0

if other:

No response

Environment

- OS:
- Virtualization:
- Clients:
- Browser:
- FFmpeg Version:
- Playback Method:
- Hardware Acceleration:
- Plugins:
- Reverse Proxy:
- Base URL:
- Networking:
- Storage:

Jellyfin logs

No response

FFmpeg logs

No response

Please attach any browser or client logs here

No response

Please attach any screenshots here

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
Originally created by @TheTyrius on GitHub (Mar 25, 2023). ### Please describe your bug While browsing the transcoding code I noticed that for nvenc the "slow" setting uses a higher preset than the "slower" one. This seemed unintuitive to me, shouldn't it be the other way around? See https://github.com/jellyfin/jellyfin/blob/9f7418cf425ac295967ec5b826d992f69fa2fa10/MediaBrowser.Controller/MediaEncoding/EncodingHelper.cs#L1554-L1564 -> slow uses p6, slower p5, but veryslow is p7 ### Jellyfin Version 10.8.0 ### if other: _No response_ ### Environment ```markdown - OS: - Virtualization: - Clients: - Browser: - FFmpeg Version: - Playback Method: - Hardware Acceleration: - Plugins: - Reverse Proxy: - Base URL: - Networking: - Storage: ``` ### Jellyfin logs _No response_ ### FFmpeg logs _No response_ ### Please attach any browser or client logs here _No response_ ### Please attach any screenshots here _No response_ ### Code of Conduct - [X] I agree to follow this project's Code of Conduct
OVERLORD added the bug label 2026-02-07 01:06:49 +03:00
Author
Owner

@nyanmisaka commented on GitHub (Mar 25, 2023):

It seems the slow and slower cases are inverted. Feel free to open a PR to fix it.

@nyanmisaka commented on GitHub (Mar 25, 2023): It seems the `slow` and `slower` cases are inverted. Feel free to open a PR to fix it.
Author
Owner

@TheTyrius commented on GitHub (Mar 25, 2023):

Closed by https://github.com/jellyfin/jellyfin/pull/9538

@TheTyrius commented on GitHub (Mar 25, 2023): Closed by https://github.com/jellyfin/jellyfin/pull/9538
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#4733