Skip to content

Conversation

Kreyren
Copy link
Contributor

@Kreyren Kreyren commented Dec 30, 2022

No description provided.

Copy link
Contributor

@EmilyGraceSeville7cf EmilyGraceSeville7cf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No mandatory fixes found, approved.
Thanks for a contribution! 😄

Comment on lines +29 to +30
# Change INPUT's FPS to 30 and save as OUTPUT (https://trac.ffmpeg.org/wiki/ChangingFrameRate)
ffmpeg -i input_video.ext -filter:v fps=fps=30 output_video.ext
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# Change INPUT's FPS to 30 and save as OUTPUT (https://trac.ffmpeg.org/wiki/ChangingFrameRate)
ffmpeg -i input_video.ext -filter:v fps=fps=30 output_video.ext
# To change INPUT's FPS to 30 and save as OUTPUT (https://trac.ffmpeg.org/wiki/ChangingFrameRate):
ffmpeg -i <in-video.ext> -filter:v fps=fps=30 <out-video.ext>

Optional fix: To can be used for consistency.
Optional fix: trailing colon can be added.
Optional fix: placeholders for input and output files can be used.
Optional fix: placeholder content can use in- and out- prefixes for input and output files respectively.

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.

2 participants