KEMBAR78
[reland] torch/lib/c10d: Use torch_check instead of throwing runtime_error by rohan-varma · Pull Request #59918 · pytorch/pytorch · GitHub
Skip to content

Conversation

@rohan-varma
Copy link
Contributor

@rohan-varma rohan-varma commented Jun 13, 2021

Stack from ghstack:

Reland of #59684
Fix is to update ProcessGroupNCCLErrorsTest appropriately. CI all is in #59919

Differential Revision: D29081452

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Jun 13, 2021

💊 CI failures summary and remediations

As of commit c65bde5 (more details on the Dr. CI page):


  • 2/2 failures possibly* introduced in this PR
    • 2/2 non-scanned failure(s)

ci.pytorch.org: 1 failed


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

@facebook-github-bot facebook-github-bot added the oncall: distributed Add this issue/PR to distributed oncall triage queue label Jun 13, 2021
rohan-varma added a commit that referenced this pull request Jun 13, 2021
…error

Reland of #59684

Differential Revision: [D29081452](https://our.internmc.facebook.com/intern/diff/D29081452/)

ghstack-source-id: 131303057
Pull Request resolved: #59918
Copy link
Contributor

@cbalioglu cbalioglu left a comment

Choose a reason for hiding this comment

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

LGTM

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 580a20f.

@facebook-github-bot facebook-github-bot deleted the gh/rohan-varma/330/head branch June 18, 2021 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed Merged oncall: distributed Add this issue/PR to distributed oncall triage queue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants