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

Airspeed Velocity benchmarking #260

Open
brynpickering opened this issue Sep 28, 2023 · 1 comment
Open

Airspeed Velocity benchmarking #260

brynpickering opened this issue Sep 28, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@brynpickering
Copy link
Contributor

What can be improved?

We currently have some basic time and memory benchmarking in our CI (see e.g. #258 #243) that rely on running one time/memory hungry example. I just saw that pandas and numpy use airspeed velocity for their benchmarking (see here for benchmark results that led to the long runtimes fixed in #258), which might provide us with a more feature-complete suite of time/memory consumption results for different parts of PAM.

Here is an example of benchmarking results in a PR CI action and a here is a discussion on how to implement it in a CI.

Version

0.2.5-dev

@brynpickering brynpickering added the enhancement New feature or request label Sep 28, 2023
@brynpickering
Copy link
Contributor Author

Digging into it a bit more, it might be a little outdated (pandas/numpy started working with it ~2015) and only pandas seems to have a hook to keep their benchmarking website up and running (asv-runner). However, the CI action might be an easy win to have a wider spread of time/memory checks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant