Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/125045
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 886dbcc with merge base 5585138 ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
@r-barnes Do you mind if I merge this first? |
|
@pytorchmergebot merge -i |
Merge failedReason: Approvers from one of the following sets are needed:
|
|
@pytorchbot merge |
Merge failedReason: Approvers from one of the following sets are needed:
|
|
@ezyang Need your approval before merge. |
|
we're discussing with @r-barnes what the best way to coordinate this with internal diff train is right now |
|
We're going to be good to merge this. I'll do so in the morning. Thanks for your help, @cyyever ! |
|
@r-barnes When you are ready, help issue the merge command. |
|
@pytorchbot merge |
Merge failedReason: This PR needs a If not, please add the To add a label, you can comment to pytorchbot, for example For more information, see Details for Dev Infra teamRaised by workflow job |
|
@pytorchbot merge |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
|
Thanks, @cyyever ! |
This PR tries to decompose pytorch#122527 into a smaller one. Caffe2 image and video folders are removed along with the related CMake code. To be noted, this was inspired and is co-dev with @r-barnes. Pull Request resolved: pytorch#125045 Approved by: https://github.com/eqy, https://github.com/albanD
|
@kit1980 I see. |
This PR tries to decompose #122527 into a smaller one. Caffe2 image and video folders are removed along with the related CMake code.
To be noted, this was inspired and is co-dev with @r-barnes.
cc @albanD