Skip to content

Add benchmarks for updating/extending multidict with kwargs#1103

Merged
asvetlov merged 1 commit intomasterfrom
benchmarks
Mar 25, 2025
Merged

Add benchmarks for updating/extending multidict with kwargs#1103
asvetlov merged 1 commit intomasterfrom
benchmarks

Conversation

@asvetlov
Copy link
Copy Markdown
Member

@asvetlov asvetlov commented Mar 25, 2025

No description provided.

@asvetlov asvetlov added the bot:chronographer:skip This PR does not need to include a change note label Mar 25, 2025
@asvetlov asvetlov marked this pull request as ready for review March 25, 2025 08:53
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 25, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.83%. Comparing base (f59d8d4) to head (25f9947).
Report is 66 commits behind head on master.

Additional details and impacted files 
@@            Coverage Diff             @@
##           master    #1103      +/-   ##
==========================================
+ Coverage   92.71%   92.83%   +0.12%     
==========================================
  Files          44       44              
  Lines        4980     5068      +88     
  Branches      383      385       +2     
==========================================
+ Hits         4617     4705      +88     
  Misses         95       95              
  Partials      268      268
Flag Coverage Δ
CI-GHA 92.83% <100.00%> (+0.12%) ⬆️
MyPy 86.52% <100.00%> (+0.22%) ⬆️
pytest 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Mar 25, 2025

CodSpeed Performance Report

Merging #1103 will not alter performance

Comparing benchmarks (25f9947) with master (f59d8d4)

Summary

✅ 118 untouched benchmarks
🆕 18 new benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
🆕 test_cimultidict_extend_istr_with_kwargs[c-extension-module] N/A 15 ms N/A
🆕 test_cimultidict_extend_istr_with_kwargs[pure-python-module] N/A 196.8 ms N/A
🆕 test_cimultidict_update_istr_with_kwargs[c-extension-module] N/A 341.5 µs N/A
🆕 test_cimultidict_update_istr_with_kwargs[pure-python-module] N/A 8.6 ms N/A
🆕 test_create_cimultidict_with_items_istr_with_kwargs[c-extension-module] N/A 136.7 µs N/A
🆕 test_create_cimultidict_with_items_istr_with_kwargs[pure-python-module] N/A 1 ms N/A
🆕 test_create_multidict_with_items_with_kwargs[case-insensitive-c-extension-module] N/A 160.7 µs N/A
🆕 test_create_multidict_with_items_with_kwargs[case-insensitive-pure-python-module] N/A 987.2 µs N/A
🆕 test_create_multidict_with_items_with_kwargs[case-sensitive-c-extension-module] N/A 100.8 µs N/A
🆕 test_create_multidict_with_items_with_kwargs[case-sensitive-pure-python-module] N/A 854.4 µs N/A
🆕 test_multidict_extend_str_with_kwargs[case-insensitive-c-extension-module] N/A 24 ms N/A
🆕 test_multidict_extend_str_with_kwargs[case-insensitive-pure-python-module] N/A 188.4 ms N/A
🆕 test_multidict_extend_str_with_kwargs[case-sensitive-c-extension-module] N/A 13 ms N/A
🆕 test_multidict_extend_str_with_kwargs[case-sensitive-pure-python-module] N/A 162.7 ms N/A
🆕 test_multidict_update_str_with_kwargs[case-insensitive-c-extension-module] N/A 362.2 µs N/A
🆕 test_multidict_update_str_with_kwargs[case-insensitive-pure-python-module] N/A 8.5 ms N/A
🆕 test_multidict_update_str_with_kwargs[case-sensitive-c-extension-module] N/A 298.6 µs N/A
🆕 test_multidict_update_str_with_kwargs[case-sensitive-pure-python-module] N/A 8.5 ms N/A

@asvetlov asvetlov merged commit f6b4db1 into master Mar 25, 2025
47 checks passed
@asvetlov asvetlov deleted the benchmarks branch March 25, 2025 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bot:chronographer:skip This PR does not need to include a change note

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@asvetlov