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: CI asv benchmarks #144

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

WIP: CI asv benchmarks #144

wants to merge 1 commit into from

Conversation

basnijholt
Copy link
Member

(original merge request on GitLab)

opened by Bas Nijholt (@basnijholt) at 2017-11-20T15:49:27.871Z

Limitations:

  • asv doesn't keep any branch information, so we will upload the website with information of master + the commits of the latest branch that was benchmarked.

We cache the benchmarks of the master branch because missing commits in the results/ dir of asv will lead to errors.

@basnijholt
Copy link
Member Author

originally posted by Joseph Weston (@jbweston) at 2017-12-01T14:10:32.062Z on GitLab

Seems Good. We just need to do the following things to get this working

@basnijholt
Copy link
Member Author

originally posted by Joseph Weston (@jbweston) at 2017-12-01T14:14:53.467Z on GitLab

This is not necessary (we are already on the specified branch)

@basnijholt
Copy link
Member Author

originally posted by Joseph Weston (@jbweston) at 2017-12-01T14:15:55.749Z on GitLab

here we should pull down the existing benchmarks from tn1. This will stop asv run ALL from running for 1 billion years

@basnijholt
Copy link
Member Author

originally posted by Joseph Weston (@jbweston) at 2017-12-01T14:16:21.249Z on GitLab

before we merge this MR we should add a static site in admin_notes

@basnijholt
Copy link
Member Author

originally posted by Joseph Weston (@jbweston) at 2017-12-08T15:49:12.332Z on GitLab

@basnijholt did you see my comments?

@basnijholt
Copy link
Member Author

originally posted by Bas Nijholt (@basnijholt) at 2017-12-08T15:50:19.444Z on GitLab

Yes, I saw them, thanks, I'm just doing other stuff ATM.

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

1 participant