Skip to content
This repository has been archived by the owner on Sep 12, 2018. It is now read-only.

make error #41

Open
kangdongh opened this issue May 24, 2017 · 1 comment
Open

make error #41

kangdongh opened this issue May 24, 2017 · 1 comment

Comments

@kangdongh
Copy link

Hello, recently I've got following error.

[ 7%] Building CXX object CMakeFiles/thpp.dir/TensorSerialization.cpp.o In file included from thpp/detail/TensorGeneric.h:1:0, from /home/kangdongh/torch/install/include/TH/THGenerateFloatTypes.h:10, from /home/kangdongh/torch/install/include/TH/THGenerateAllTypes.h:10, from /home/kangdongh/Torch/depend/thpp/thpp/../thpp/detail/Tensor.h:30, from /home/kangdongh/Torch/depend/thpp/thpp/../thpp/Tensor.h:16, from /home/kangdongh/Torch/depend/thpp/thpp/TensorSerialization.cpp:11: /home/kangdongh/Torch/depend/thpp/thpp/../thpp/detail/TensorGeneric.h: In static member function ??static void thpp::detail::TensorOps<thpp::Tensor<float> >::_max(THFloatTensor*, THLongTensor*, THFloatTensor*, int)??: /home/kangdongh/Torch/depend/thpp/thpp/../thpp/detail/TensorGeneric.h:191:50: error: too few arguments to function ??void THFloatTensor_max(THFloatTensor*, THLongTensor*, THFloatTensor*, int, int)?? return THTensor_(max)(values, indices, t, dim); ^ In file included from /home/kangdongh/torch/install/include/TH/THStorage.h:4:0, from /home/kangdongh/Torch/depend/thpp/thpp/../thpp/detail/Storage.h:14, from /home/kangdongh/Torch/depend/thpp/thpp/../thpp/Storage.h:30, from /home/kangdongh/Torch/depend/thpp/thpp/../thpp/Tensor.h:14, from /home/kangdongh/Torch/depend/thpp/thpp/TensorSerialization.cpp:11:
(... many lines are omitted)

w/ 16.04 LTS with g++ 5.4.0
I guess that this error is due to conflicting w/ current torch version, then how can I solve this problem?

thanks.

codeVerySlow added a commit to codeVerySlow/thpp that referenced this issue May 26, 2017
the torch7 had Add a keepdim parameter for reduction functions over a single dimension.For detail torch/torch7@c5e5487
so that commit make thpp can't compile.
@gbolin
Copy link

gbolin commented Feb 12, 2018

so the final solution is to change the Torch source code in THTensorMath.h and THTensorMath.c?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants