filters: add ewa_lanczosradius and missing polar bc-splines - #308
Open
Artoriuz wants to merge 2 commits into
Open
filters: add ewa_lanczosradius and missing polar bc-splines#308Artoriuz wants to merge 2 commits into
Artoriuz wants to merge 2 commits into
Conversation
Collaborator
|
@haasn bump? |
Collaborator
|
This needs API version bump |
llyyr
reviewed
Jan 19, 2026
| }; | ||
|
|
||
| const struct pl_filter_config pl_filter_ewa_lanczosradius = { | ||
| .name = "ewa_lanczosradius", |
Collaborator
There was a problem hiding this comment.
"ewa_lanczosradius3" or "ewa_lanczos3radius", perhaps?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ewa_lanczosradius is an interesting filter because it produces results that are roughly as sharp as those of ortho lanczos, while being slightly faster than ewa_lanczossharp since it has a shorter radius. The naming convention comes from ImageMagick and I believe it's supposed to indicate that it has an integer radius (3 in this case).
The missing named BC-Splines were added as well to make them more convenient to access. I don't think they're that useful in general, but having them defined probably doesn't hurt.
I wanted to open this on the actual repo instead, but my VideoLAN account is stuck in approval limbo. If you think this is an unnecessary addition please feel free to close. The functionality already exists, these changes only make these filters easier to use.