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

Cahce dependencies in CI to speedup workflows #471

Merged
merged 50 commits into from
Aug 10, 2023
Merged

Cahce dependencies in CI to speedup workflows #471

merged 50 commits into from
Aug 10, 2023

Conversation

savente93
Copy link
Contributor

@savente93 savente93 commented Aug 3, 2023

Issue addressed

Fixes #393

Explanation

If I'm honest I'm somewhat unsatisfied with this solution, but trying to optimise further is yielding all sorts of problems I don't quite understand, and it is actually faster, so I've decided to leave it where it is for now. If I gain new understanding or motivation I can come back to it later.

Because GitHub for now has a 10GB limit of caching, which is actually quite easy to bump up against if you're not careful, I've added some cache purging. Apparently it is quite easy for conda environments to get out of sync, so I've gone with what seems to be best practice: simply empty the cache at midnight. This might mean that the first Ci you run of the day might take longer, but after that it should be faster than usual. I think this is actually a decent mechanism to use our available cache at least somewhat effectively.

Checklist

  • Updated tests or added new tests
  • Branch is up to date with main
  • Tests & pre-commit hooks pass
  • Updated documentation if needed
  • Updated changelog.rst if needed

@savente93 savente93 changed the title it can't be this simple right? Cahce dependencies in CI to speedup workflows Aug 3, 2023
@savente93 savente93 marked this pull request as ready for review August 10, 2023 05:58
@savente93 savente93 merged commit 9c8052a into main Aug 10, 2023
8 checks passed
@savente93 savente93 deleted the ci-cache branch August 10, 2023 13:29
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.

CI environments should be cached
2 participants