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

[FlexFec] Adding FlexFec Flex variant encoder implementation #212

Merged
merged 1 commit into from
Oct 24, 2023

Conversation

pougetat
Copy link
Contributor

Description

  • Context:

FlexFec is a packet loss recovery technique through which redundant RTP packets are sent and from which missing RTP packets can potentially be reconstructed.
https://datatracker.ietf.org/doc/html/rfc8627

  • Summary:

There are multiple variants of FlexFec and I'm adding an implementation of the "Flex" encoder as part of this PR.
The other variants are not included nor is the decoder included which I hope to add in subsequent PRs, this first one is meant to be the start of a series.
This PR also doesn't add the interceptor on top of the encoder.

I'm labeling it as DRAFT for now until I write some proper unit tests to get us some confidence that it works as expected.

Reference issue

Fixes #...

@codecov
Copy link

codecov bot commented Oct 24, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

see 2 files with indirect coverage changes

📢 Thoughts on this report? Let us know!.

@Sean-Der
Copy link
Member

Hey @pougetat it is great to see you :)

If you can fix those lint errors I would like to merge this. We can add tests/improve other things later.

@pougetat pougetat changed the title [DRAFT] Adding FlexFec encoder implementation [FlexFec] Adding FlexFec encoder implementation Oct 24, 2023
@pougetat pougetat changed the title [FlexFec] Adding FlexFec encoder implementation [FlexFec] Adding FlexFec Flex variant encoder implementation Oct 24, 2023
@pougetat pougetat force-pushed the pion_flexfec_encode branch from 6b5244a to c74445c Compare October 24, 2023 18:24
@pougetat pougetat merged commit 0440a7d into master Oct 24, 2023
16 checks passed
@pougetat pougetat deleted the pion_flexfec_encode branch October 24, 2023 18:26
Marker: false,
PayloadType: flex.payloadType,
SequenceNumber: flex.baseSN,
Timestamp: 54243243,
Copy link

Choose a reason for hiding this comment

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

Hi @pougetat. Could you please explain this fixed Timestamp value = 54243243? Is it taken from RFC?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's a leftover hardcoded value. I'll get this fixed soon.

Copy link

Choose a reason for hiding this comment

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

Okay. No problem 👍 Thanks for the explanation)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants