Nj0be / HandBrake-SVT-AV1-PSY

41 stars 2 forks source link

Certain presets missing: 6, 12 #1

Closed DesertCookie closed 1 month ago

DesertCookie commented 1 month ago

These two presets are missing and can't be selected. I'm only able to jump from 13 to 11 and from 7 to 5.

Unbenannt

Nj0be commented 1 month ago

This behavior is intentional, as it is also present in the original HandBrake implementation. You can easily see that from the patch responsible for adding presets. You can see that I only add presets -2 and -3. The reason for this is that presets 13-12 and 7-6 are identical, making it unnecessary to include both. You can see that on the official SVT-AV1 presets documentation. Thanks for reporting though. If you found any other problem, feel free to open another issue.

DesertCookie commented 1 month ago

Interesting. For the past two years, Handbrake's AV1 encoder allowed these options. I've been encoding with speed 6 for a while now (well, according to the GUI).

Perhaps you could add a disclaimer to the tooltip so people don't wonder about it.

Nj0be commented 1 month ago

Interesting. For the past two years, Handbrake's AV1 encoder allowed these options. I've been encoding with speed 6 for a while now (well, according to the GUI).

Perhaps you could add a disclaimer to the tooltip so people don't wonder about it.

The presets 12 and 6 were removed 4 months ago with this commit. You were probably using an older version of HandBrake. If I install the latest stable version from github, I can't use either presets 12 or 6. Anyway, preset 7 should be exactly the same as preset 6. You can try both, and you should get the same encoding time. At the moment, I don't have much time to write a good and detailed tooltip, but when I do, I'll definitely add the disclaimer to it. Thanks again