Skip to content

Conversation

cyyever
Copy link
Contributor

@cyyever cyyever commented Oct 9, 2024

Description

Python 3.8 has reached End Of Life.

Motivation and Context

@cyyever cyyever requested a review from a team as a code owner October 9, 2024 01:11
Signed-off-by: cyy <cyyever@outlook.com>
Copy link
Member

@justinchuby justinchuby 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 need archinfra-SIG consensus. cc @gramalingam @liqunfu @andife

Copy link

codecov bot commented Oct 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 57.41%. Comparing base (c0c18d0) to head (47c4b70).
Report is 231 commits behind head on main.

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.
📢 Have feedback on the report? Share it here.

@andife
Copy link
Member

andife commented Oct 9, 2024

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)

@andife andife changed the title Remove python 3.8 workflows Remove python 3.8 workflows (python 3.8 is eol) Oct 9, 2024
@justinchuby
Copy link
Member

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.

@andife
Copy link
Member

andife commented Oct 10, 2024

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>
@andife andife added the run release CIs Use this label to trigger release tests in CI label Dec 20, 2024
@andife
Copy link
Member

andife commented Dec 20, 2024

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.

@xadupre @gramalingam @justinchuby

@andife andife self-requested a review December 20, 2024 12:22
@andife andife temporarily deployed to testpypi_onnxweekly December 23, 2024 06:15 — with GitHub Actions Inactive
@andife andife added this pull request to the merge queue Dec 23, 2024
Merged via the queue into onnx:main with commit b50fb87 Dec 23, 2024
76 checks passed
@cyyever cyyever deleted the no_py_38 branch December 23, 2024 14:21
@ramkrishna2910 ramkrishna2910 temporarily deployed to testpypi_onnxweekly May 12, 2025 22:55 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
run release CIs Use this label to trigger release tests in CI topic: bc breaking
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants