Skip to content

Add a CUDAContext header include#9662

Closed
ezyang wants to merge 3 commits intopytorch:masterfrom
ezyang:pr/cuda-context-header
Closed

Add a CUDAContext header include#9662
ezyang wants to merge 3 commits intopytorch:masterfrom
ezyang:pr/cuda-context-header

Conversation

@ezyang
Copy link
Contributor

@ezyang ezyang commented Jul 20, 2018

No description provided.

ezyang added 2 commits July 20, 2018 14:10
Signed-off-by: Edward Z. Yang <ezyang@fb.com>
@goldsborough
Copy link
Contributor

goldsborough commented Jul 20, 2018

Sorry could you actually put #include <ATen/cuda/CUDAContext.h> in the torch/csrc/functions/comm.h (the header)? I think otherwise someone using the data_parallel building WITH_CUDA might not get that header included, since it's also not in ATen/ATen.h so I'm not sure how else it would end up in the include path. Maybe after CI passes or something

@goldsborough
Copy link
Contributor

I think this also causing the build failure

@goldsborough
Copy link
Contributor

I pushed with potential fix

@ezyang
Copy link
Contributor Author

ezyang commented Jul 21, 2018

I don't quite understand why the non-ROCm builds pass without it though :/

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.

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

jramseyer pushed a commit to jramseyer/pytorch that referenced this pull request Jul 30, 2018
Summary: Pull Request resolved: pytorch#9662

Differential Revision: D8945581

Pulled By: ezyang

fbshipit-source-id: 2fe0adc96456788579f7d6f1c4513fe45360c030
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary: Pull Request resolved: pytorch#9662

Differential Revision: D8945581

Pulled By: ezyang

fbshipit-source-id: 2fe0adc96456788579f7d6f1c4513fe45360c030
@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