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

feat: Goblin Honk Composer/Prover/Verifier #1220

Merged
merged 7 commits into from
Jul 27, 2023
Merged

Conversation

ledwards2225
Copy link
Contributor

Description

The Composer, Prover and Verifier for Goblin Ultra Honk. These are implemented as instantiations of the existing "Ultra" template classes, e.g. GoblinUltraComposer = UltraComposer_<flavor::GoblinUltra>, etc.

This work makes it possible to create and verify a Goblin Ultra Honk proof. "Verification" here means confirmation that the ECC op gates have been correctly incorporated into the circuit (checked via a new relation) plus genuine verification of the conventional UltraHonk proof. The correctness of the the ECC op gates is of course not verified, as that is the role of the other components of the Goblin stack.

The new "op gate consistency" relation simply checks that the op wire values have been correctly copied into the conventional wires (needed for copy constraints) and that the op wire polynomials vanish everywhere outside the range of ECC op gates.

Checklist:

  • I have reviewed my diff in github, line by line.
  • Every change is related to the PR description.
  • I have linked this pull request to the issue(s) that it resolves.
  • There are no unexpected formatting changes, superfluous debug logs, or commented-out code.
  • The branch has been merged or rebased against the head of its merge target.
  • I'm happy for the PR to be merged at the reviewer's next convenience.

@netlify
Copy link

netlify bot commented Jul 27, 2023

Deploy Preview for preeminent-bienenstitch-606ad0 canceled.

Name Link
🔨 Latest commit 61e054b
🔍 Latest deploy log https://app.netlify.com/sites/preeminent-bienenstitch-606ad0/deploys/64c2765613bdd900088e14d8

@ledwards2225 ledwards2225 changed the title Lde/goblin composer feat: Goblin Honk Composer/Prover/Verifier Jul 27, 2023
@ledwards2225 ledwards2225 requested a review from Rumata888 July 27, 2023 00:54
@ledwards2225 ledwards2225 merged commit 6743ec1 into master Jul 27, 2023
@ledwards2225 ledwards2225 deleted the lde/goblin_composer branch July 27, 2023 14:49
@ledwards2225 ledwards2225 self-assigned this Jul 27, 2023
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.

3 participants