-
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
Remove the lightning app code #20039
Conversation
3d67684
to
5631299
Compare
0cfbb06
to
875766c
Compare
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.
🎉
⚡ 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 ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #20039 +/- ##
==========================================
+ Coverage 83% 90% +7%
==========================================
Files 427 266 -161
Lines 35460 22939 -12521
==========================================
- Hits 29599 20720 -8879
+ Misses 5861 2219 -3642 |
* updated v2.3.3 * Remove lightning-app licenses Xref Lightning-AI/pytorch-lightning#20039 --------- Co-authored-by: Wei Ji <23487320+weiji14@users.noreply.github.com>
What does this PR do?
Deletes the source code under src/lightning/app, the corresponding tests, docs, ci workflows and related configuration.
Fixes GHSA-mr7h-w2qc-ffc2
Fixes CVE-2024-5980
📚 Documentation preview 📚: https://pytorch-lightning--20039.org.readthedocs.build/en/20039/
cc @Borda @carmocca @justusschock @awaelchli