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

Allow control over how the encoding matrix is built #110

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hivenet-mathieu-lacage
Copy link

@hivenet-mathieu-lacage hivenet-mathieu-lacage commented Jun 18, 2024

Allow users to pick an encoding matrix that is compatible with the klauspost/reedsolomon library.

The zfec library uses a vector of alpha^i to create its encoding matrix from the vandermonde matrix. From the original paper: https://www.cs.utexas.edu/users/lam/395t/papers/FEC-rizzo.pdf

Screenshot 2024-06-18 at 06-38-02 FEC-rizzo pdf

Some libraries use another common convention where the encoding matrix is built from the vandermonde matrix from a vector of sequential integers (0, 1, 2, 3, ...). The objective of this patch is to allow zfec to generate and consume data using this convention.

@hivenet-mathieu-lacage
Copy link
Author

Hi @exarkun , I see you have merged a couple of PRs over the project's recent history. Do you have any interest in this PR ? I do not mind taking time to rework this if the project is interested but if you are not, it would be nice to know :)

@exarkun
Copy link
Member

exarkun commented Jul 14, 2024

Hi @exarkun , I see you have merged a couple of PRs over the project's recent history. Do you have any interest in this PR ? I do not mind taking time to rework this if the project is interested but if you are not, it would be nice to know :)

I'm not as active on this project any more but I may be able to take a look sometime (esp. after I return from a trip I'm currently on).

@hivenet-mathieu-lacage
Copy link
Author

@exarkun if you are not able to have a look this, can you suggest anyone who might care enough to tell me if this makes any sense for you or if I need to move on and find an alternate solution ?

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.

2 participants