-
Notifications
You must be signed in to change notification settings - Fork 24.3k
[CD] Use Anaconda cmake for Mac builds #143054
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
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/143054
Note: Links to docs will display an error until the docs builds have been completed. ❌ 3 New FailuresAs of commit c821a62 with merge base 1b3f8b7 ( NEW FAILURES - The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
1f2ed7d
to
a5d20e3
Compare
a5d20e3
to
17b7f7e
Compare
17b7f7e
to
1505bc2
Compare
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, but should we apply this on MacOS only if its affects MacOS ?
It is only applies to MacOS (but I can leave this comment on PR, as I've been confused by our build script names myself) |
As newer one fails to find OpenMP
1505bc2
to
d6efa1f
Compare
d6efa1f
to
c821a62
Compare
@pytorchbot merge -f "MacOS libtorch builds are green again" |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
@pytorchbot cherry-pick --onto release/2.6 -c critical |
To find Anaconda-env-installed OpenMP (As OpenMP from PyPI is looking for it in a different places) For posterity: our build script names are very confusing: - [`.ci/wheel/build_wheel.sh`](https://github.com/pytorch/pytorch/blob/6cb6e8d790fce6a24e16bf03afd7275bd294ad9c/.ci/wheel/build_wheel.sh) is only used for MacOS wheel/libtorch builds - [`.ci/manywheel/build.sh`](https://github.com/pytorch/pytorch/blob/6cb6e8d790fce6a24e16bf03afd7275bd294ad9c/.ci/manywheel/build.sh) are used for Linux wheel/libtorch builds - [`.ci/pytorch/windows/build_pytorch.bat`](https://github.com/pytorch/pytorch/blob/6cb6e8d790fce6a24e16bf03afd7275bd294ad9c/.ci/pytorch/windows/build_pytorch.bat) is used for Windows wheel builds Fixes #142873 Pull Request resolved: #143054 Approved by: https://github.com/Jack-Khuu, https://github.com/atalman (cherry picked from commit 4d8357e)
Cherry picking #143054The cherry pick PR is at #143133 and it is recommended to link a critical cherry pick PR with an issue. The following tracker issues are updated: Details for Dev Infra teamRaised by workflow job |
[CD] Use Anaconda cmake for Mac builds (#143054) To find Anaconda-env-installed OpenMP (As OpenMP from PyPI is looking for it in a different places) For posterity: our build script names are very confusing: - [`.ci/wheel/build_wheel.sh`](https://github.com/pytorch/pytorch/blob/6cb6e8d790fce6a24e16bf03afd7275bd294ad9c/.ci/wheel/build_wheel.sh) is only used for MacOS wheel/libtorch builds - [`.ci/manywheel/build.sh`](https://github.com/pytorch/pytorch/blob/6cb6e8d790fce6a24e16bf03afd7275bd294ad9c/.ci/manywheel/build.sh) are used for Linux wheel/libtorch builds - [`.ci/pytorch/windows/build_pytorch.bat`](https://github.com/pytorch/pytorch/blob/6cb6e8d790fce6a24e16bf03afd7275bd294ad9c/.ci/pytorch/windows/build_pytorch.bat) is used for Windows wheel builds Fixes #142873 Pull Request resolved: #143054 Approved by: https://github.com/Jack-Khuu, https://github.com/atalman (cherry picked from commit 4d8357e) Co-authored-by: Nikita Shulga <nshulga@meta.com>
To find Anaconda-env-installed OpenMP (As OpenMP from PyPI is looking for it in a different places) For posterity: our build script names are very confusing: - [`.ci/wheel/build_wheel.sh`](https://github.com/pytorch/pytorch/blob/6cb6e8d790fce6a24e16bf03afd7275bd294ad9c/.ci/wheel/build_wheel.sh) is only used for MacOS wheel/libtorch builds - [`.ci/manywheel/build.sh`](https://github.com/pytorch/pytorch/blob/6cb6e8d790fce6a24e16bf03afd7275bd294ad9c/.ci/manywheel/build.sh) are used for Linux wheel/libtorch builds - [`.ci/pytorch/windows/build_pytorch.bat`](https://github.com/pytorch/pytorch/blob/6cb6e8d790fce6a24e16bf03afd7275bd294ad9c/.ci/pytorch/windows/build_pytorch.bat) is used for Windows wheel builds Fixes pytorch#142873 Pull Request resolved: pytorch#143054 Approved by: https://github.com/Jack-Khuu, https://github.com/atalman
To find Anaconda-env-installed OpenMP
(As OpenMP from PyPI is looking for it in a different places)
For posterity: our build script names are very confusing:
.ci/wheel/build_wheel.sh
is only used for MacOS wheel/libtorch builds.ci/manywheel/build.sh
are used for Linux wheel/libtorch builds.ci/pytorch/windows/build_pytorch.bat
is used for Windows wheel buildsFixes #142873