-
Notifications
You must be signed in to change notification settings - Fork 13
Add Duck IVF parsing support to the elementary stream. #18
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
base: main
Are you sure you want to change the base?
Conversation
zlatinski
left a comment
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.
LGTM Thank you!
| #define DKIF_FRAME_CONTAINER_HEADER_SIZE 12 | ||
| #define DKIF_HEADER_MAGIC *((const uint32_t*)"DKIF") | ||
| #define DKIF_FILE_HEADER_SIZE 32 | ||
| #define USE_SIMPLE_MALLOC 1 |
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.
I'm a bit confused about this USE_SIMPLE_MALLOC. Can you add a comment to explain the use of that ?
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.
I'll remove it the define. It's there for debugging purposes.
|
We should remove the USE_SIMPLE_MALLOC from this change. @vasn1k, per our offline discussion, please do that before we merge this change. |
Signed-off-by: Vassili Nikolaev (NVIDIA) <vnikolaev@nvidia.com>
|
@vasn1k could you make the change needed for this PR ? |
Add support for .ivf files when loading AV1 compressed video files and using elementary streams.