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

Initial support for electra #41

Closed
wants to merge 4 commits into from

Conversation

gballet
Copy link

@gballet gballet commented Jan 26, 2023

This is based on a branch by Pari, which backports things from 0.60 to 0.59, but I am opening this PR here in the hope of getting some feedback. I can rebase afterwards.

Implementing the specObj methods confused me, I am probably doing something wrong and would love some input.

@gballet gballet force-pushed the verkle-proof-support branch 4 times, most recently from f81bd3c to 4b81c7c Compare February 1, 2023 20:10
implement Deserializable for all proof subtypes

add support for tree hashing

implement serialization for ExecutionWitness

add tests

fix value (de)serialization

test+fix: SuffixStateDiff

test+fix: StemStateDiffs

fixes for the full StateDiffs serialization

various fixes to the VerkleProof side

workaround: implement a custom ByteVector

use the PR version and remove the workaround

fix deserialization of variable fields in IPAProof + test

fix: declare execution witness as a root, not a complex type

fix: use ExecutionWitness when dumping genesis.ssz

add NewValue

fix all tests and improve serialization packing
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