-
Notifications
You must be signed in to change notification settings - Fork 6
Move analysis tools (such as plotting) to pyperf #380
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
Comments
Wishlist:
I'm not sure where all the above belongs to. If something cannot be moved, I would be more than happy also to make bench runner disjoint from GitHub so it can be run manually or on another automation system (Gitlab, Jenkins, cron) |
I'd put these things into 2 categories: Things that are (a) about analyzing 1-2 benchmark runs, and (b) things that are about managing a whole set of benchmark runs. The profiling plots and violin plots are in (a), and could reasonably be moved to pyperf (they are just analogous to tools that already exist there). Things like (b) don't really make sense for pyperf. If we want to handle things like (b) on things other than GitHub Actions, that really just means reimplementing the workflow descriptions in Let me propose this as a way forward:
I can handle (1) as a first step. It sounds like you are interested in (2). (3) is sort of a "nice to have" and is disjoint and optional from the rest, IIUC. |
Step (1) is complete. |
We should probably be working to move anything that isn't strictly related to automation inside of GitHub Actions upstream to pyperf.
The text was updated successfully, but these errors were encountered: