feat: add asv-codspeed package for airspeed velocity support#104
Draft
feat: add asv-codspeed package for airspeed velocity support#104
Conversation
Add a new `asv-codspeed` workspace package that enables running ASV (airspeed velocity) benchmarks with CodSpeed instrumentation. The package discovers `time_*` benchmarks, runs them through the existing pytest-codspeed instruments (walltime and valgrind), and reports results in the same format so the CodSpeed platform treats them identically. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
CodSpeed Performance ReportMerging this PR will create unknown performance changesComparing Summary
Performance Changes
Footnotes
|
Restructure the repository into a three-package monorepo: - codspeed: shared core with CFFI/C bindings, instruments (walltime, valgrind), config, and utilities. Owns the instrument-hooks submodule. - pytest-codspeed: pure Python pytest plugin, depends on codspeed - asv-codspeed: ASV benchmark runner, depends on codspeed (no longer depends on pytest-codspeed or pytest) Also adds asv-codspeed benchmarks job to the CodSpeed CI workflow. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The TheAlgorithms submodule contains test files that import numpy/requests which aren't installed in CI, causing collection errors. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add integration_name/integration_version params to Instrument base class - Move type-only imports to TYPE_CHECKING blocks (TC001/TC003) - Remove unused variables (F841) - Fix line length violations (E501) - Exclude TheAlgorithms submodule from ruff and mypy - Apply ruff format to all files Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Pre-commit naturally skips git submodule files, so explicit exclude config is not needed. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
asv-codspeedworkspace package that runs ASV (airspeed velocity)time_*benchmarks with CodSpeed instrumentationpytest-codspeedto the CodSpeed platformpython -m asv_codspeed run), parameterized benchmark support, and setup/teardown lifecycleTest plan
codspeed run --mode simulation— results uploaded successfully🤖 Generated with Claude Code