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

Phase 1 checklist #686

Closed
3 of 7 tasks
vbuterin opened this issue Feb 25, 2019 · 2 comments
Closed
3 of 7 tasks

Phase 1 checklist #686

vbuterin opened this issue Feb 25, 2019 · 2 comments
Labels

Comments

@vbuterin
Copy link
Contributor

vbuterin commented Feb 25, 2019

  • Determine whether substantial conceptual simplifications to the proof of custody game are possible; if so implement them, if not, conclusively decide that it's fine as is.
  • Grammar fixes and edits
  • Naming changes
  • Economic review, particularly around sizes and structure of rewards and penalties
  • Evaluate possible change to branch challenges: a MultiBranchChallenge where a challenger challenges a particular data root, and all attestation signers are assigned random chunks to respond. This is theoretically much cheaper because it reuses the attestation verification.
  • Evaluate a possible change where each recipient needs to respond with a contiguous array of eg. 16 chunks instead of a single chunk. Currently, we have very low data efficiency for responses because we use a ~500 byte branch to reveal/verify 32 bytes of data, we can extend this to ~900 bytes to reveal/verify 500 bytes of data, a much more favorable tradeoff if the goal is to force revealing the entire block.
  • Replace the current mixing function for custody leaf generation (hash(data, subkey)) with a more MPC-friendly alternative
@hwwhww hwwhww added the phase1 label Feb 25, 2019
@vbuterin
Copy link
Contributor Author

Evaluate possible change to branch challenges: a MultiBranchChallenge where a challenger challenges a particular data root, and all attestation signers are assigned random chunks to respond. This is theoretically much cheaper because it reuses the attestation verification.

FWIW my latest thinking on this is that especially with the 512 byte chunk change and the use of a non-slashable attestation, this is not necessary.

@JustinDrake
Copy link
Contributor

Closing in favour of #864.

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

No branches or pull requests

3 participants