Add performance tests#138
Merged
isteinbrecher merged 1 commit intoJun 18, 2026
Merged
Conversation
isteinbrecher
temporarily deployed
to
cubit_secrets_trusted
GitHub Actions
Inactive
isteinbrecher
force-pushed
the
add_performance_test
branch
from
e8983a5 to
b4ca09c
Compare
isteinbrecher
had a problem deploying
to
cubit_secrets_trusted
GitHub Actions
Failure
isteinbrecher
force-pushed
the
add_performance_test
branch
from
b4ca09c to
f3a6a6f
Compare
isteinbrecher
temporarily deployed
to
cubit_secrets_trusted
GitHub Actions
Inactive
isteinbrecher
force-pushed
the
add_performance_test
branch
from
f3a6a6f to
815b85c
Compare
isteinbrecher
temporarily deployed
to
cubit_secrets_trusted
GitHub Actions
Inactive
There was a problem hiding this comment.
Pull request overview
This PR adds a small performance benchmark suite for CubitPy’s local vs remote execution paths, using pytest-benchmark to time common operations and compare them against reference thresholds.
Changes:
- Added
tests/test_cubitpy_performance.pywith fourpytest-benchmarkperformance tests and a sharedbenchmark_cubitpyhelper fixture. - Added
pytest-benchmarkto thedevoptional dependencies inpyproject.toml.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.
| File | Description |
|---|---|
| tests/test_cubitpy_performance.py | Introduces benchmark-based performance tests and a helper fixture to compare runtimes to local/remote reference thresholds. |
| pyproject.toml | Adds pytest-benchmark to the development dependencies so the new benchmark tests can run. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
isteinbrecher
force-pushed
the
add_performance_test
branch
from
815b85c to
31e1c88
Compare
isteinbrecher
temporarily deployed
to
cubit_secrets_trusted
GitHub Actions
Inactive
isteinbrecher
force-pushed
the
add_performance_test
branch
from
31e1c88 to
6209ca0
Compare
isteinbrecher
temporarily deployed
to
cubit_secrets_trusted
GitHub Actions
Inactive
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.
No description provided.