-
Notifications
You must be signed in to change notification settings - Fork 391
docs(aggregation_mode): upload benchmarks #1972
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
Conversation
MauroToscano
left a comment
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.
The most important number, time per proof, is missing
Thanks for pointing that out, it's now included in 8e0137c. I initially left it out because I wasn't sure it would be meaningful in isolation as the time per proof isn’t constant and depends on the number of proofs. That said, now that I think about it, I do agree it’s a useful point of reference, especially when comparing different setups under similar aggregation volumes and observing the amortization as we increase the batch size. |
|
Closing it as we'll only keep it as reference. |
Description
Uploads benchmark results for the aggregation mode.
Type of change
Checklist
testnet, everything else tostaging