-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix nvenc preset order #9538
Fix nvenc preset order #9538
Conversation
@TheTyrius Can you rebase to the release-10.8.z branch? |
Will do. Should I generally target release branches (for future PRs)? I skimmed the contribution docs, and they told me to target master :) |
Bug fixes -> Release branches (we will backport it to the master branch later) |
13a686f
to
6f29d04
Compare
6f29d04
to
bf37db7
Compare
Looks fine now (and learnt about |
Fix nvenc preset order Original-merge: 79bb756 Merged-by: Cody Robibero <cody@robibe.ro> Backported-by: Bond_009 <bond.009@outlook.com>
Changes
Changed the ordering of the "slow" and "slower" presets in the transcoding helper, so they are aligned with the order in the settings dropdown.
Did the same to a switch-case where there is no change in behavior right now (but might prevent a bug like this in the future).
Issues
First raised in #9537 thanks nyanmisaka for giving it a look so incredibly quickly (and also for all the other work you do on Jellyfin)