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

Adds an onchain compatible transcript and removes Merlin #402

Merged
merged 4 commits into from
Jun 25, 2024

Conversation

aleph-v
Copy link
Collaborator

@aleph-v aleph-v commented Jun 24, 2024

Change list:

  1. Replaces the Merlin transcript which uses keccack1600 permutations with a transcript which we define for jolt and which uses keccack256. This transcript squeezes all inputs into 32bytes with an action counter. It doesn't abstract over the hasher object to support snark friendly recursion hashes, but this would be only a small change.
  2. Removes Merlin from our dependency graph
  3. Because we removed Merlin we bump the sha3 version to be modern as opposed to the 5 year old current import, and make small interface changes in key.rs and perdersen.rs
  4. Most labels are removed in favor of the increasing index which is hashed into each step.

There is no performance change.

@aleph-v aleph-v requested a review from moodlezoup June 24, 2024 12:58
@aleph-v aleph-v merged commit dbda0d4 into main Jun 25, 2024
3 checks passed
@aleph-v aleph-v mentioned this pull request Jun 25, 2024
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.

None yet

2 participants