hughperkins / cltorch

An OpenCL backend for torch.
Other
291 stars 26 forks source link

Cannot install cltorch on a system that previously ran it fine #66

Closed ViliusT closed 8 years ago

ViliusT commented 8 years ago

First of all - amazing work. I've encountered an error when installing this on a fresh system (exact same hardware, and most of the software was used before and worked fine - maybe slightly older versions)


[ 52%] Building CXX object src/lib/CMakeFiles/THCl.dir/THClTensorIndex.cpp.o
/home/vilius/cltorch/src/lib/THClTensorMath.cpp: In function ‘void THClTensor_minall_gpu(THClState*, THClTensor*, THClTensor*)’:
/home/vilius/cltorch/src/lib/THClTensorMath.cpp:268:19: error: ‘THInf’ was not declared in this scope
           (float) THInf, self->storage->wrapper)) {
                   ^
/home/vilius/cltorch/src/lib/THClTensorMath.cpp: In function ‘void THClTensor_maxall_gpu(THClState*, THClTensor*, THClTensor*)’:
/home/vilius/cltorch/src/lib/THClTensorMath.cpp:282:20: error: ‘THInf’ was not declared in this scope
           (float) -THInf, self->storage->wrapper)) {
                    ^
/home/vilius/cltorch/src/lib/THClTensorMath.cpp: In function ‘float THClTensor_minall(THClState*, THClTensor*)’:
/home/vilius/cltorch/src/lib/THClTensorMath.cpp:290:23: error: ‘THInf’ was not declared in this scope
   float val = (float) THInf;
                       ^
/home/vilius/cltorch/src/lib/THClTensorMath.cpp: In function ‘float THClTensor_maxall(THClState*, THClTensor*)’:
/home/vilius/cltorch/src/lib/THClTensorMath.cpp:312:24: error: ‘THInf’ was not declared in this scope
   float val = (float) -THInf;
                        ^
make[2]: *** [src/lib/CMakeFiles/THCl.dir/THClTensorMath.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [src/lib/CMakeFiles/THCl.dir/all] Error 2
make: *** [all] Error 2

Any ideas why this could be happening?

ViliusT commented 8 years ago

Spent last 3 days thinking this was driver/hardware issue, and didn't see the pull request. #65 fixed this.

hughperkins commented 8 years ago

Thanks! Sorry about that. Glad that you and kosmonautisk found out the issue and fixed it. Thanks! :-)