Skip to content

Install hpp headers for CPP Extensions#9182

Closed
fmassa wants to merge 1 commit intopytorch:masterfrom
fmassa:install_hpp
Closed

Install hpp headers for CPP Extensions#9182
fmassa wants to merge 1 commit intopytorch:masterfrom
fmassa:install_hpp

Conversation

@fmassa
Copy link
Member

@fmassa fmassa commented Jul 5, 2018

With the Cppzation of a few files in TH/THC, the CPP extensions got broken whenever the user uses feature from THC in their files, when pytorch is installed via python setup.py install.

This addresses issues such as

/home/me/.conda/envs/pytorch/lib/python3.6/site-packages/torch/lib/include/THC/THCDeviceTensorUtils.cuh:5:25: fatal error: THCTensor.hpp: No such file or directory

@fmassa fmassa added the module: cpp-extensions Related to torch.utils.cpp_extension label Jul 5, 2018
@fmassa fmassa requested a review from goldsborough July 5, 2018 12:45
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.

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

goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary:
With the Cppzation of a few files in `TH`/`THC`, the CPP extensions got broken whenever the user uses feature from `THC` in their files, when pytorch is installed via `python setup.py install`.

This addresses issues such as
```
/home/me/.conda/envs/pytorch/lib/python3.6/site-packages/torch/lib/include/THC/THCDeviceTensorUtils.cuh:5:25: fatal error: THCTensor.hpp: No such file or directory
```
Closes pytorch#9182

Reviewed By: soumith

Differential Revision: D8734581

Pulled By: fmassa

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

Labels

module: cpp-extensions Related to torch.utils.cpp_extension

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants