Skip to content

[reland]Enable distributed package on windows, Gloo backend supported only#45335

Closed
gunandrose4u wants to merge 67 commits intopytorch:masterfrom
gunandrose4u:enable_win_dist_v2
Closed

[reland]Enable distributed package on windows, Gloo backend supported only#45335
gunandrose4u wants to merge 67 commits intopytorch:masterfrom
gunandrose4u:enable_win_dist_v2

Conversation

@gunandrose4u
Copy link
Copy Markdown
Contributor

Fixes #{issue number}
This is resubmit for PR #42897 . Together with fix for Windows build issue introduced by PR #44344 .

gunandrose4u and others added 30 commits August 17, 2020 11:27
Update my fork from Pytorch repo
Rebase from pytorch/pytorch master
Rebase latest code from master branch of PyTorch repo
Rebase to latest commits
Rebase to latest commits
Rebase to latest commit
Rebase to latest commits
Rebase to latest commits
Rebase to latest commits
@facebook-github-bot facebook-github-bot added the oncall: jit Add this issue/PR to JIT oncall triage queue label Sep 25, 2020
Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@mrshenli has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@gunandrose4u gunandrose4u changed the title Fix Windows build failure after DDP PR merged [reland]Enable distributed package on windows, Gloo backend supported only Sep 25, 2020
@zou3519 zou3519 added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Sep 25, 2020
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 25, 2020

Codecov Report

Merging #45335 into master will decrease coverage by 0.02%.
The diff coverage is 38.18%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #45335      +/-   ##
==========================================
- Coverage   68.06%   68.04%   -0.03%     
==========================================
  Files         393      393              
  Lines       50987    51024      +37     
==========================================
+ Hits        34705    34719      +14     
- Misses      16282    16305      +23     
Impacted Files Coverage Δ
.../testing/_internal/distributed/distributed_test.py 30.62% <9.67%> (-0.23%) ⬇️
torch/testing/_internal/common_distributed.py 67.53% <62.50%> (-0.47%) ⬇️
torch/testing/_internal/common_utils.py 77.20% <66.66%> (-0.03%) ⬇️
torch/distributed/rendezvous.py 84.54% <80.00%> (-0.90%) ⬇️
torch/testing/_internal/dist_utils.py 34.28% <100.00%> (+0.95%) ⬆️
...ternal/distributed/ddp_under_dist_autograd_test.py 31.84% <100.00%> (ø)
torch/testing/_internal/expecttest.py 78.57% <0.00%> (+1.02%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 99e0a87...8404d1e. Read the comment docs.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@mrshenli merged this pull request in f07ac6a.

facebook-github-bot pushed a commit that referenced this pull request Apr 29, 2021
Summary:
Fixes test failure after #56598

Introduced by #45335.

Pull Request resolved: #57000

Reviewed By: zou3519

Differential Revision: D28030360

Pulled By: seemethere

fbshipit-source-id: 4871d51e6b80dceef8bf95c6c658441287575f63
krshrimali pushed a commit to krshrimali/pytorch that referenced this pull request May 19, 2021
Summary:
Fixes test failure after pytorch#56598

Introduced by pytorch#45335.

Pull Request resolved: pytorch#57000

Reviewed By: zou3519

Differential Revision: D28030360

Pulled By: seemethere

fbshipit-source-id: 4871d51e6b80dceef8bf95c6c658441287575f63
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged oncall: jit Add this issue/PR to JIT oncall triage queue open source triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants