Skip to content

link libgloo_cuda.a explictly from setup.py#12951

Closed
anderspapitto wants to merge 1 commit intopytorch:masterfrom
anderspapitto:link-gloo
Closed

link libgloo_cuda.a explictly from setup.py#12951
anderspapitto wants to merge 1 commit intopytorch:masterfrom
anderspapitto:link-gloo

Conversation

@anderspapitto
Copy link
Copy Markdown
Contributor

rather than pass a list through a text file

Comment thread setup.py Outdated

This comment was marked as off-topic.

@anderspapitto anderspapitto force-pushed the link-gloo branch 3 times, most recently from ea9c880 to 9639740 Compare October 23, 2018 17:27
Comment thread test/cpp/api/dataloader.cpp Outdated

This comment was marked as off-topic.

Copy link
Copy Markdown
Collaborator

@soumith soumith left a comment

Choose a reason for hiding this comment

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

lgtm, remove bottom commit

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.

anderspapitto is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

rather than pass a list through a text file
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.

anderspapitto is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@anderspapitto anderspapitto deleted the link-gloo branch October 25, 2018 04:33
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
rather than pass a list through a text file
Pull Request resolved: pytorch#12951

Differential Revision: D10528309

Pulled By: anderspapitto

fbshipit-source-id: d94befcd61b6304815859694b623046f256462df
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