Skip to content

Rollback ddp tutorial fix #1641

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

Closed
wants to merge 6 commits into from

Conversation

bowangbj
Copy link
Contributor

@bowangbj bowangbj commented Aug 6, 2021

Subsume: #1621

Context: #1621 base branch was messed up due to use of ghstack tool.

@bowangbj
Copy link
Contributor Author

bowangbj commented Aug 6, 2021

Hey Shen, Brian -- this is follow up of #1621
This PR will subsume it since before I was using ghstack which messed up the base branch. Please review this one instead.

@netlify
Copy link

netlify bot commented Aug 6, 2021

✔️ Deploy Preview for pytorch-tutorials-preview ready!

🔨 Explore the source changes: e0214e9

🔍 Inspect the deploy log: https://app.netlify.com/sites/pytorch-tutorials-preview/deploys/611a948597d93c0007dbc445

😎 Browse the preview: https://deploy-preview-1641--pytorch-tutorials-preview.netlify.app/intermediate/ddp_tutorial

@bowangbj bowangbj requested a review from rohan-varma August 6, 2021 18:24
@brianjo
Copy link
Contributor

brianjo commented Aug 16, 2021

@bowangbj Do we still need to land this? If so, I'll do so after confirm. Thanks!

@bowangbj
Copy link
Contributor Author

@bowangbj Do we still need to land this? If so, I'll do so after confirm. Thanks!

My hunch is that we still need it. @mrshenli Shen, can you take a quick review? Thanks!

@brianjo
Copy link
Contributor

brianjo commented Sep 7, 2021

Hey, I'm going to close this as I think we took care of it in another PR. If you want to change the file, can you do a new PR? Thanks!

@brianjo brianjo closed this Sep 7, 2021
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.

3 participants