Skip to content

Conversation

adriencaccia
Copy link
Member

No description provided.

Copy link
Contributor

@GuillaumeLagrange GuillaumeLagrange left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm to me

Copy link

codspeed-hq bot commented Oct 14, 2025

CodSpeed Performance Report

Merging #140 will degrade performances by 65.9%

Comparing feat/no-benchmarks-message (8923641) with main (d71bcc0)

Summary

⚡ 15 improvements
❌ 21 regressions
✅ 327 untouched

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

Benchmarks breakdown

Mode Benchmark BASE HEAD Change
WallTime build_vec 224 ns 206 ns +8.74%
WallTime large_drop 832.3 µs 783.1 µs +6.28%
WallTime large_setup 24 ns 20 ns +20%
WallTime from_elem[1024] 84 ns 90 ns -6.67%
WallTime from_elem[2048] 248 ns 280 ns -11.43%
WallTime from_elem[4096] 337 ns 364 ns -7.42%
WallTime from_elem[8192] 524 ns 542 ns -3.32%
WallTime from_elem_decimal[1024] 84 ns 89 ns -5.62%
WallTime from_elem_decimal[2048] 249 ns 276 ns -9.78%
WallTime recursive[0] 1 ns 2 ns -50%
Instrumentation recursive_memoized[BTreeMap<u64, u64>, 0] 400.8 ns 371.7 ns +7.85%
WallTime recursive_memoized[BTreeMap<u64, u64>, 20] 2 µs 2.3 µs -11.52%
Instrumentation recursive_memoized[HashMap<u64, u64>, 0] 958.3 ns 929.2 ns +3.14%
WallTime generate_combinations[5] 959 ns 1,079 ns -11.12%
WallTime generate_combinations[6] 1.9 µs 1.8 µs +5.27%
WallTime generate_combinations[7] 3.2 µs 2.7 µs +19.46%
WallTime generate_combinations[9] 7.7 µs 5.2 µs +46.91%
WallTime generate_parentheses[4] 8.2 µs 7.7 µs +6.72%
WallTime graph_coloring[3] 1.6 µs 1.8 µs -14.39%
WallTime hamiltonian_cycle[4] 788 ns 764 ns +3.14%
... ... ... ... ... ...

ℹ️ Only the first 20 benchmarks are displayed. Go to the app to view all benchmarks.

@adriencaccia adriencaccia merged commit 8923641 into main Oct 14, 2025
17 of 18 checks passed
@adriencaccia adriencaccia deleted the feat/no-benchmarks-message branch October 14, 2025 14:46
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.

2 participants