-
Notifications
You must be signed in to change notification settings - Fork 270
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
Add additional VVC program suits into autobuild media #2213
Conversation
Not tested yet - Martin Eesmaa
vvenc is now tested and success, but a little bug. It's easy to fix the bug, no problem at all:
|
vvdec is now tested and success without issues:
|
VVEnc is installation target for vvencapp.exe and vvencFFapp.exe only. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please also add these entries inside the README.md near uvg266 for now
Co-authored-by: Christopher Degawa <[email protected]>
Co-authored-by: Christopher Degawa <[email protected]>
Co-authored-by: Christopher Degawa <[email protected]>
Co-authored-by: Christopher Degawa <[email protected]>
Approved commit by Martin Eesmaa <[email protected]> Co-authored-by: Christopher Degawa <[email protected]>
Approved commit by Martin Eesmaa <[email protected]> Co-authored-by: Christopher Degawa <[email protected]>
Approved commit by Martin Eesmaa <[email protected]> Co-authored-by: Christopher Degawa <[email protected]>
Approved commit by Martin Eesmaa <[email protected]> Co-authored-by: Christopher Degawa <[email protected]>
Let me check, git pull and see how works on my virtual machine. I'll test it out... |
Signed by Martin Eesmaa <[email protected]>
Report commit 25b6f2e log:
|
lgtm, do you want me to cleanup the commits, rebase and leave them as is, or squash them into just one commit? |
@1480c1, might be just leave this commits unless I tell it's ready to merge. I tried compile of my forked repository + vvc commits, but it did not work, so tbiat/FFmpeg works only modified version without latest commits of 2022.
|
Seems it may need a few changes for the ffmpeg plugin, I can try to look into that once I have more time, but I would suggest first going through with adding vvenc and vvdec as executables first, so more people can test it and see if there are any other issues |
Added vvenc & vvdec in README. Signed by <[email protected]>
@1480c1, I think we can let anyone try build vvenc & vvdec after you merge my commits into this repository, so I'm ready to merge my commits by you. Sounds good idea!
|
Sounds good, but do you want me to cleanup the commits, rebase and leave them as is, or squash them into just one commit? |
@1480c1, it might be best to squash. also make sure my commits and my profile were seen by anyone on your repository :)
|
@MartinEesmaa Thanks for the work! |
No problem, @1480c1! :)
|
See issue: #1810
Added vvenc & vvdec, but not tested yet, others vvc programs are coming soon...
Update of 16th June 2022: vvenc & vvdec is only now support of media autobuild suite, others were not.
vvenc & vvdec
Bitmovin VVDec Player
FFmpeg (libvvdec plugin)
VLC InterDigital Plugin?
Martin Eesmaa