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

Add more codec references #918

Merged
merged 6 commits into from
Nov 24, 2024
Merged

Conversation

robUx4
Copy link
Contributor

@robUx4 robUx4 commented Nov 24, 2024

And cleanup some text

@robUx4 robUx4 added codec mapping spec_codecs Codec Matroska spec document target labels Nov 24, 2024
Copy link
Contributor

@JeromeMartinez JeromeMartinez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wouldn't describe UCI and VfW here as they are implementations and not a technical description of the stream based on specs, but not blocking.

@robUx4
Copy link
Contributor Author

robUx4 commented Nov 24, 2024

I wouldn't describe UCI and VfW here as they are implementations and not a technical description of the stream based on specs, but not blocking.

Yes, for now I kept a similar text. I need to dig deeper in the specs to see if there's a proper reference to explain the differences. I don't think we need to care about hacks needed in AVI to store the same codecs.

@robUx4 robUx4 merged commit e3dcf94 into ietf-wg-cellar:master Nov 24, 2024
2 checks passed
@robUx4 robUx4 deleted the more-video-refs branch November 24, 2024 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
codec mapping spec_codecs Codec Matroska spec document target
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants