pytorch
50fd20b6 - fix bug on setup.py to include header files on caffe2/utils/math (#28869)

Commit
5 years ago
fix bug on setup.py to include header files on caffe2/utils/math (#28869) Summary: This problem is from issue [https://github.com/pytorch/pytorch/issues/28753](https://github.com/pytorch/pytorch/issues/28753) The header files on directories`math` and `threadpool` should be included on the built package because they are included on the other header files, such as on file `torch/include/caffe2/utils/math.h` ``` #include "caffe2/core/common.h" #include "caffe2/core/types.h" #include "caffe2/utils/math/broadcast.h" #include "caffe2/utils/math/elementwise.h" #include "caffe2/utils/math/reduce.h" #include "caffe2/utils/math/transpose.h" #include "caffe2/utils/math/utils.h" ``` But the `setup.py` doesn't include the header files on `master` branch. The header files on `utils` directory of a built `torch` package are the following: ``` > ls include/caffe2/utils bench_utils.h conversions.h eigen_utils.h map_utils.h murmur_hash3.h proto_wrap.h smart_tensor_printer.h cast.h cpuid.h filler.h math-detail.h proto_convert.h signal_handler.h string_utils.h cblas.h cpu_neon.h fixed_divisor.h math.h proto_utils.h simple_queue.h zmq_helper.h ``` Pull Request resolved: https://github.com/pytorch/pytorch/pull/28869 Differential Revision: D18226319 Pulled By: soumith fbshipit-source-id: 51575ddc559181c069b3324aa9b2d1669310ba25
Author
Parents
Loading