Skip to content

Add python 3.12 to wheel builds#4768

Merged
atalman merged 3 commits intopytorch:mainfrom
atalman:add_python_312
Dec 1, 2023
Merged

Add python 3.12 to wheel builds#4768
atalman merged 3 commits intopytorch:mainfrom
atalman:add_python_312

Conversation

@atalman
Copy link
Copy Markdown
Contributor

@atalman atalman commented Nov 29, 2023

Follow up after pytorch/pytorch#112882.
Please note conda-build and conda is not available yet for Python=3.12. Hence building wheels under Conda Python 3.9.

@vercel
Copy link
Copy Markdown

vercel bot commented Nov 29, 2023

@atalman is attempting to deploy a commit to the Meta Open Source Team on Vercel.

A member of the Team first needs to authorize it.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 29, 2023
Copy link
Copy Markdown
Contributor

@malfet malfet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why any changes to setup-binary-builds action is necessary? Are you saying that we build both wheels and conda packages for domains using conda dependencies?

@atalman
Copy link
Copy Markdown
Contributor Author

atalman commented Nov 29, 2023

We used similar code for python 3.11, which was cleaned recently: 3be953c

@atalman
Copy link
Copy Markdown
Contributor Author

atalman commented Nov 30, 2023

Why any changes to setup-binary-builds action is necessary? Are you saying that we build both wheels and conda packages for domains using conda dependencies?

Created an issue here: #4773 to get rid of conda env when running wheel builds.

Copy link
Copy Markdown
Contributor

@malfet malfet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@atalman atalman merged commit 32a4ef5 into pytorch:main Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/binaries/all CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants