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

criterion: cuprate-cryptonight #327

Draft
wants to merge 20 commits into
base: main
Choose a base branch
from

Conversation

hinto-janai
Copy link
Contributor

What

Adds Criterion benchmarks for cuprate-cryptonight in benches/criterion/cuprate-cryptonight.

Requires #196

@github-actions github-actions bot added A-dependency Related to dependencies, or changes to a Cargo.{toml,lock} file. A-workspace Changes to a root workspace file or general repo file. A-docs Related to documentation. A-benches Related to benchmarks. A-books Related to Cuprate's books. A-book-architecture Related to the Architecture book. labels Oct 24, 2024
@hinto-janai
Copy link
Contributor Author

Initial results.

Benchmark naming scheme: format!("{hash_fn_name}_{}", input.len())

E.g. cuprate_cryptonight::cryptonight_hash_v0(&[0; 8]) is v0_8.

benchmark average time per call
r_8 57.091 ms
r_64 58.804 ms
r_512 58.892 ms
r_4096 58.645 ms
r_65536 58.925 ms
v0_8 25.941 ms
v0_64 26.790 ms
v0_512 26.750 ms
v0_4096 26.300 ms
v0_65536 26.787 ms
v1_8 3.4884 ns
v1_64 26.920 ms
v1_512 26.406 ms
v1_4096 26.896 ms
v1_65536 27.092 ms
v2_64 28.900 ms
v2_512 29.331 ms
v2_4096 28.683 ms
v2_65536 29.345 ms

@hinto-janai hinto-janai mentioned this pull request Oct 25, 2024
27 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-benches Related to benchmarks. A-book-architecture Related to the Architecture book. A-books Related to Cuprate's books. A-dependency Related to dependencies, or changes to a Cargo.{toml,lock} file. A-docs Related to documentation. A-workspace Changes to a root workspace file or general repo file.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant