-
Notifications
You must be signed in to change notification settings - Fork 1
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 hacky NALU completion handling #39
Conversation
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 believe we can leave it that way, as after some inspection I've came to the conclusion that it shouldn't break the existing cases.
Please address the Mateusz's comment and optionally bump the version, if the release is needed now.
# | ||
# It happens that some streams have broken frame numbers (that are either non-monotically | ||
# increasing or just reset on a key frame) but the `first_mb_in_slice` set to zero can mean that | ||
# we are dealin with a new AU (given a proper `nal_unit_type`). It seems that it is sufficient |
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.
# we are dealin with a new AU (given a proper `nal_unit_type`). It seems that it is sufficient | |
# we are dealing with a new AU (given a proper `nal_unit_type`). It seems that it is sufficient |
# | ||
# NOTE: The following check is not a part of the original H264 specification unlike the other checks below. | ||
# | ||
# It happens that some streams have broken frame numbers (that are either non-monotically |
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.
# It happens that some streams have broken frame numbers (that are either non-monotically | |
# It happens that some streams have broken frame numbers (that are either non-monotonically |
This PR adds support for AU completion detection that works in mysterious ways (taken from GStreamer).
https://github.com/GStreamer/gst-plugins-bad/blob/ca8068c6d793d7aaa6f2e2cc6324fdedfe2f33fa/gst/videoparsers/gsth264parse.c#L1183C45-L1185C49