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

Reconcile PlonK and PlonkUp paper with the implementation #631

Closed
moCello opened this issue Nov 23, 2021 · 1 comment · Fixed by #649
Closed

Reconcile PlonK and PlonkUp paper with the implementation #631

moCello opened this issue Nov 23, 2021 · 1 comment · Fixed by #649
Assignees
Labels
area:documentation Anything to improve our doc base team:Core Low Level Core Development Team (Rust) type:docs deliverable should be documentation

Comments

@moCello
Copy link
Member

moCello commented Nov 23, 2021

Describe what kind of specification you want to have created

  • Add comments to the code-base:
    • make sure that every step of the protocol can be found in the code implementation when searched
    • mark the sections where the implementation differs from the paper and explain why and how they are different
@moCello moCello added area:documentation Anything to improve our doc base type:docs deliverable should be documentation team:Core Low Level Core Development Team (Rust) labels Nov 23, 2021
@moCello
Copy link
Member Author

moCello commented Nov 29, 2021

Closing this issue because it has been broken into the following smaller issues to better keep track of the progress being made:
Setup: #633
Proof: #643, #644, #645, #646, #647, #648
Verification: #639, #640, #641, #642

Edit:
Reopening because braking up showed not to be feasible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:documentation Anything to improve our doc base team:Core Low Level Core Development Team (Rust) type:docs deliverable should be documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants