-
Notifications
You must be signed in to change notification settings - Fork 487
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 github action speed for docs test #936
Conversation
noxan
commented
Nov 7, 2022
•
edited
Loading
edited
- Use pip cache in docs test github action (reduces runtime by ~2x, from 4.5 minutes down to 2.5 minutes)
Model Benchmark
Model TrainingPeytonManningYosemiteTempsAirPassengers |
Codecov Report
@@ Coverage Diff @@
## main #936 +/- ##
=======================================
Coverage 87.41% 87.41%
=======================================
Files 17 17
Lines 4464 4464
=======================================
Hits 3902 3902
Misses 562 562 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Really good idea! Can we also add this to the other tests that use the pip requirements file? eg. the metrics.yml test. |
@karl-richter yes, we could, I'd also support the idea. happy to follow up on this, might make sense to fix our dependency versions along that task. |
Is this for version 0.5.0? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm