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

Allow FFMPEG command to be overridden #1320

Merged
merged 11 commits into from Sep 19, 2022
Merged

Allow FFMPEG command to be overridden #1320

merged 11 commits into from Sep 19, 2022

Conversation

gexin1
Copy link
Contributor

@gexin1 gexin1 commented Sep 16, 2022

Add the ffmpegArgsHook parameter in renderMedia. You can control the final running parameters of ffmpeg yourself

2.  ffmpegArgsHook adds parameter validation
3.  Add FfmpegArgsHook type
@vercel
Copy link

vercel bot commented Sep 16, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
remotion ✅ Ready (Inspect) Visit Preview Sep 19, 2022 at 2:17PM (UTC)

@JonnyBurger
Copy link
Member

Thanks a lot! Since this is a powerful feature it also is a footgun so I want to put in some protections and warning labels.
I'll review it next week!

@JonnyBurger JonnyBurger changed the title Add the ffmpegArgsHook parameter in renderMedia. You can control the final running parameters of ffmpeg yourself Allow FFMPEG command to be overridden Sep 19, 2022
Copy link
Member

@JonnyBurger JonnyBurger left a comment

Choose a reason for hiding this comment

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

Thanks a lot! Also added it to prestitcher, enabled it from the config file as well and added some more docs - and warnings 😄 Good to ship for me!

@gexin1
Copy link
Contributor Author

gexin1 commented Sep 19, 2022

Thank you for your great open source project👍

@JonnyBurger JonnyBurger merged commit 0822f49 into remotion-dev:main Sep 19, 2022
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.

None yet

2 participants