-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Remove python 3.8 workflows (python 3.8 is eol) #6434
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
Signed-off-by: cyy <cyyever@outlook.com>
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 need archinfra-SIG consensus. cc @gramalingam @liqunfu @andife
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6434 +/- ##
========================================
Coverage 57.41% 57.41%
========================================
Files 507 507
Lines 31538 31538
Branches 3549 3063 -486
========================================
Hits 18106 18106
Misses 12583 12583
Partials 849 849 ☔ View full report in Codecov by Sentry. |
After @gramalingam comment in #6298, I would still keep version 3.8. Pytorch currently still allows this. (When we release maybe every 3 months, which would be end of December we could reevaluate that again? I wonder if we need new version faster because of the windows build problems?) But I am also open to removing it. Just don't see the need yet. (Possibly it will come very soon, when some dependencies stop support, and it would mean an increased effort for us to support python 3.8) |
PyTorch has dropped 3.8 for the up coming 2.6 release. That would be similar timing when we release 1.18 so I think that’s ok. |
The https://github.com/pytorch/pytorch/blob/main/RELEASE.md#release-compatibility-matrix and release https://github.com/pytorch/pytorch/blob/main/RELEASE.md#release-compatibility-matrix looks nice. In my opinion we should have something like that publicly available. From my experience with Onnx, a release every 4 months might be realistic? |
Signed-off-by: Andreas Fehlner <fehlner@arcor.de>
Now that pytorch and onnxruntime have discontinued support, I can well imagine removing 3.8. Now that we have added 3.13, this would also reduce CI runtimes. |
Description
Python 3.8 has reached End Of Life.
Motivation and Context