Skip to content

[Build] Use open-source NCCL2 in PyTorch #12359

Closed
teng-li wants to merge 3 commits intopytorch:masterfrom
teng-li:nccl_submodule
Closed

[Build] Use open-source NCCL2 in PyTorch #12359
teng-li wants to merge 3 commits intopytorch:masterfrom
teng-li:nccl_submodule

Conversation

@teng-li
Copy link
Contributor

@teng-li teng-li commented Oct 5, 2018

  • Removed the old nccl file
  • Make open-source NCCL a submodule
  • CMake to make NCCL itself

NCCL2 now is in the default build.

@teng-li
Copy link
Contributor Author

teng-li commented Oct 5, 2018

@pytorchbot retest this please

1 similar comment
@teng-li
Copy link
Contributor Author

teng-li commented Oct 5, 2018

@pytorchbot retest this please

Copy link
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.

teng-li has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
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.

teng-li has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

zdevito pushed a commit to zdevito/ATen that referenced this pull request Oct 9, 2018
Summary:
- Removed the old nccl file
- Make open-source NCCL a submodule
- CMake to make NCCL itself

NCCL2 now is in the default build.
Pull Request resolved: pytorch/pytorch#12359

Reviewed By: orionr, yns88

Differential Revision: D10219665

Pulled By: teng-li

fbshipit-source-id: 134ff47057512ba617b48bf390c1c816fff3f881
@ezyang ezyang added the merged label Jun 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants