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

[RFC] Use uv package manager in GHA #2713

Closed
wants to merge 2 commits into from
Closed

[RFC] Use uv package manager in GHA #2713

wants to merge 2 commits into from

Conversation

saitcakmak
Copy link
Contributor

@saitcakmak saitcakmak commented Jan 30, 2025

I came across uv while debugging the scipy slowdowns and noticed how much faster it is than vanilla pip. Using uv saves a couple minutes in installing dependencies, which can add up to meaningful compute savings over many CI runs we do every day. Should we update all workflows to use uv?

This PR updates reusable_tutorials.yml as an example.
Before, with pip: https://github.com/pytorch/botorch/actions/runs/13058152281/job/36434279975

  • Total setup time: 2 + 2 + 12 + 44 + 50 + 9 = 119s
Screenshot 2025-01-30 at 12 59 40 PM

After, with uv pip: https://github.com/pytorch/botorch/actions/runs/13058212963/job/36434483617

  • Total setup time: 2 + 1 + 10 + 10 + 7 + 6 = 36s
Screenshot 2025-01-30 at 1 00 19 PM

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Jan 30, 2025
Copy link

codecov bot commented Jan 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.98%. Comparing base (472de87) to head (7f83132).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2713   +/-   ##
=======================================
  Coverage   99.98%   99.98%           
=======================================
  Files         202      202           
  Lines       18602    18602           
=======================================
  Hits        18600    18600           
  Misses          2        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Balandat
Copy link
Contributor

sure that sounds like a good change, let's update this across the board.

@facebook-github-bot
Copy link
Contributor

@saitcakmak has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@saitcakmak merged this pull request in 4a595f2.

@saitcakmak saitcakmak deleted the saitcakmak-uv branch January 31, 2025 16:34
saitcakmak added a commit to facebook/Ax that referenced this pull request Feb 3, 2025
Summary:
Replicates pytorch/botorch#2713 for Ax.
`uv` is much faster than `pip`, which leads to a couple minutes of speed up in each workflow, which can add up to meaningful compute savings over many CI runs we do every day.

Pull Request resolved: #3290

Test Plan:
Successful CI run

Compared to a previous CI run
Before - 155 secs of setup time (until test & coverage step): https://github.com/facebook/Ax/actions/runs/13040634315/job/36381474759
After - 48 secs of setup time (until test & coverage step): https://github.com/facebook/Ax/actions/runs/13061685440/job/36445887187

Reviewed By: Balandat

Differential Revision: D68919180

Pulled By: saitcakmak
saitcakmak added a commit to facebook/Ax that referenced this pull request Feb 3, 2025
Summary:
Replicates pytorch/botorch#2713 for Ax.
`uv` is much faster than `pip`, which leads to a couple minutes of speed up in each workflow, which can add up to meaningful compute savings over many CI runs we do every day.

Pull Request resolved: #3290

Test Plan:
Successful CI run

Compared to a previous CI run
Before - 155 secs of setup time (until test & coverage step): https://github.com/facebook/Ax/actions/runs/13040634315/job/36381474759
After - 48 secs of setup time (until test & coverage step): https://github.com/facebook/Ax/actions/runs/13061685440/job/36445887187

Reviewed By: Balandat

Differential Revision: D68919180

Pulled By: saitcakmak
facebook-github-bot pushed a commit to facebook/Ax that referenced this pull request Feb 3, 2025
Summary:
Replicates pytorch/botorch#2713 for Ax.
`uv` is much faster than `pip`, which leads to a couple minutes of speed up in each workflow, which can add up to meaningful compute savings over many CI runs we do every day.

Pull Request resolved: #3290

Test Plan:
Successful CI run

Compared to a previous CI run
Before - 155 secs of setup time (until test & coverage step): https://github.com/facebook/Ax/actions/runs/13040634315/job/36381474759
After - 48 secs of setup time (until test & coverage step): https://github.com/facebook/Ax/actions/runs/13061685440/job/36445887187

Reviewed By: Balandat

Differential Revision: D68919180

Pulled By: saitcakmak

fbshipit-source-id: 3953ce5ff1aaae78587e56ab4046f70fee86cdf7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants