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

[NO MERGE] Verkle trees #7890

Draft
wants to merge 21 commits into
base: master
Choose a base branch
from
Draft

[NO MERGE] Verkle trees #7890

wants to merge 21 commits into from

Conversation

zilm13
Copy link
Contributor

@zilm13 zilm13 commented Jan 16, 2024

PR Description

Verkle trees implementation.
Experimental feature #6346
As it's not implemented as a feature which could be turned off or on, it shouldn't be merged to the master and kept in branch until it's scheduled for some fork. After that it could be split in smaller chunks for better PRs

Fixed Issue(s)

Documentation

  • I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

  • I thought about adding a changelog entry, and added one if I deemed necessary.

import tech.pegasys.teku.spec.config.SpecConfigCapella;
import tech.pegasys.teku.spec.util.DataStructureUtil;

class ExecutionWitnessTest {

Check notice

Code scanning / CodeQL

Unused classes and interfaces Note test

Unused class: ExecutionWitnessTest is not referenced within this codebase. If not used as an external API it should be removed.
}

private <T> Optional<T> randomOptional(final Supplier<T> elementSupplier) {
return new Random(nextSeed()).nextBoolean()

Check warning

Code scanning / CodeQL

Random used only once Warning test

Random object created and used only once.
@zilm13 zilm13 added the DO NOT MERGE Not ready to merge label Mar 6, 2024
@zilm13 zilm13 self-assigned this Apr 2, 2024
@zilm13
Copy link
Contributor Author

zilm13 commented Jul 31, 2024

Follow ethereum/consensus-specs#3827
we will need update when it's merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT MERGE Not ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants