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 support for remote stream's decoder configuration parsing #24

Merged
merged 3 commits into from
Jun 1, 2023

Conversation

Qizot
Copy link
Member

@Qizot Qizot commented May 24, 2023

This PR adds the support of parsing of the decoder configuration record that arrives in stream format, just like in membrane_h264_ffmpeg_plugin package.

On DCR arrival the parser will try to parse it to extract PPS and SPS values that will be prepended to the following buffer received by the parser.

@Qizot Qizot requested a review from varsill May 24, 2023 13:09
@Qizot Qizot requested a review from DominikWolek as a code owner May 24, 2023 13:09
lib/membrane_h264_plugin/parser.ex Outdated Show resolved Hide resolved
lib/membrane_h264_plugin/parser.ex Outdated Show resolved Hide resolved
Copy link
Contributor

@varsill varsill left a comment

Choose a reason for hiding this comment

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

Just

lib/membrane_h264_plugin/parser.ex Outdated Show resolved Hide resolved
@Qizot Qizot force-pushed the decoder-configuration-record-parsing branch from 5b908c2 to 6014df0 Compare May 31, 2023 09:01
@Qizot Qizot requested a review from varsill May 31, 2023 09:01
@Qizot Qizot merged commit 13384f4 into master Jun 1, 2023
@Qizot Qizot deleted the decoder-configuration-record-parsing branch June 1, 2023 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants