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

Adding constRoot to the transcript and tags #51

Merged
merged 13 commits into from
Oct 16, 2023

Conversation

RogerTaule
Copy link
Contributor

@RogerTaule RogerTaule commented Oct 12, 2023

This PR adds the following changes:

  • ConstRoot is now added as part of the transcript.
  • Several comments has been added to the bn128 circuits, with specific attention to bn1togl3 circuit.
  • Tags has been added to have a much better control over the maximum number of bits that the GLCMul operation can have
  • A few more tests have been added.

@cla-bot cla-bot bot added the cla-signed label Oct 12, 2023
@RogerTaule RogerTaule force-pushed the feature/constRootTranscript branch 2 times, most recently from 4752368 to 255c5d8 Compare October 12, 2023 14:15
@RogerTaule RogerTaule force-pushed the feature/constRootTranscript branch from 255c5d8 to 48800ca Compare October 12, 2023 14:42
@RogerTaule RogerTaule force-pushed the feature/constRootTranscript branch from 9841209 to 421fd6d Compare October 13, 2023 09:45
…initial number is positive, adding LessThan64Bits to check that number is less than 64 bits
@RogerTaule RogerTaule merged commit 4204fd2 into develop Oct 16, 2023
4 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants