Skip to content

bench: tsort_input_parsing_heavy reduce the input side.#9067

Merged
cakebaker merged 1 commit intomainfrom
sylvestre-patch-2
Oct 29, 2025
Merged

bench: tsort_input_parsing_heavy reduce the input side.#9067
cakebaker merged 1 commit intomainfrom
sylvestre-patch-2

Conversation

@sylvestre
Copy link
Copy Markdown
Contributor

it takes 12+ sec for now

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Oct 28, 2025

CodSpeed Performance Report

Merging #9067 will degrade performances by 2.65%

Comparing sylvestre-patch-2 (d1be7b6) with main (5c5494a)

Summary

❌ 3 regressions
✅ 101 untouched
🆕 1 new

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark BASE HEAD Change
du_balanced_tree[(5, 4, 10)] 9.1 ms 9.3 ms -2.65%
du_human_balanced_tree[(5, 4, 10)] 10.2 ms 10.5 ms -2.46%
du_summarize_balanced_tree[(5, 4, 10)] 8.4 ms 8.5 ms -2.06%
🆕 tsort_input_parsing_heavy[5000] N/A 122.4 ms N/A

@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

Skipping an intermittent issue tests/tail/overlay-headers (passes in this run but fails in the 'main' branch)

@RenjiSann
Copy link
Copy Markdown
Collaborator

75 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.

Note that you should archive tsort_wide_dag[50000] benchmark

@cakebaker cakebaker merged commit 99b8475 into main Oct 29, 2025
237 of 238 checks passed
@cakebaker cakebaker deleted the sylvestre-patch-2 branch October 29, 2025 15:04
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