manodeep / Corrfunc

⚡️⚡️⚡️Blazing fast correlation functions on the CPU.
https://corrfunc.readthedocs.io
MIT License
163 stars 50 forks source link

benchmarks on CI #293

Closed manodeep closed 1 year ago

manodeep commented 1 year ago

Trying to setup the benchmarks in the CI. We will need actual python code to run the benchmarks.

We might have to merge this PR into master before we can see whether the benchmarks are doing anything

lgarrison commented 1 year ago

Looks like the benchmark actions are running but failing. Can they be fixed in this PR before merging?

manodeep commented 1 year ago

Sorry - should have waited to click that "Review" button :D

Many more things need to be added before this PR is ready for a review.

manodeep commented 1 year ago

Here is a nice CI workflow that summarises the benchmark through a comment on the open PR: sympy/benchmark-summariser

Example comment in use sympy-pr-#23746

manodeep commented 1 year ago

Will restart this PR once I have finished it