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

Update the solidity verifier to take in the proof (with pairing points obj) and inner public inputs instead of proof (without pairing points obj) and public inputs #1314

Open
lucasxia01 opened this issue Mar 28, 2025 · 0 comments
Assignees

Comments

@lucasxia01
Copy link
Contributor

The honk solidity verifier currently takes in the public inputs and proof, where the public inputs contain ALL of the public inputs including the pairing point accumulator. This clashes with other public inputs/proof splits, where the pairing point accumulator is stored with the proof, even though it technically is part of the public inputs.

We should pick one. I think the pairing point object should just be hidden from the users and stored with the proof.

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

1 participant