-
Notifications
You must be signed in to change notification settings - Fork 38
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
feat: revamp zero prove function #793
base: develop
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good, but I'd still like to run some quick benchmarks myself to make sure it works as expected. Will report back.
FYI I'm getting a ~5% improvement over a small sequence (3 blocks, 600 txns). |
94d1474
to
86d619a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nits
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am by no means an expert, but it looks good to me!
Resolves #627
Resolves #399
Resolves #403
Segmentation and task dispatch logic has been changed to handle proving of the individual segments of one batch on different workers. Aggregation is performed after the segments of one batch are proven (not waiting of segments of all the batches to be completed). Segments are generated sequentially and consumed immediately.