Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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: zk sumcheck #7517
feat: zk sumcheck #7517
Changes from all commits
36523c1
4692e4c
da478a6
ffdf418
573f13c
be77077
ec0bfc7
0fbf781
051a966
afa284e
74cf2e7
2487f75
63bd81b
d449355
c5fab2b
4dbae08
eaeca2d
14fb4e1
c2bf3f6
0c07e19
2002fe3
1dd93cc
94c95f3
e75c4f9
70d04c9
332eab9
f6916b9
c62a745
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Large diffs are not rendered by default.
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.
why do you need this concept to be so complicated?
Large diffs are not rendered by default.
Large diffs are not rendered by default.
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.
why is this typename = void needed?
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.
so that claimed_libra_evaluations are added to the output automatically if the flavor has zk, this way I don't need to deel with the optional access
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.
would be good to get yourself used to not overusing
auto
and also you could return directlylibra_round_univariate
+round_univariate
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.
rather than carrying the round_idx you could determine it based on round length
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.
do you think it would be more readable? cause sumcheck prover round is operating on the size of polynomials/2^{round_idx}. i thought about deriving it this way, but seems more complicated