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

Improve caches #242

Merged
merged 2 commits into from
Apr 20, 2024
Merged

Improve caches #242

merged 2 commits into from
Apr 20, 2024

Conversation

RunDevelopment
Copy link
Member

My current guess is that our CI fails because we cache too many models. GH only gives us 10GB, so we were almost hitting that limit previously.

This PR changes two things:

  1. Don't cache pip installs. We don't really get a huge speedup for that, but we do have to pay 250MB per runner.
  2. Remove model files that were downloaded from GitHub. We mainly use the cache to speed up tests, but there's no difference in download speed between GitHub CI downloading a model from cache vs downloading it from a GitHub release page.

I also formatted our YAML files.

@RunDevelopment RunDevelopment merged commit ecb2f2b into main Apr 20, 2024
8 checks passed
@RunDevelopment RunDevelopment deleted the improve-caches branch April 20, 2024 17:03
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.

2 participants