Skip to content

Tests: benchmarks#69

Open
nstarman wants to merge 1 commit intopatrick-kidger:mainfrom
nstarman:tests/benchmarks
Open

Tests: benchmarks#69
nstarman wants to merge 1 commit intopatrick-kidger:mainfrom
nstarman:tests/benchmarks

Conversation

@nstarman
Copy link
Contributor

@nstarman nstarman commented Jun 16, 2025

Followup to #67.

Requires:

  1. Codspeed to be set up
  2. the tag run-benchmarks to be added.

Signed-off-by: Nathaniel Starkman <nstarman@users.noreply.github.com>
@nstarman nstarman marked this pull request as ready for review June 17, 2025 02:00
@nstarman
Copy link
Contributor Author

@patrick-kidger I think this PR may be ready.
If you could hook quax up to codspeed then we can maybe try it out. Alternatively, if you don't want codspeed I can try refactoring this for vanilla pytest-benchmark.

@johannahaffner
Copy link

I had a longer discussion with @jpbrodrick on this topic, codspeed and JAX really do not seem to mesh well.
(Discussion started here.)

What I have settled on is using just pytest-benchmark, and running these locally (not in CI).

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.

2 participants