-
Notifications
You must be signed in to change notification settings - Fork 416
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
Implement an FfmpegVideoRenderer #1591
base: main
Are you sure you want to change the base?
Conversation
Is there any progress? @microkatz |
@rabbitknight Could you provide me with aar for testing? I don’t have a compilation environment. |
Use the provided link. https://drive.google.com/file/d/1gIZYH6dR780BfPqll1w1VAMaaSkcLqRM/view?usp=drive_link |
@rabbitknight thx |
There is no problem in my test, unlike nextlib which needs to reset the surface. |
Thanks for your feedback. There may be slight differences, but overall the code is similar. I will keep watching until successfully merged. :P |
I finally found the bug video_2024-08-17_00-07-21.mp4 |
Thanks~ let me see what happened. I'll feedback on Monday. |
@FongMi |
Thx, I will test it. |
@rabbitknight Thers is no problem now, thanks. |
@rabbitknight Can you support mepg and mpeg2 like nextlib |
@FongMi you can get the latest aar from: https://drive.google.com/drive/folders/1pwOkFUzMwpfm492IFrUebkn7MUGBA2My?usp=drive_link |
Is your audio codecs as good as nextlib? |
It worked fine for testing, thank you. |
@FongMi I will compare the |
@FongMi, I conducted a comparison:
|
@rabbitknight Thanks, I understand. |
It has been over a month. Any progress so far? @microkatz |
Attention! ExoPlayer can now decode Apple ProRes videos using ffmpeg.(Apple ProRes MAX is still not supported). |
Merge actual implementation in google/ExoPlayer#7132.