Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: add codspeed benchmarks [skip cirrus] #4678

Merged
merged 2 commits into from
May 18, 2024

Conversation

ev-br
Copy link
Contributor

@ev-br ev-br commented May 6, 2024

@martin-frbg martin-frbg changed the title WIP: add codspeed benchmarks WIP: add codspeed benchmarks [skip cirrus] May 7, 2024
@ev-br ev-br force-pushed the codspeed branch 2 times, most recently from 3935085 to b21ba1d Compare May 17, 2024 11:49
@ev-br
Copy link
Contributor Author

ev-br commented May 18, 2024

Phew. Finally managed to make it run on codspeed. Will clean up the codspeed action and revert back the other CI.

However, https://github.com/OpenMathLib/OpenBLAS/actions/runs/9138697348/job/25130179181?pr=4678 reports that

Upload the results
  CodSpeed Run Hash: "47ae471c56db8c1163d892307489c4ea202671cd8d4197ca08586f9ab23bddfd"
  Preparing upload...
  Error: Error Failed to retrieve upload data: 401 Unauthorized {"error":"The repository is not enabled"}
  Error: Process completed with exit code 1.

WDYT @martin-frbg ?

@martin-frbg
Copy link
Collaborator

Idk, this repository has never worn a codpiece before. Their site has now asked me to enable the repo and add a codspeed_token (which I don`t think it did before) so maybe it will work now

Copy link

codspeed-hq bot commented May 18, 2024

CodSpeed Performance Report

Congrats! CodSpeed is installed 🎉

🆕 16 new benchmarks were detected.

You will start to see performance impacts in the reports once the benchmarks are run from your default branch.

Detected benchmarks

  • test_daxpy[1000] (48.9 µs)
  • test_daxpy[100] (41.6 µs)
  • test_dot[1000] (43.4 µs)
  • test_dot[100] (36.9 µs)
  • test_gemm[1000] (1.1 s)
  • test_gemm[100] (1.7 ms)
  • test_gesdd[mn0] (349.8 µs)
  • test_gesdd[mn1] (451.7 ms)
  • test_gesv[1000] (454.4 ms)
  • test_gesv[100] (1.8 ms)
  • test_nrm2[1000] (53.8 µs)
  • test_nrm2[100] (56.4 µs)
  • test_syev[200] (333.8 ms)
  • test_syev[50] (8 ms)
  • test_syrk[1000] (642.6 ms)
  • test_syrk[100] (1.1 ms)

@ev-br ev-br force-pushed the codspeed branch 2 times, most recently from 5970ae9 to 53f8a77 Compare May 18, 2024 11:43
@ev-br
Copy link
Contributor Author

ev-br commented May 18, 2024

Yup, it did work this time: https://codspeed.io/OpenMathLib/OpenBLAS/branches/ev-br:codspeed !
The PR is ready from my side.

@martin-frbg martin-frbg added this to the 0.3.28 milestone May 18, 2024
@martin-frbg martin-frbg merged commit 25b34e6 into OpenMathLib:develop May 18, 2024
75 of 76 checks passed
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