Skip to content

Commit

Permalink
fixed bug in setup.py
Browse files Browse the repository at this point in the history
  • Loading branch information
Saeed Maleki committed Aug 23, 2019
1 parent eda4e4e commit 4491b32
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 1 deletion.
5 changes: 4 additions & 1 deletion MANIFEST.in
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
recursive-include * *.h *.hpp *.cc *.md
recursive-include * *.h *.hpp *.cc *.md *.ipp

include LICENSE horovod.lds horovod.exp
prune .eggs
Expand All @@ -19,3 +19,6 @@ exclude third_party/eigen/Eigen/src/SparseCholesky/*
graft third_party/gloo/cmake
recursive-include third_party/gloo CMakeLists.txt
recursive-include third_party/gloo *.in

# include cmake related files for msallreduce kernel
graft horovod/common/ops/cuda/
3 changes: 3 additions & 0 deletions horovod/common/ops/msallreduce_cuda_operations.cc
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,9 @@ cudaDataType_t CublasContext::GetCublasDataType(T* variable) {
}

//thread_local cublasHandle_t MsCudaAllreduceOp::cublas_Handle;
thread_local double* MsCudaAllreduceOp::device_normsq_memory_a;
thread_local double* MsCudaAllreduceOp::device_normsq_memory_b;
thread_local double* MsCudaAllreduceOp::device_dot_product_memory;

MsCudaAllreduceOp::MsCudaAllreduceOp(MPIContext* mpi_context, CUDAContext* cuda_context, HorovodGlobalState* global_state)
: MsAllreduceOp(mpi_context, global_state), cuda_context_(cuda_context) {
Expand Down
1 change: 1 addition & 0 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -661,6 +661,7 @@ def get_common_options(build_ext):
if have_cuda:
MACROS += [('HAVE_CUDA', '1')]
INCLUDES += cuda_include_dirs
INCLUDES += ['horovod/common/ops/cuda']
SOURCES += ['horovod/common/ops/cuda_operations.cc',
'horovod/common/ops/mpi_cuda_operations.cc',
'horovod/common/ops/msallreduce_cuda_operations.cc']
Expand Down

0 comments on commit 4491b32

Please sign in to comment.