Skip to content

Conversation

@huydhn
Copy link
Contributor

@huydhn huydhn commented Dec 18, 2023

Remove the custom tweak for 3.12 wheel and enable 3.12 for all domains in both wheel and conda builds. If nightly looks ok after this, we should continue to enable 3.12 for test and release according to 2.2.0 release matrix.

@huydhn huydhn requested a review from atalman December 18, 2023 20:42
@vercel
Copy link

vercel bot commented Dec 18, 2023

@huydhn 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 Dec 18, 2023
@huydhn huydhn marked this pull request as ready for review December 18, 2023 22:33
@vercel
Copy link

vercel bot commented Dec 18, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
torchci ⬜️ Ignored (Inspect) Visit Preview Dec 19, 2023 0:53am

@huydhn
Copy link
Contributor Author

huydhn commented Dec 19, 2023

Conda torchtext build failures are caused by torchdata, which doesn't have 3.12 ready. It's easier to just merge this change and rebuild torchdata nightly

@huydhn huydhn merged commit f702f81 into pytorch:main Dec 19, 2023
atalman pushed a commit to atalman/test-infra that referenced this pull request Dec 20, 2023
Remove the custom tweak for 3.12 wheel and enable 3.12 for all domains
in both wheel and conda builds. If nightly looks ok after this, we
should continue to enable 3.12 for test and release according to 2.2.0
release matrix.
atalman added a commit that referenced this pull request Dec 20, 2023
Follow up of: #4834
Make sure we have python 3.12 in test env
atalman added a commit that referenced this pull request Dec 20, 2023
Cherry Pick of #4834
Add 3.12 to test environment

---------

Co-authored-by: Huy Do <[email protected]>
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.

4 participants