Skip to content

Conversation

NivekT
Copy link
Contributor

@NivekT NivekT commented Sep 27, 2022

Stack from ghstack:

CI is expected to fail for now. This should be merged only after pytorch/pytorch#85667 has been merged into nightly and internal.

NivekT added a commit that referenced this pull request Sep 27, 2022
@NivekT NivekT changed the base branch from gh/NivekT/1/base to main September 27, 2022 20:18
@NivekT NivekT changed the title Replace torch.utils.data.graph.traverse with traverse_dps Replace torch.utils.data.graph.traverse with traverse_dps Sep 27, 2022
@NivekT NivekT requested review from ejguan and pmeier September 27, 2022 20:19
…ps`"


CI is expected to fail for now. This should be merged only after pytorch/pytorch#85667 has been merged into nightly and internal.



[ghstack-poisoned]
NivekT added a commit that referenced this pull request Sep 27, 2022
Copy link
Contributor

@ejguan ejguan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Same as the comment in TorchData, let's merge it until nightly is updated

Copy link
Collaborator

@pmeier pmeier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @NivekT!

@pmeier pmeier merged commit 0fd4736 into main Sep 28, 2022
@pmeier pmeier deleted the gh/NivekT/1/head branch September 28, 2022 12:21
@github-actions
Copy link

Hey @pmeier!

You merged this PR, but no labels were added. The list of valid labels is available at https://github.com/pytorch/vision/blob/main/.github/process_commit.py

facebook-github-bot pushed a commit that referenced this pull request Sep 29, 2022
#6657)

Summary:
* Replace torch.utils.data.graph.traverse with traverse_dps

[ghstack-poisoned]

* Update on "Replace `torch.utils.data.graph.traverse` with `traverse_dps`"

CI is expected to fail for now. This should be merged only after pytorch/pytorch#85667 has been merged into nightly and internal.

[ghstack-poisoned]

Reviewed By: YosuaMichael

Differential Revision: D39885426

fbshipit-source-id: 3079b5ad4343bb3a4b2e514461472aced8fb6998

Co-authored-by: Philip Meier <github.pmeier@posteo.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants