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

Builder simulator for "native" verification #673

Closed
ledwards2225 opened this issue Aug 16, 2023 · 1 comment
Closed

Builder simulator for "native" verification #673

ledwards2225 opened this issue Aug 16, 2023 · 1 comment

Comments

@ledwards2225
Copy link
Collaborator

For Honk, we'd like to have a single Verifier class for both recursive and "native" verification. Since the recursive verifier is the first class object, the logic will be set up to work for stdlib types and we will achieve native verification via swapping a genuine builder (e.g. UltraCircuitBuilder) for a "simulated" builder, a la Cody's work. Once this is done, verifier classes like SumcheckVerifier, GeminiVerifier, ShplonkVerifier and UltraVerifier will only every be instantiated with stdlib types. This will eliminate the need for the constexpr branching (e.g. on IsRecursiveFlavor or Curve::is_stdlib_type) in these classes which currently facilitates supporting both stdlib and native logic. Instead, these classes will be templated over the appropriate recursive concept, e.g. RecursiveFlavor.

@codygunton
Copy link
Collaborator

The CircuitSimulator was scrapped as being lower priority now that the rollup logic will be written in Noir. Closing this, at least for now.

@codygunton codygunton closed this as not planned Won't fix, can't repro, duplicate, stale Oct 24, 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

No branches or pull requests

2 participants