Skip to content

Remove asv benchmarking#960

Merged
NicolaCourtier merged 4 commits into
developfrom
remove-asv-benchmarking
Jul 14, 2026
Merged

Remove asv benchmarking#960
NicolaCourtier merged 4 commits into
developfrom
remove-asv-benchmarking

Conversation

@NicolaCourtier

Copy link
Copy Markdown
Member

Description

Remove the files and comments relating to the Airspeed Velocity benchmarking, which is no longer working or needed.

Fixes #917

Type of change

Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #).

Important checks:

Please confirm the following before marking the PR as ready for review:

  • No style issues: $ pre-commit run or $ nox -s pre-commit (see CONTRIBUTING.md for how to set this up to run automatically when committing locally, in just two lines of code)
  • All tests pass: nox -s tests
  • The documentation builds: nox -s doctest
  • Code is commented for hard-to-understand areas
  • Tests added that prove fix is effective or that feature works

Comment thread benchmarks/__init__.py Outdated
@agriyakhetarpal

Copy link
Copy Markdown
Member

I think we can also remove multiprocessing_bench.py from the repo root while we are at it and resolve #916, since it was also there for benchmarking-related reasons. Thanks!

@NicolaCourtier NicolaCourtier linked an issue Jul 14, 2026 that may be closed by this pull request
@codecov

codecov Bot commented Jul 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.12%. Comparing base (4628a65) to head (1bfbc53).
⚠️ Report is 64 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #960      +/-   ##
===========================================
+ Coverage    89.41%   92.12%   +2.71%     
===========================================
  Files           63       74      +11     
  Lines         4857     5793     +936     
===========================================
+ Hits          4343     5337     +994     
+ Misses         514      456      -58     

☔ View full report in Codecov by Harness.
📢 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.

@NicolaCourtier NicolaCourtier merged commit a1f63a2 into develop Jul 14, 2026
34 checks passed
@NicolaCourtier NicolaCourtier deleted the remove-asv-benchmarking branch July 14, 2026 10: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.

remove benchmarking remove multiprocessing_bench.py script

2 participants