Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

docs: add an example on measuring import times #73

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

art049
Copy link
Member

@art049 art049 commented Feb 19, 2025

No description provided.

@art049 art049 mentioned this pull request Feb 19, 2025
Copy link

codspeed-hq bot commented Feb 19, 2025

CodSpeed Walltime Performance Report

Merging #73 will degrade performances by 12.46%

Comparing docs-import-time (651d394) with master (c92304c)

Summary

⚡ 10 improvements
❌ 7 regressions
✅ 40 untouched benchmarks
🆕 3 new benchmarks

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

Benchmarks breakdown

Benchmark BASE HEAD Change
test_iir_filter_process 2.8 µs 2.7 µs +3.57%
test_make_bandpass 4.8 µs 5 µs -3.3%
test_combination_lists[4-2] 3.5 µs 3.4 µs +5.2%
test_generate_all_combinations[4-2] 5.6 µs 5.5 µs +2.69%
test_generate_parenthesis[3] 7.4 µs 7.9 µs -6.05%
test_generate_sum_of_subsets_soln[nums0-9] 21.4 µs 20.4 µs +5.07%
test_is_safe[board0-1-1] 9.4 µs 9.1 µs +3.67%
test_open_knight_tour[1] 3 µs 2.8 µs +5.96%
test_solve_maze[maze0] 20 µs 18.5 µs +7.9%
🆕 test_bench_module_import[.module_a] N/A 1.9 ms N/A
🆕 test_bench_module_import[.module_b] N/A 3 ms N/A
test_count_even[count_even_slow] 1.3 ms 1.5 ms -12.46%
test_sum_of_squares[sum_of_squares_for_loop_power] 196.1 µs 206.4 µs -5.01%
test_sum_of_squares[sum_of_squares_sum_comprehension_power] 194.9 µs 208.2 µs -6.41%
test_sum_of_squares[sum_of_squares_sum_labmda_product] 217.4 µs 231.2 µs -5.98%
🆕 test_bench_hash N/A 55.1 µs N/A
test_noop_ellipsis_decorated 304 ns 280.9 ns +8.25%
test_noop_lambda 558.4 ns 547 ns +2.08%
test_noop_lambda_decorated 611.1 ns 645 ns -5.25%
test_noop_pass_decorated 298.9 ns 291 ns +2.69%

Copy link

codspeed-hq bot commented Feb 19, 2025

CodSpeed Instrumentation Performance Report

Merging #73 will not alter performance

Comparing docs-import-time (651d394) with master (c92304c)

Summary

✅ 57 untouched benchmarks
🆕 3 new benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
🆕 test_bench_module_import[.module_a] N/A 3.1 ms N/A
🆕 test_bench_module_import[.module_b] N/A 5.5 ms N/A
🆕 test_bench_hash N/A 159.9 µs N/A

@art049 art049 force-pushed the docs-import-time branch 6 times, most recently from c968dfe to 651d394 Compare February 21, 2025 15:24
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant