Skip to content
This repository has been archived by the owner on Jul 5, 2024. It is now read-only.

MPT new descriptors & benchmark #1492

Merged
merged 7 commits into from
Jun 26, 2023
Merged

MPT new descriptors & benchmark #1492

merged 7 commits into from
Jun 26, 2023

Conversation

miha-stopar
Copy link
Collaborator

@miha-stopar miha-stopar commented Jun 22, 2023

Description

New descriptors are used which have Vec<Node> structure and thus the conversion in witness_row.rs from [][]byte to Vec<Node> was removed. Note that keccak_data is now part of the Node struct - each node holds the byte streams that need to be hashed.

Additionally, an MPT benchmark was added (some structs needed to be made public).

Type of change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Contents

  • All tests updated.
  • prepare_witness function removed from witness_row.rs.
  • MPT test updated to not use prepare_witness.
  • Benchmark added.

@github-actions github-actions bot added the crate-zkevm-circuits Issues related to the zkevm-circuits workspace member label Jun 22, 2023
@github-actions github-actions bot added crate-circuit-benchmarks Issues related to the circuit-benchmarks workspace member T-bench Type: benchmark improvements labels Jun 22, 2023
@miha-stopar miha-stopar changed the title MPT new descriptors MPT new descriptors & benchmark Jun 22, 2023
Copy link
Collaborator

@Brechtpd Brechtpd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for doing the witness refactor!

I think there's something up with the benchmark, but otherwise looks great.

circuit-benchmarks/src/mpt_circuit.rs Outdated Show resolved Hide resolved
circuit-benchmarks/src/mpt_circuit.rs Outdated Show resolved Hide resolved
circuit-benchmarks/src/mpt_circuit.rs Outdated Show resolved Hide resolved
zkevm-circuits/src/mpt_circuit/witness_row.rs Outdated Show resolved Hide resolved
@miha-stopar miha-stopar merged commit d341d7a into mpt2 Jun 26, 2023
@miha-stopar miha-stopar deleted the mpt-new-descriptors branch June 26, 2023 12:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
crate-circuit-benchmarks Issues related to the circuit-benchmarks workspace member crate-zkevm-circuits Issues related to the zkevm-circuits workspace member T-bench Type: benchmark improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants