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: adds cs.AssertIsDifferent API in the ConstraintSystem #131

Merged
merged 5 commits into from
Sep 1, 2021

Conversation

ThomasPiellard
Copy link
Collaborator

This PR fixes #128 , that is a constraint to assert that 2 variables are different has been added.

@gbotrel internal changes --> the ConstraintSystem struct now contains debugInfoComputation []logEntry and debugInfoAssertion []logEntry, one for the computational constraints and the other for the assertions (formerly named debugInfo).

A computational constraint fails only when a division by 0 occurs, so the debugInfoComputation is populated each time an api call to a function using division is made. When solving the constraint system, a counter is incremented each time a division is done, so that when a division by zero happens the corresponding entry ins debugInfoComputation (of index counter) is displayed (using the callstack).

@gbotrel gbotrel merged commit 5aaf531 into develop Sep 1, 2021
@gbotrel gbotrel changed the title Fix/i 128 feat: adds cs.AssertIsDifferent API in the ConstraintSystem Sep 1, 2021
@gbotrel gbotrel deleted the fix/i_128 branch September 1, 2021 16:58
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