Skip to content

Receipt Indexing working with non-consecutive blocks#451

Closed
Zyouell wants to merge 45 commits into
feat/support_non_consecutive_blocks_ryhopefrom
zyouell/rt-nonconsecutive-block
Closed

Receipt Indexing working with non-consecutive blocks#451
Zyouell wants to merge 45 commits into
feat/support_non_consecutive_blocks_ryhopefrom
zyouell/rt-nonconsecutive-block

Conversation

@Zyouell

@Zyouell Zyouell commented Feb 6, 2025

Copy link
Copy Markdown
Contributor

This PR adds the functionality to the Receipt based circuits to work even if a block contains no relevant receipts. It does this by using dummy proofs for the value_extraction proof on the MPT side and an empty proof for the block proof on the verifiable_db side.

This way the IVC proof still links all the blocks together but if no rows are added to the table for a specific block the tree doesn't change. This PR resolves CRY-37.

silathdiir and others added 30 commits February 6, 2025 10:28
- Move the dependencies (including version) to the workspace
`Cargo.toml`.
- Update the available dependencies. As update `alloy` to `0.4` (`0.5`
and `0.6` of alloy could not work with integration test).
- Keep the current version of `ethereum-types` and `rlp`, since both is
related with our forked `eth-trie.rs`.

Integration test could work.
@linear

linear Bot commented Feb 6, 2025

Copy link
Copy Markdown

@Zyouell Zyouell force-pushed the zyouell/rt-nonconsecutive-block branch from a437f05 to 0b77fc4 Compare February 10, 2025 09:45
@Zyouell Zyouell changed the base branch from feat/support_non_consecutive_blocks_ryhope to zyouell/receipt-rebase February 10, 2025 10:57
@nicholas-mainardi nicholas-mainardi changed the base branch from zyouell/receipt-rebase to feat/support_non_consecutive_blocks_ryhope February 10, 2025 11:07
@Zyouell

Zyouell commented Feb 11, 2025

Copy link
Copy Markdown
Contributor Author

Closing this pull request as it has been moved to #453

@Zyouell Zyouell closed this Feb 11, 2025
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.

3 participants