Skip to content

Conversation

@xtqqczze
Copy link
Contributor

@xtqqczze xtqqczze commented Dec 31, 2025

The implementation is identical.
@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/timeout/timeout (passes in this run but fails in the 'main' branch)

@codspeed-hq
Copy link

codspeed-hq bot commented Dec 31, 2025

CodSpeed Performance Report

Merging #9957 will improve performance by 10.79%

Comparing xtqqczze:bigdecimal-powi (c442fda) with main (1eea517)

Summary

⚡ 9 improvements
✅ 127 untouched
⏩ 24 skipped1

Benchmarks breakdown

Benchmark BASE HEAD Efficiency
sort_case_insensitive[500000] 276.7 ms 265.9 ms +4.07%
sort_mixed_data[500000] 328.1 ms 317.3 ms +3.42%
sort_ascii_utf8_locale 22.2 ms 20.1 ms +10.79%
sort_numeric_utf8_locale 23.8 ms 22.8 ms +4.49%
sort_case_sensitive[500000] 174 ms 163.1 ms +6.64%
sort_ascii_only[500000] 353.8 ms 343 ms +3.16%
sort_accented_data[500000] 362.6 ms 351.8 ms +3.08%
sort_reverse_locale[500000] 361.8 ms 350.9 ms +3.09%
sort_ascii_c_locale 22.4 ms 20.3 ms +10.69%

Footnotes

  1. 24 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@cakebaker cakebaker merged commit 9ae5be3 into uutils:main Jan 1, 2026
129 checks passed
@cakebaker
Copy link
Contributor

Thanks!

@oech3
Copy link
Contributor

oech3 commented Jan 1, 2026

Did this broke GNU tests for shuf shred factor at main?

@cakebaker
Copy link
Contributor

cakebaker commented Jan 1, 2026

Hm, I don't know. Why do you think it broke them?

@oech3
Copy link
Contributor

oech3 commented Jan 1, 2026

@xtqqczze
Copy link
Contributor Author

xtqqczze commented Jan 1, 2026

https://github.com/uutils/coreutils/actions/runs/20635323381/job/59259777511

Based on the checked-out ref 1e11921 for that job, this PR appears to have been the most recently merged, so it is possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants