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

tests: add an env variable bench for testing #89

Merged

Conversation

art049
Copy link
Member

@art049 art049 commented Mar 12, 2025

No description provided.

Copy link

codspeed-hq bot commented Mar 12, 2025

CodSpeed Instrumentation Performance Report

Merging #89 will degrade performances by 7.82%

Comparing cod-658-i-cannot-pass-an-env-variable-to-the-thing-running (eb092d6) with main (d223cd8)

Summary

❌ 7 regressions
✅ 161 untouched benchmarks
🆕 1 new benchmarks

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

Benchmarks breakdown

Benchmark BASE HEAD Change
bench_array1[42] 1 µs 1.1 µs -5.36%
bench_array2[42] 1 µs 1.1 µs -5.36%
init_array[42] 1 µs 1.1 µs -5.36%
mut_borrow 866.7 ns 895.8 ns -3.26%
🆕 print_env_hello N/A 8 µs N/A
recursive[10] 903.3 ns 932.5 ns -3.13%
instant 343.9 ns 373.1 ns -7.82%
system_time 345.3 ns 374.4 ns -7.79%

Copy link

codspeed-hq bot commented Mar 12, 2025

CodSpeed Walltime Performance Report

Merging #89 will degrade performances by 48.73%

Comparing cod-658-i-cannot-pass-an-env-variable-to-the-thing-running (1b8f549) with main (d223cd8)

Summary

⚡ 10 improvements
❌ 7 regressions
✅ 127 untouched benchmarks
🆕 1 new benchmarks

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

Benchmarks breakdown

Benchmark BASE HEAD Change
build_vec 215 ns 201 ns +6.97%
large_drop 181.8 µs 166.9 µs +8.91%
large_setup 12 ns 11 ns +9.09%
from_elem[2048] 249 ns 237 ns +5.06%
from_elem[8192] 489 ns 471 ns +3.82%
bench_array1[4] 3.2 µs 3.9 µs -18.8%
bench_array2[4] 3.2 µs 3.9 µs -18.8%
init_array[1000] 6.8 µs 1.7 µs ×3.9
init_array[4] 3.2 µs 3.9 µs -18.8%
🆕 print_env_hello N/A 1.9 µs N/A
generate_combinations[8] 3.2 µs 3.1 µs +3.1%
graph_coloring[5] 3.9 µs 4 µs -3.01%
rat_in_maze[6] 2.9 µs 5.6 µs -48.73%
add_two_integers[(255, 255)] 3.4 µs 5.1 µs -33.83%
add_two_integers[(42, 13)] 3.4 µs 5.1 µs -33.83%
sleep_100ns 54.4 µs 42.2 µs +29.09%
sleep_100us 157.7 µs 140.2 µs +12.46%
sleep_1ns 57.6 µs 24.6 µs ×2.3

Base automatically changed from time-scale-benches to main April 7, 2025 07:27
@art049 art049 force-pushed the cod-658-i-cannot-pass-an-env-variable-to-the-thing-running branch 2 times, most recently from dda8c07 to eb092d6 Compare April 7, 2025 07:40
@art049 art049 force-pushed the cod-658-i-cannot-pass-an-env-variable-to-the-thing-running branch from eb092d6 to 1b8f549 Compare April 7, 2025 07:40
@art049 art049 merged commit 1b8f549 into main Apr 7, 2025
12 of 13 checks passed
@art049 art049 deleted the cod-658-i-cannot-pass-an-env-variable-to-the-thing-running branch April 7, 2025 07:57
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.

1 participant