Add basic rateless codec support - #102
Conversation
|
I've been looking through the changes you made and overall it looks like it's pretty much on the right track. So - great job on figuring that stuff out as the logic of my code is definitely not obvious with a lot of cruft from it's development history as research code to get the protocol operation sorted out. I do have a couple of initial questions/comments:
|
ae61d13 to
fe49501
Compare
|
Thanks for taking the time to look through this. I will address (1) and (3) in a follow up commit. You are 100% spot on (2) we can leave as a future refinement. For (4), I'm obviously biased towards nanorq, I recently added a lightweight core interface that would work well with norm, but I'm concerned about complicating the build system too much. I planned to take a closer look at integration approaches after we hashed this pr out. |
|
(1) and (3) are addressed. I realized |
Hello,
This PR adds basic support for a rateless codec, working toward the RaptorQ / non-systematic codec support discussed in #7.
What's included
normApi.h/normApi.cppto enable and configure it.examples/normRatelessTest.cppexample demonstrating the feature.Notes
NormTelemetryso rateless encoders could implement their own rate control for both proactive and reactive parity using metrics provided by norm.If the interface looks ok, I can follow this pr with an actual implementation of a rateless codec.