-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Refactor loop.setup_data
with utility functions
#16918
Conversation
loop.setup_data
with utility functions
⚡ 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: Azure HPU
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 🟢 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 #16918 +/- ##
=========================================
- Coverage 82% 59% -23%
=========================================
Files 437 414 -23
Lines 31755 31481 -274
=========================================
- Hits 25993 18681 -7312
- Misses 5762 12800 +7038 |
What does this PR do?
Follow-up to #16726
This PR is pure refactoring into smaller functional utilities that we share. No intended logic changes
cc @Borda @justusschock @awaelchli @carmocca