-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
cksum: symlink to md5sum,b2sum, and sha*sum #9984
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
CodSpeed Performance ReportMerging #9984 will improve performance by ×8,800Comparing Summary
Benchmarks breakdown
Footnotes
|
a79d274 to
a132677
Compare
|
I considered that CodSpeed was broken, but it seems reproducible... |
|
|
|
@naoNao89 Any idea about CodSpeed breakage? |
|
We should fix or remove #10002 before discarding |
|
The file creation happens outside the benchmark loop, so the benchmarks are measuring almost nothing. uu/cksum/benches/cksum_bench.rs:14-24 fn $algo_name(bencher: Bencher) {
let data = text_data::generate_by_size(100, 80);
let file_path = setup_test_file(&data); // File created ONCE, outside loop
bencher.bench(|| {
black_box(run_util_function(
uumain,
&["--algorithm", $algo_str, file_path.to_str().unwrap()],
));
}); // This loop only measures reading a cached file
}I think we should:
|
4f404b2 to
bc9fcbd
Compare
|
GNU testsuite comparison: |
0aca569 to
39d9dad
Compare
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
GNU testsuite comparison: |
|
|
|
GNU testsuite comparison: |
f5d29b8 to
ed07374
Compare
|
GNU testsuite comparison: |
044bb60 to
de41913
Compare
|
GNU testsuite comparison: |
d5e6a80 to
944ea9a
Compare
|
GNU testsuite comparison: |
|
@cakebaker @RenjiSann I made a symlink plan. I think we can make individual bins after hashsum deletion. Failing test is |
|
GNU testsuite comparison: |
|
This looks very dangerous option if we don't have an extension |
Alt #9776 . but without individual bins.