-
Notifications
You must be signed in to change notification settings - Fork 163
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 additional flags, options #671
Merged
Merged
Conversation
This file contains 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
I've rebased this on top of master here: https://github.com/mqudsi/Av1an/tree/pr671 |
Acts as the opposite of the overwrite -y flag
This allows the user to control which scaling algorithm is used during scene detection downscaling and VMAF calculations, and also allows the width of the lanczos scaler to be selected from 1 to 9
This allows the user to have VMAF calculations use the input resolution automatically, without having to type it in manually per file
This looks smoother on the default Windows command prompt
Allows the user to specify extra splits in seconds. If both frames and seconds are specified, frames will take priority
Allows the user to ignore any reported frame mismatches between the encoder and chunk frame counts, which is useful if an ffmpeg filter changes the frame count, or the input video is encoded badly (ex. "Missing key frame while searching for timestamp" ffmpeg warning)
Rebased again (thanks @mqudsi) with some misc. minor changes, a typo correction, and added documentation to CLI.md |
6f0fd95
to
5543ceb
Compare
oops
shssoichiro
approved these changes
Jun 25, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This commit adds:
-n
flag, acts as the opposite of the overwrite-y
flag--scaler
flag, allows the user to choose any scaler available within ffmpeg, including defining the width of the lanczos scaler by typing lanczos1 through lanczos9 (could go higher but there's not really a point to have width values above 6)--extra-splits-sec
flag, allows the user to define the maximum scene length in seconds (frames takes precedent if both are defined because the clap crate can't make these options conflict with each other, for some reason)--ignore-frame-mismatch
flag, allows the user to ignore any frame mismatch warning in case an ffmpeg filter changes the frame count, or if the input file is badly encoded (acts as a workaround for Unable to change video fps #650)--vmaf-res
, sets the VMAF resolution to be the same as the input video's resolution if--vmaf-res inputres
is used