-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Drop support for PyTorch 1.11 #18691
Conversation
for more information, see https://pre-commit.ci
⚡ Required checks status: All passing 🟢Groups summary🟢 pytorch_lightning: Tests workflow
These checks are required after the changes to 🟢 pytorch_lightning: Azure GPU
These checks are required after the changes to 🟢 pytorch_lightning: Benchmarks
These checks are required after the changes to 🟢 fabric: Docs
These checks are required after the changes to 🟢 pytorch_lightning: Docs
These checks are required after the changes to 🟢 pytorch_lightning: Docker
These checks are required after the changes to 🟢 lightning_fabric: CPU workflowThese checks are required after the changes to 🟢 lightning_fabric: Azure GPU
These checks are required after the changes to 🟢 mypy
These checks are required after the changes to 🟢 installThese checks are required after the changes to Thank you for your contribution! 💜
|
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #18691 +/- ##
=========================================
- Coverage 83% 55% -29%
=========================================
Files 428 423 -5
Lines 33546 33409 -137
=========================================
- Hits 27972 18273 -9699
- Misses 5574 15136 +9562 |
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.
over all lgtm, but would wait for @lantiga approval
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.
Nice, love the simplifications that come with deprecating 1.11
Typo in release notes: says 3.11 instead of 1.11 |
Thank you @adamjstewart |
What does this PR do?
With support for the new PyTorch 2.1 version, we're removing PyTorch 1.11 support too keep our 4-version support window (1.12, 1.13, 2.0, 2.1).
Previous PR: #16492
cc @carmocca @Borda @justusschock @awaelchli