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.

Sign up for GitHub

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

Jump to bottom

feat: trie multi diff comparison #655

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

feat: trie multi diff comparison #655

wants to merge 4 commits into from
+374 −112

Conversation

atanmarko
Copy link
Member

@atanmarko atanmarko commented Sep 24, 2024 •
edited
Loading

Resolves #636

@atanmarko atanmarko added this to the Testing and Validation milestone Sep 24, 2024
@atanmarko atanmarko self-assigned this Sep 24, 2024
@atanmarko atanmarko changed the base branch from develop to feat/trie-diff-tool September 24, 2024 14:41
@github-actions github-actions bot added crate: trace_decoder Anything related to the trace_decoder crate. crate: mpt_trie Anything related to the mpt_trie crate. crate: zero_bin Anything related to the zero-bin subcrates. labels Sep 24, 2024
@github-actions github-actions bot removed the crate: trace_decoder Anything related to the trace_decoder crate. label Sep 24, 2024
@atanmarko atanmarko changed the title feat: multi trie diff comparison feat: trie multi diff comparison Sep 24, 2024
@atanmarko atanmarko mentioned this pull request Sep 24, 2024
Merged
Base automatically changed from feat/trie-diff-tool to develop September 24, 2024 15:40
@github-actions github-actions bot added the crate: evm_arithmetization Anything related to the evm_arithmetization crate. label Sep 25, 2024
@atanmarko atanmarko force-pushed the feat/deep-tries-comparison branch 2 times, most recently from 4db375a to 793a081 Compare September 25, 2024 15:01
@atanmarko atanmarko marked this pull request as ready for review September 25, 2024 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@muursh muursh muursh approved these changes

@0xaatif 0xaatif Awaiting requested review from 0xaatif 0xaatif is a code owner

@Nashtare Nashtare Awaiting requested review from Nashtare Nashtare is a code owner

@LindaGuiga LindaGuiga Awaiting requested review from LindaGuiga LindaGuiga is a code owner

@wborgeaud wborgeaud Awaiting requested review from wborgeaud wborgeaud is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

@atanmarko atanmarko

Labels
crate: evm_arithmetization Anything related to the evm_arithmetization crate. crate: mpt_trie Anything related to the mpt_trie crate. crate: zero_bin Anything related to the zero-bin subcrates.
Projects
Zero EVM
Status: Ready to Review
Milestone
Testing and Validation
Development

Successfully merging this pull request may close these issues.

Update trie_diff tool to show all the differences in the tries
2 participants
@atanmarko @muursh