-
Notifications
You must be signed in to change notification settings - Fork 160
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
Integrate LogUp-GKR into the VM #1493
Conversation
Here are the numbers on my machine using Next branch (serial)
Next branch (concurrent 8 threads)
This branch (serial)
This branch (concurrent 8 threads)
|
As one can see, constraint evaluation for the Lagrange kernel and s-columns incurs an addition 3s in the multi-threaded benchmarks. This is more than 60% of the overhead of LogUp-GKR. This should go down once we parallelize constraint evaluation for the LogUp-GKR columns. |
Set up Next
This
The above numbers suggest that there is about half a second of overhead between constraint evaluation and building the DEEP poly. This means that GKR is taking about 1.5 seconds in the above. |
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.
LGTM! I love PRs that remove more code than they add :)
Should we merge this into some (new) logup-gkr
branch so that we can start working on integrating more bus/tables into LogUp-GKR? Ideally we wouldn't cram all of those into this PR.
😄
Agreed! |
Next BLAKE3 50 iterations
BLAKE3 100 iterations
Current BLAKE3 50 iterations
BLAKE3 100 iterations
|
* chore: fix no-std errors * wip * wip: migrate to new winterfell * wip: migrate to logup-gkr branch of winterfell * chore: fix issues after next rebase * wip: migrate range checker to use logup-gkr * wip: remove random row values * wip: remove range checker b-column from aux segment * chore: remove random trace rows at the end * chore: remove random trace rows at the end (continue) * chore: remove range checker from aux trace (continue) * chore: adapt to updated AirContext * fix: clippy * chore: update to changes in Winterfell * fix: clippy * changelog --------- Co-authored-by: Philippe Laferriere <[email protected]>
* chore: fix no-std errors * wip * wip: migrate to new winterfell * wip: migrate to logup-gkr branch of winterfell * chore: fix issues after next rebase * wip: migrate range checker to use logup-gkr * wip: remove random row values * wip: remove range checker b-column from aux segment * chore: remove random trace rows at the end * chore: remove random trace rows at the end (continue) * chore: remove range checker from aux trace (continue) * chore: adapt to updated AirContext * fix: clippy * chore: update to changes in Winterfell * fix: clippy * changelog --------- Co-authored-by: Philippe Laferriere <[email protected]>
* chore: fix no-std errors * wip * wip: migrate to new winterfell * wip: migrate to logup-gkr branch of winterfell * chore: fix issues after next rebase * wip: migrate range checker to use logup-gkr * wip: remove random row values * wip: remove range checker b-column from aux segment * chore: remove random trace rows at the end * chore: remove random trace rows at the end (continue) * chore: remove range checker from aux trace (continue) * chore: adapt to updated AirContext * fix: clippy * chore: update to changes in Winterfell * fix: clippy * changelog --------- Co-authored-by: Philippe Laferriere <[email protected]>
Describe your changes
Once this is merged, the PR will be ready for review.
Checklist before requesting a review
next
according to naming convention.