Skip to content

Conversation

@atalman
Copy link
Contributor

@atalman atalman commented Jul 30, 2024

Removing python 3.8 support in nightly builds. As per PR: #120718

@atalman atalman requested a review from a team as a code owner July 30, 2024 13:04
@pytorch-bot
Copy link

pytorch-bot bot commented Jul 30, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/132138

Note: Links to docs will display an error until the docs builds have been completed.

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

@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Jul 30, 2024
Copy link
Collaborator

@albanD albanD left a comment

Choose a reason for hiding this comment

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

Do we want ciflow/binaries to make sure nothing weird still depend on these?

Otherwise sounds good

@albanD albanD added topic: bc breaking topic category release notes: python_frontend python frontend release notes category and removed topic: not user facing topic category labels Jul 30, 2024
@ZainRizvi ZainRizvi added the ciflow/binaries Trigger all binary build and upload jobs on the PR label Jul 30, 2024
Copy link
Contributor

@huydhn huydhn left a comment

Choose a reason for hiding this comment

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

LGTM! As ExecuTorch build doesn't work with 3.8, we can ping ExecuTorch folks about this and pytorch/test-infra#5517 so that they don't need to figure out a way to support that version anymore (cc @guangy10 @dbort)

@kit1980
Copy link
Contributor

kit1980 commented Jul 30, 2024

ping ExecuTorch folks about this

Thanks for pointing this out. I was actually looking at implementing matrix without 3.8 for ET.

@atalman
Copy link
Contributor Author

atalman commented Jul 31, 2024

@pytorchmergebot merge -f "lint sanity check is expected"

@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

malfet added a commit that referenced this pull request Sep 30, 2024
By migrating some of the workflows to Python-3.9 as 3.8 has been deprecated by #132138
@malfet malfet mentioned this pull request Sep 30, 2024
@pytorchbot pytorchbot mentioned this pull request Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/binaries Trigger all binary build and upload jobs on the PR Merged release notes: python_frontend python frontend release notes category topic: bc breaking topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants