You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
…on (#5188)
This PR solve the verification issue with bigint division in Noir:
noir-lang/noir#4530
---------
Co-authored-by: Tom French <15848336+TomAFrench@users.noreply.github.com>
Co-authored-by: Rumata888 <isennovskiy@gmail.com>
AztecBot
pushed a commit
to AztecProtocol/barretenberg
that referenced
this issue
Mar 20, 2024
…on (#5188)
This PR solve the verification issue with bigint division in Noir:
noir-lang/noir#4530
---------
Co-authored-by: Tom French <15848336+TomAFrench@users.noreply.github.com>
Co-authored-by: Rumata888 <isennovskiy@gmail.com>
# Description
## Problem\*
Resolves#4578. The fix for issue #4530 has also being merged back so I
also activate the related test case.
## Summary\*
Field inversion for bigints had a typo.
## Additional Context
I did not consider biguint256 since it was decided not to merge the PR.
## Documentation\*
Check one:
- [X] No documentation needed.
- [ ] Documentation included in this PR.
- [ ] **[For Experimental Features]** Documentation to be submitted in a
separate PR.
# PR Checklist\*
- [X] I have tested the changes locally.
- [x] I have formatted the changes with [Prettier](https://prettier.io/)
and/or `cargo fmt` on default settings.
Problem
The doc snippet in the big int test (the function big_int_example(..)), from PR #4487, has been disabled because it fails verification.
Happy Case
The code should verify.
Project Impact
None
Impact Context
No response
Workaround
None
Workaround Description
No response
Additional Context
No response
Would you like to submit a PR for this Issue?
None
Support Needs
No response
The text was updated successfully, but these errors were encountered: