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: implement Plonk Bn254 proof verification for Solidity format #171

Closed
wants to merge 2 commits into from

Conversation

huitseeker
Copy link
Contributor

@huitseeker huitseeker commented Sep 11, 2024

  • Introduced a new VerifySolidity function in verify.go for proof verification with Solidity, complete with unmarshalling of solidity proof, public witness creation, and proof verification
  • Created a new function verify_plonk_bn254_solidity in native.rs to handle plonk bn254 verification for solidity
  • Incorporated a new function VerifyPlonkBn254Solidity in main.go for Plonk Bn254 proofs verification for Solidity

This adds a bit of go code to deserialize encoded proofs, see argumentcomputer/lurk-hs#3
Leaving draft to allow flexibility in sequencing w.r.t. #164 (and because I have to resolve my feelings on whether we want to actually merge this).

wwared and others added 2 commits September 10, 2024 22:56
- Introduced a new `VerifySolidity` function in `verify.go` for proof verification with Solidity, complete with unmarshalling of solidity proof, public witness creation, and proof verification
- Created a new function `verify_plonk_bn254_solidity` in `native.rs` to handle plonk bn254 verification for solidity
- Incorporated a new function `VerifyPlonkBn254Solidity` in `main.go` for Plonk Bn254 proofs verification for Solidity
@huitseeker huitseeker force-pushed the with_encoded_proof_verification branch from bfaf106 to da279e2 Compare September 11, 2024 02:56
@huitseeker
Copy link
Contributor Author

We solved our interop issues by switching Kadena to raw proof.

@huitseeker huitseeker closed this Sep 24, 2024
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