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.

Sign up for GitHub

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

Jump to bottom

Investigate regression issue with double_verify_proof #1927

Closed
kevaundray opened this issue Sep 1, 2023 · 2 comments · Fixed by #1986
Closed

Investigate regression issue with double_verify_proof #1927

kevaundray opened this issue Sep 1, 2023 · 2 comments · Fixed by #1986
Labels
C-bb.js Component: bb.js - wrapping bberg in js

Comments

@kevaundray
Copy link
Contributor

Problem

double_verify_proof is currently failing to prove and verify on both bb binaries and on the bb.js CLI.

## Solution

Investigate why this is -- currently there are no clues, I'll use this issue to place more information upon discovery
@kevaundray kevaundray added the C-bb.js Component: bb.js - wrapping bberg in js label Sep 1, 2023
@github-project-automation github-project-automation bot moved this to Todo in A3 Sep 1, 2023
@kevaundray
Copy link
Contributor Author

double_verify_proof tests were removed/ignored in #1925

@kevaundray
Copy link
Contributor Author

I've created a Noir issue here as it seems to be a Noir related bug or a bug in serde-reflect, still to be confirmed.

Right now we have a workaround so that we can re-enable the tests

@kevaundray kevaundray mentioned this issue Sep 4, 2023
Merged
4 tasks
@github-project-automation github-project-automation bot moved this from Todo to Done in A3 Sep 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-bb.js Component: bb.js - wrapping bberg in js
Projects
A3
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: Add back double verify proof to test suite AztecProtocol/aztec-packages
1 participant
@kevaundray