Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

chore!: Remove create_proof and verify functions #82

Merged
merged 3 commits into from
Mar 13, 2023

Conversation

phated
Copy link
Contributor

@phated phated commented Mar 13, 2023

This removes the create_proof and verify functions and reimplements the acvm_interop functions with @vezenovm's pk/vk functions.

I believe we also need to deprecate the acvm_interop stuff and eventually remove it from ACVM and here.

chore: Reimplement acvm_interop with pk/vk functions
Copy link
Member

@TomAFrench TomAFrench left a comment

Choose a reason for hiding this comment

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

We have a similar implementation in barretenberg_wasm. Do we not want to remove this at the same time?

@phated phated requested a review from TomAFrench March 13, 2023 19:33
Copy link
Member

@TomAFrench TomAFrench left a comment

Choose a reason for hiding this comment

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

LGTM

@TomAFrench TomAFrench merged commit ad0c422 into master Mar 13, 2023
@TomAFrench TomAFrench deleted the phated/remove-proof-verify branch March 13, 2023 20:24
@github-actions github-actions bot mentioned this pull request May 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants