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

[CHIA-1655] simplify pre_validate_blocks_multiprocessing() #18836

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

[CHIA-1655] simplify pre_validate_blocks_multiprocessing() #18836

wants to merge 1 commit into from
+291 −280

Conversation

arvidn
Copy link
Contributor

@arvidn arvidn commented Nov 7, 2024

by making it pre-validate one block at a time. That's how we use it in production code anyway

Purpose:

TODO

Current Behavior:

New Behavior:

Testing Notes:

@arvidn arvidn added Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes labels Nov 7, 2024
@arvidn
simplify pre_validate_blocks_multiprocessing() by making it pre-valid…
cf7a872 
…ated one block at a time. That's how we use it in production code anyway
@arvidn arvidn force-pushed the simplify-multiprocess-validaton branch from 1916826 to cf7a872 Compare November 7, 2024 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@arvidn