Cleanup DataLoader 2 #1329
Cleanup DataLoader 2 #1329
Conversation
facebook-github-bot
added
the
CLA Signed
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/data/1329
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 4b13f2f with merge base 6b762ba ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
There was a problem hiding this comment.
Do other references to torchdata.datapipes in this file need to stay?
There was a problem hiding this comment.
I think we'll just delete the whole thing in one go, i wanted to try and split it up into 2-ish PRs (dlv2 and datapipes) so we can at least have some partial rollback if necessary
There was a problem hiding this comment.
Cool, in that case maybe we want to update the PR title?
Accepting, since tests seem to pass (except some Meta specific bots).
|
@andrewkho has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
1 similar comment
|
@andrewkho has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
@andrewkho has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
andrewkho
force-pushed
the
andrewkh/cleanup-datapipes-dlv2
branch
from
4b13f2f
to
edff40a
Compare
|
@andrewkho has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
@andrewkho has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Starting off the cleanup by deleting DataLoader2, CI files, and a bunch of tests