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

Adds FFmpegFilterFrame sample for Android #550

Merged
merged 2 commits into from
Nov 7, 2016

Conversation

pachev
Copy link
Contributor

@pachev pachev commented Nov 3, 2016

No description provided.

@saudet
Copy link
Member

saudet commented Nov 4, 2016

Great! Though since this is 99% identical to the original RecordActivity sample, could you instead simply modify that sample? Thanks!

@pachev
Copy link
Contributor Author

pachev commented Nov 4, 2016

Ok, will do. I didn't want to edit your original code in case someone didn't want a filter. I'll add a control structure for that. I'll send a pull request in the morning.

@saudet
Copy link
Member

saudet commented Nov 4, 2016

Awesome! You can also simply update the branch of this pull request

@pachev pachev force-pushed the sample/rotatevideo branch 2 times, most recently from 00db458 to 50981ef Compare November 4, 2016 15:29
@pachev
Copy link
Contributor Author

pachev commented Nov 4, 2016

Ok, I added it to the original RecordActivity sample. Have a good day 😄 !

@saudet
Copy link
Member

saudet commented Nov 5, 2016

Looks great! Although it would probably be more efficient to create and start the FFmpegFrameFilter only once, alongside FFmpegFrameRecorder, or is this not working because of some bug in JavaCV?

@pachev
Copy link
Contributor Author

pachev commented Nov 5, 2016

Yes, you're right. It's better to start it once when the recorder starts. The code has been updated. Good catch! I've also tested it on a Nexus 5x but with an RTMP feed instead of saving it to the device.

@saudet
Copy link
Member

saudet commented Nov 7, 2016

Looks good! Thanks a lot

@saudet saudet merged commit 5cea7c6 into bytedeco:master Nov 7, 2016
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