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

Bring Circuit to 100% correctness #71

Closed
3 of 4 tasks
porcuquine opened this issue Mar 26, 2022 · 4 comments
Closed
3 of 4 tasks

Bring Circuit to 100% correctness #71

porcuquine opened this issue Mar 26, 2022 · 4 comments
Assignees
Labels
bug Something isn't working circuit Addressing this issue will affect the circuit. epic Umbrella issue

Comments

@porcuquine
Copy link
Collaborator

porcuquine commented Mar 26, 2022

This is a tracking issue which should not be closed until all known circuit issues have been resolved.

Known issues:

  • (fib 100) proofs fail to verify. [verified fixed in Only synthesize one circuit_frame at a time in Nova tests. #113]
  • test_create_open_and_verify_complicated_higher_order_functional_commitment2 fails (fcomm).
  • Error handling: in some cases either eval or circuit synthesis panics in error cases. Both should return explicit error.
  • FIXME: There are some cases in the circuit marked with FIXME because some known implementation detail has been deferred.
@porcuquine
Copy link
Collaborator Author

@emmorais Can you confirm that the last two boxes here can be checked? I also just tested the second and found it to pass after uncommenting. It would be valuable to get independent confirmation of that also.

@porcuquine porcuquine added circuit Addressing this issue will affect the circuit. epic Umbrella issue labels Dec 23, 2022
@huitseeker
Copy link
Member

@porcuquine @emmorais Can this be closed?

@emmorais
Copy link
Contributor

emmorais commented May 1, 2023

Yes, all FIXES where either fixed or an issue were created; bugs were fixed; and error handling was implemented. In my opinion it can be closed. @porcuquine, do you see anything still missing here?

@huitseeker
Copy link
Member

@porcuquine Feel free to reopen if you think something is missing. 100% of the audit correctness items are fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working circuit Addressing this issue will affect the circuit. epic Umbrella issue
Projects
None yet
Development

No branches or pull requests

3 participants