10000 [CD] Use Anaconda cmake for Mac builds by malfet · Pull Request #143054 · pytorch/pytorch · GitHub
[go: up one dir, main page]

Skip to content

[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

Closed
wants to merge 2 commits into from
Closed

Conversation

malfet
Copy link
Contributor
@malfet malfet commented Dec 11, 2024

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:

Fixes #142873

@malfet malfet requested a review from a team as a code owner December 11, 2024 22:23
Copy link
pytorch-bot bot commented Dec 11, 2024

🔗 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 Failures

As of commit c821a62 with merge base 1b3f8b7 (image):

NEW FAILURES - The following jobs have failed:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@malfet malfet added ciflow/binaries_wheel Trigger binary build and upload jobs for wheel on the PR ciflow/binaries_libtorch Trigger binary build and upload jobs for libtorch on the PR labels Dec 11, 2024
@malfet malfet force-pushed the malfet/pin-cmake-to-3.26 branch from 1f2ed7d to a5d20e3 Compare December 11, 2024 22:30
@malfet malfet added release notes: build release notes category topic: bug fixes topic category labels Dec 11, 2024
@malfet malfet force-pushed the malfet/pin-cmake-to-3.26 branch from a5d20e3 to 17b7f7e Compare December 11, 2024 22:37
@malfet malfet changed the title [CD] Pin cmake to 3.26 for MacOS wheel/torch builds [CD] Pin cmake to 3.26.4 for MacOS wheel/torch builds Dec 11, 2024
@malfet malfet force-pushed the malfet/pin-cmake-to-3.26 branch from 17b7f7e to 1505bc2 Compare December 12, 2024 00:04
@malfet malfet changed the title [CD] Pin cmake to 3.26.4 for MacOS wheel/torch builds [CD] Use Anaconda cmake Dec 12, 2024
Copy link
Contributor
@atalman atalman left a 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 ?

@malfet
Copy link
Contributor Author
malfet commented Dec 12, 2024

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)

@malfet malfet changed the title [CD] Use Anaconda cmake [CD] Use Anaconda cmake for Mac builds Dec 12, 2024
As newer one fails to find OpenMP
@malfet malfet force-pushed the malfet/pin-cmake-to-3.26 branch from 1505bc2 to d6efa1f Compare December 12, 2024 01:26
@malfet malfet force-pushed the malfet/pin-cmake-to-3.26 branch from d6efa1f to c821a62 Compare December 12, 2024 01:34
@malfet
Copy link
Contributor Author
malfet commented Dec 12, 2024

@pytorchbot merge -f "MacOS libtorch builds are green again"

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

@atalman
Copy link
Contributor
atalman commented Dec 12, 2024

@pytorchbot cherry-pick --onto release/2.6 -c critical

pytorchbot pushed a commit that referenced this pull request Dec 12, 2024
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)
@pytorchbot
Copy link
Collaborator

Cherry picking #143054

The 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 team Raised by workflow job

kit1980 pushed a commit that referenced this pull request Dec 12, 2024
[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>
bluenote10 pushed a commit to bluenote10/pytorch that referenced this pull request Dec 14, 2024
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
@github-actions github-actions bot deleted the malfet/pin-cmake-to-3.26 branch January 12, 2025 02:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/binaries_libtorch Trigger binary build and upload jobs for libtorch on the PR ciflow/binaries_wheel Trigger binary build and upload jobs for wheel on the PR Merged release notes: build release notes category topic: bug fixes topic category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MacOS libtorch builds are failing since Nov 12
5 participants
0