-
Notifications
You must be signed in to change notification settings - Fork 84
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
Benchmarking the Library #49
Comments
@hwwhww @ChihChengLiang do you have any suggestions for the /cc @pipermerriam @carver |
Some idea I can come up with now. Most of these can only start after the migration of BLS aggregation API from Trinity to here. Benchmark that focuses on the bottleneckThis protects the most important concerns, so we don't worry about breaking things. Example:
References for signature aggregation and crosslink calculation https://ethresear.ch/t/pragmatic-signature-aggregation-with-bls/2105 Benchmark that gives insights on APIsThis helps the calculation to use cases more straight forward. Example:
Benchmark that gives insights on unitsSo we can be more sensitive to the performance change. Example: This page shows how performance changes for each operation over time. https://speed.z.cash/timeline/#/?exe=1,2&base=1+9&ben=bls12_381::ec::g1::bench_g1_add_assign&env=1&revs=50&equid=off&quarts=on&extr=on |
Collection of updates
What is wrong?
I personally think that it would be better if there were some
benchmarks
added to this library, so that to make sure that we are not altering any optimizations made etc. while adding new functionalities. Further this is a pretty primitive module where I believe speed matters.How can it be fixed
I think we could something similar to what we are doing here
The text was updated successfully, but these errors were encountered: