Skip to content

feat: Unify some node and builder validation logic with essential-validate#173

Draft
mitchmindtree wants to merge 1 commit intomainfrom
mitchmindtree/unify-validation
Draft

feat: Unify some node and builder validation logic with essential-validate#173
mitchmindtree wants to merge 1 commit intomainfrom
mitchmindtree/unify-validation

Conversation

@mitchmindtree
Copy link
Copy Markdown
Contributor

@mitchmindtree mitchmindtree commented Feb 10, 2025

This new crate provides functions for validating sequences of the SolutionSets in parallel, useful for both block validation and block building. SolutionSet's Predicates and Programs are also loaded in parallel.

Closes #123

To-Do

  • Refactor builder's parallel validation into general-use essential-validate crate.
  • Refactor essential-node to use essential-validate for validating a block's solutions in parallel (supporting dry-run too).
  • Refactor essential-builder to use essential-validate.

…lidate`

This new crate provides functions for validating sequences of the
`SolutionSet`s in parallel, useful for both block validation and block
building.

A `SolutionSet`'s `Predicate`s and `Program`s are also loaded in
parallel.

Not yet used within the node or builder as of this commit.
@mitchmindtree mitchmindtree added 📈 optimization Optimizing without necessarily extending functionality ♻️ refactor Restructuring without necessarily extending functionality ✔️ node: validation Related to node's validation function labels Feb 10, 2025
@mitchmindtree mitchmindtree self-assigned this Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✔️ node: validation Related to node's validation function 📈 optimization Optimizing without necessarily extending functionality ♻️ refactor Restructuring without necessarily extending functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unify builder and node validation

1 participant