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

benchmarks on CI #293

Closed
wants to merge 3 commits into from
Closed

benchmarks on CI #293

wants to merge 3 commits into from

Conversation

manodeep
Copy link
Owner

@manodeep manodeep commented May 16, 2023

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

@manodeep manodeep requested a review from lgarrison May 16, 2023 00:24
@manodeep manodeep self-assigned this May 16, 2023
@lgarrison
Copy link
Collaborator

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

@manodeep
Copy link
Owner Author

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
Copy link
Owner Author

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 manodeep marked this pull request as draft May 17, 2023 00:40
@manodeep
Copy link
Owner Author

manodeep commented Jun 9, 2023

Will restart this PR once I have finished it

@manodeep manodeep closed this Jun 9, 2023
@manodeep manodeep deleted the CI-benchmarks branch October 10, 2023 01:18
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.

None yet

2 participants