-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Added option to export with alpha channel #679
Conversation
There is a syntax error in your commit. Did you try it on your machine ? |
Sorry about this. |
thanks for the pull request. Can you replace "withmask" by "with_mask" ? Also, do you have by chance some (non-copyrighted) bit of masked video (just a few seconds) that we could try this on (and write a test for the test suite) ? Ideally the story of a test would be (1) load the video in moviepy with VideoFileClip, (2) write the video to another file, with the mask, (3) load the video from the new file and check that it still has a mask. |
2 similar comments
Hey @jonata , thanks for your PR, using it locally I was able to export a clip with 4444 for integrating machine learning in python into a post workflow. It would be great to get this merged into master… need someone to clean it up? |
Hi @darknoon , sorry my lack of experience, what I am supposed to do now? |
Linking #1122. |
This has been added in #1237. |
This add the option to export with alpha channel, may solve #559