Use uv in CI to speed-up requirements install #2072
Conversation
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2072 +/- ##
==========================================
+ Coverage 82.64% 82.95% +0.30%
==========================================
Files 102 102
Lines 9475 9475
==========================================
+ Hits 7831 7860 +29
+ Misses 1644 1615 -29 ☔ View full report in Codecov by Sentry. |
| uv venv | ||
| - name: Install dependencies | ||
| run: uv pip install "huggingface_hub[dev] @ ." |
There was a problem hiding this comment.
You should be able to do uv pip install --system "huggingface_hub[dev] @ ." on the latest release, if you want to remove the virtualenv (per the PR summary), but using a virtualenv is also good.
There was a problem hiding this comment.
Thanks @charliermarsh! I just saw the issue has been merged on uv's repo 🎉. Thanks for the direct ping here ❤️
There was a problem hiding this comment.
Of course! It's pretty new so of course ping if you see issues with it.
ArthurZucker
left a comment
ArthurZucker
left a comment
There was a problem hiding this comment.
Nice! We'll also do something similar in transformers! Speeds up a lot the resolution as well Related PR: huggingface/transformers#29202
Quick benchmarks show it saves around 30s-40s per workflow (both for code quality and CI tests)
Note: running
uvinstead ofpipin Github CI is not yet a first-class citizen but should be the case once astral-sh/uv#1526 is addressed. This is why we need to source env in each step.