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

[4/4 EIP 4844 in inserter circuit] prover: insertion_circuit: implement EIP 4844 #14

Merged
merged 3 commits into from
Jun 29, 2024

Conversation

wzmuda
Copy link
Collaborator

@wzmuda wzmuda commented Jun 13, 2024

This is patch 4/4 of the EIP 4844 in inserter circuit

Modify the inserter circuit to accept 4844 commitment and perform evaluation of its genuineness.

@wzmuda wzmuda requested a review from kustosz June 13, 2024 22:04
@wzmuda wzmuda changed the base branch from next to wz/barycentric-pr June 20, 2024 14:32
@wzmuda wzmuda force-pushed the wz/barycentric-pr branch from 432077c to 95aa6a3 Compare June 20, 2024 20:19
@wzmuda wzmuda changed the base branch from wz/barycentric-pr to next June 21, 2024 13:39
prover/insertion_circuit.go Outdated Show resolved Hide resolved
prover/insertion_circuit.go Outdated Show resolved Hide resolved
prover/insertion_circuit.go Outdated Show resolved Hide resolved
prover/insertion_circuit.go Outdated Show resolved Hide resolved
prover/insertion_circuit.go Outdated Show resolved Hide resolved
wzmuda and others added 3 commits June 26, 2024 22:36
Gnark does it automatically now.

Signed-off-by: Wojciech Zmuda <[email protected]>
Turn tree_test.go into a separate package called poseidon_native. This
package holds an implementation of the out-of-circuit poseidon tree.
To distinguish it from the in-circuit implementation in the poseidon
package, the _native suffix is added.

Signed-off-by: Wojciech Zmuda <[email protected]>
Co-authored-by: Marcin Kostrzewa <[email protected]>
Signed-off-by: Wojciech Zmuda <[email protected]>
@wzmuda wzmuda merged commit e3341d2 into next Jun 29, 2024
1 check passed
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