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

Improve benchmarking #202

Merged
merged 1 commit into from
Aug 23, 2023
Merged

Improve benchmarking #202

merged 1 commit into from
Aug 23, 2023

Conversation

richardpringle
Copy link
Contributor

Use criterion and don't benchmark key generation

firewood/benches/hashops.rs Outdated Show resolved Hide resolved
firewood/benches/hashops.rs Outdated Show resolved Hide resolved
firewood/benches/hashops.rs Outdated Show resolved Hide resolved
@richardpringle richardpringle marked this pull request as ready for review August 22, 2023 17:32
@richardpringle richardpringle force-pushed the improve-benchmarking branch 2 times, most recently from d84ee4b to 93d8548 Compare August 22, 2023 20:44
@rkuris rkuris self-requested a review August 22, 2023 22:28
firewood/benches/hashops.rs Outdated Show resolved Hide resolved
firewood/benches/hashops.rs Outdated Show resolved Hide resolved
shale/benches/shale-bench.rs Show resolved Hide resolved
shale/benches/shale-bench.rs Outdated Show resolved Hide resolved
@rkuris rkuris self-requested a review August 23, 2023 13:41
@richardpringle richardpringle force-pushed the improve-benchmarking branch 2 times, most recently from e2e2b75 to ee433bd Compare August 23, 2023 15:49
rkuris
rkuris previously requested changes Aug 23, 2023
firewood/benches/hashops.rs Outdated Show resolved Hide resolved
@richardpringle
Copy link
Contributor Author

@richardpringle richardpringle merged commit f380444 into main Aug 23, 2023
5 checks passed
@richardpringle richardpringle deleted the improve-benchmarking branch August 23, 2023 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants