Skip to content

prod: enable parallel testing for faster tests and fix codespeed deprecation#167

Closed
beckermr wants to merge 6 commits into
mainfrom
beckermr-patch-1
Closed

prod: enable parallel testing for faster tests and fix codespeed deprecation#167
beckermr wants to merge 6 commits into
mainfrom
beckermr-patch-1

Conversation

@beckermr

@beckermr beckermr commented Feb 7, 2026

Copy link
Copy Markdown
Collaborator

Hopefully the CI is a bit less painful with this.

I also fixed a deprecation coming in codspeed.

closes #166

Comment thread .github/workflows/python_package.yaml Outdated
@beckermr beckermr changed the title prod: enable parallel testing for faster tests prod: enable parallel testing for faster tests and fix codespeed deprecation Feb 7, 2026
Comment thread .github/workflows/python_package.yaml Outdated
Comment thread .github/workflows/python_package.yaml Outdated
@codspeed-hq

codspeed-hq Bot commented Feb 7, 2026

Copy link
Copy Markdown

Merging this PR will degrade performance by 38.29%

❌ 2 (👁 1) regressed benchmarks
✅ 29 untouched benchmarks

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

Performance Changes

Benchmark BASE HEAD Efficiency
👁 test_benchmarks_lanczos_interp[xval-no_conserve_dc-run] 677.3 µs 1,097.4 µs -38.29%
test_benchmark_spergel_calcfluxrad[run] 1.1 ms 1.5 ms -28.25%

Comparing beckermr-patch-1 (22351cf) with main (40845dd)

Open in CodSpeed

@beckermr

beckermr commented Feb 7, 2026

Copy link
Copy Markdown
Collaborator Author

Even with two workers GitHub is killing the jobs. That means we'll need split tests if we want this to be faster.

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.

adopt pytest-xdist for faster tests

1 participant