-
Notifications
You must be signed in to change notification settings - Fork 79
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
docs: clarify rowRoots to Repair #240
Conversation
Codecov Report
@@ Coverage Diff @@
## main #240 +/- ##
=======================================
Coverage 77.96% 77.96%
=======================================
Files 7 7
Lines 531 531
=======================================
Hits 414 414
Misses 71 71
Partials 46 46
|
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.
👍
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.
LGTM
// Repair attempts to repair an incomplete extended data square (EDS). The | ||
// parameters rowRoots and colRoots are the expected Merkle roots for each row | ||
// and column. rowRoots and colRoots are used to verify that a repaired row or | ||
// column is correct. Prior to the repair process, if a row or column is already |
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.
Q: you can technically repair an EDS without the rowRoots and colRoots right?
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.
Yes it is possible. I'm pretty confident it is because the underlying library (klauspost/reedsolomon) doesn't have a notion of verifying the repaired data against a row root or col root.
Closes #220