Skip to content
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

Add benchmarking for significant operations #97

Open
TomAFrench opened this issue Jan 15, 2025 · 2 comments · May be fixed by #100
Open

Add benchmarking for significant operations #97

TomAFrench opened this issue Jan 15, 2025 · 2 comments · May be fixed by #100
Assignees
Labels
enhancement New feature or request

Comments

@TomAFrench
Copy link
Member

This is a high importance library but we don't have any measurements on performance. I've described the CI setup necessary to do this in noir-lang/noir-library-starter#7.

@noir-lang/libraries can you add benchmarking for the gate costs of important functions so that we can track how these change as development continues.

@kashbrti
Copy link
Contributor

I've been debating what good benchmarks for bignum would be.
Does reusing some of the test functions for benchmarking make sense?

@Savio-Sou Savio-Sou added the enhancement New feature or request label Jan 15, 2025
@Savio-Sou Savio-Sou added this to Noir Jan 15, 2025
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Noir Jan 15, 2025
@kashbrti kashbrti moved this from 📋 Backlog to 🏗 In progress in Noir Jan 16, 2025
@kashbrti
Copy link
Contributor

this is being tackled in #100

@Savio-Sou Savio-Sou linked a pull request Jan 16, 2025 that will close this issue
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 🏗 In progress
Development

Successfully merging a pull request may close this issue.

4 participants