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

plonk: multiprover: multiprover-kzg: Add batch commit and open methods #5

Merged
merged 1 commit into from
Oct 14, 2023

Conversation

joeykraut
Copy link
Member

Purpose

This PR implements batched analogs of the commit and open methods to provide a convenient PCS interface for many polynomials. Note that prover side the batching is dead simple: just commit individually to each polynomial and open at a different point. This matches the single-prover implementation which only combines the commitments verifier-side

Testing

  • Unit and integration tests pass
  • Tested against the UnivariateKzgPCS::batch_verify method

@joeykraut joeykraut requested a review from akirillo October 14, 2023 02:13
Copy link

@akirillo akirillo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@@ -117,6 +117,7 @@ where
// | Commitment Scheme Implementation |
// ------------------------------------

// TODO: Implement a more efficient batching scheme

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like even Jellyfish hasn't implemented the batched version of their KZG scheme as described in https://eprint.iacr.org/2020/1536, this will be fun to crack into later on, lol

@joeykraut joeykraut merged commit c8cc583 into main Oct 14, 2023
0 of 2 checks passed
@joeykraut joeykraut deleted the joey/batch-multiprover-kzg branch October 21, 2023 17:06
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