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