Skip to content
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

Add new unutilized interpolation filter search levels to slower presets and better CLI docs #39

Closed
wants to merge 2 commits into from

Conversation

BlueSwordM
Copy link
Collaborator

@BlueSwordM BlueSwordM commented Apr 6, 2024

Title is the main subject.

This time, I added speed features which are currently not utilized by svt-av1 and I decided to make them available to svt-av1-psy. More testing is needed until it's merged into the main branch.

The main goal is to increase quality with better AV1 interpolation filter search choices like Regular, Sharp and Smooth.

Currently, pruning with this encoding feature is very crude.
It either fully enables it in the research presets or greatly prunes the search for any realistic encoding preset.

I've done some testing on my side and it gives a non-negligible boost to fidelity at higher bitrates and a small boost to quality at lower presets.

Be sure to test this patch before suggesting any preset changes.

I've also added some new docs to fix the CLI qp-scale-compress option description :)

@BlueSwordM BlueSwordM changed the title Add new unutilized interpolation filter search levels to slower presets Add new unutilized interpolation filter search levels to slower presets and better CLI docs Apr 6, 2024
@gianni-rosato
Copy link
Owner

After testing this, there appears to be almost zero perceptible change with an added speed penalty. Additionally, we are not interested in qualifying mainline defaults in our encoder. Unfortunately this does not seem reasonable to merge, my apologies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants