[GitHub] ZiyueHuang commented on issue #8338: master branch cannot build on centos 7 with cuda-8.0

2018-01-16 Thread GitBox
ZiyueHuang commented on issue #8338: master branch cannot build on centos 7 
with cuda-8.0
URL: 
https://github.com/apache/incubator-mxnet/issues/8338#issuecomment-357890465
 
 
   Good catch! Glad to see this is fixed. Thanks @zhreshold 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] ZiyueHuang commented on issue #8338: master branch cannot build on centos 7 with cuda-8.0

2017-11-01 Thread GitBox
ZiyueHuang commented on issue #8338: master branch cannot build on centos 7 
with cuda-8.0
URL: 
https://github.com/apache/incubator-mxnet/issues/8338#issuecomment-341131550
 
 
   @mseeger Thanks! The issue is solved.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] ZiyueHuang commented on issue #8338: master branch cannot build on centos 7 with cuda-8.0

2017-10-21 Thread GitBox
ZiyueHuang commented on issue #8338: master branch cannot build on centos 7 
with cuda-8.0
URL: 
https://github.com/apache/incubator-mxnet/issues/8338#issuecomment-338387761
 
 
   I tried the latest master but it fails too.
   
   I replaced smooth_l1_* with returning DType(0), the warnings disappear, but 
there are still error message,
   ```
   /home/hanfeng/zyh/mxnet/mshadow/mshadow/././././cuda/tensor_gpu-inl.cuh(75): 
error: expression preceding parentheses of apparent call must have 
(pointer-to-) function type
   
   /home/hanfeng/zyh/mxnet/mshadow/mshadow/././././cuda/tensor_gpu-inl.cuh(75): 
error: expression preceding parentheses of apparent call must have 
(pointer-to-) function type
   
   2 errors detected in the compilation of 
"/tmp/tmpxft_964a_-24_activation.compute_30.cpp2.i".
   make: *** [build/src/operator/activation_gpu.o] Error 2
   ```


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] ZiyueHuang commented on issue #8338: master branch cannot build on centos 7 with cuda-8.0

2017-10-19 Thread GitBox
ZiyueHuang commented on issue #8338: master branch cannot build on centos 7 
with cuda-8.0
URL: 
https://github.com/apache/incubator-mxnet/issues/8338#issuecomment-338100092
 
 
   @mseeger Yes, if I checkout to 9f97dac76e43b2ca0acb09a4ff96d416e9edea60, the 
one just before your commit. It can build successfully on my machine without 
these warnings. I also have no idea where uint8_t comes from.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] ZiyueHuang commented on issue #8338: master branch cannot build on centos 7 with cuda-8.0

2017-10-18 Thread GitBox
ZiyueHuang commented on issue #8338: master branch cannot build on centos 7 
with cuda-8.0
URL: 
https://github.com/apache/incubator-mxnet/issues/8338#issuecomment-337790437
 
 
   Seems that @wa1618i got the same problem in 
https://github.com/apache/incubator-mxnet/issues/8333. Have you got any 
solutions so far? @wa1618i


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] ZiyueHuang commented on issue #8338: master branch cannot build on centos 7 with cuda-8.0

2017-10-18 Thread GitBox
ZiyueHuang commented on issue #8338: master branch cannot build on centos 7 
with cuda-8.0
URL: 
https://github.com/apache/incubator-mxnet/issues/8338#issuecomment-337789562
 
 
   Thanks for your suggestions.
   
   Seems wired. It can build successfully on ubuntu. Why centos cannot?
   
   Yes, the error messages are complete. 
   
   Warnings are not fixed by reverting `smooth_l1`, I change these codes to
   ```
   struct smooth_l1_loss {
  // a is x, b is sigma2
  template
  MSHADOW_XINLINE static DType Map(DType a, DType b) {
b *= b;
if (a > (1.0f) / b) {
  return a - (0.5f) / b;
} else if (a < (-1.0f) / b) {
  return -a - (0.5f) / b;
} else {
  return (0.5f) * a * a * b;
}
  }
};  // struct smooth_l1_loss
   
   struct smooth_l1_gradient {
  // a is x, b is sigma2
  template
  MSHADOW_XINLINE static DType Map(DType a, DType b) {
b *= b;
if (a > (1.0f) / b) {
  return (1.0f);
} else if (a < (-1.0f) / b) {
  return DType(-1.0f);
} else {
  return b * a;
}
  }
};  // struct smooth_l1_derivative
   ```
   Below is the whole chunk of warning and error messages,
   ```
   src/operator/tensor/./../mshadow_op.h(1093): warning: floating-point value 
does not fit in required integral type
 detected during:
   instantiation of "DType 
mxnet::op::mshadow_op::smooth_l1_gradient::Map(DType, DType) [with 
DType=uint8_t]"
   /home/hanfeng/zyh/build/mshadow/mshadow/././expr_engine-inl.h(131): here
   instantiation of "DType 
mshadow::expr::Plan, 
DType>::Eval(mshadow::index_t, mshadow::index_t) const [with 
OP=mxnet::op::mshadow_op::smooth_l1_gradient, TA=mshadow::Tensor, TB=mshadow::expr::ScalarExp, etype=1, DType=uint8_t]"
   /home/hanfeng/zyh/build/mshadow/mshadow/././expr_engine-inl.h(131): here
   instantiation of "DType 
mshadow::expr::Plan, 
DType>::Eval(mshadow::index_t, mshadow::index_t) const [with 
OP=mshadow::op::mul, TA=mshadow::Tensor, 
TB=mshadow::expr::BinaryMapExp, mshadow::expr::ScalarExp, 
uint8_t, 1>, etype=1, DType=uint8_t]"
   /home/hanfeng/zyh/build/mshadow/mshadow/././././cuda/tensor_gpu-inl.cuh(75): 
here
   instantiation of "void 
mshadow::cuda::MapPlanProc(DstPlan, 
mshadow::index_t, mshadow::Shape<2>, Plan, int) [with 
Saver=mshadow::sv::saveto, 
DstPlan=mshadow::expr::Plan, uint8_t>, 
Plan=mshadow::expr::Plan, 
mshadow::expr::BinaryMapExp, mshadow::expr::ScalarExp, 
uint8_t, 1>, uint8_t, 1>, uint8_t>, block_dim_bits=8]"
   /home/hanfeng/zyh/build/mshadow/mshadow/././././cuda/tensor_gpu-inl.cuh(83): 
here
   instantiation of "void 
mshadow::cuda::MapPlanKernel(DstPlan, 
mshadow::index_t, mshadow::Shape<2>, Plan) [with Saver=mshadow::sv::saveto, 
block_dim_bits=8, DstPlan=mshadow::expr::Plan, uint8_t>, 
Plan=mshadow::expr::Plan, 
mshadow::expr::BinaryMapExp, mshadow::expr::ScalarExp, 
uint8_t, 1>, uint8_t, 1>, uint8_t>]"
   
/home/hanfeng/zyh/build/mshadow/mshadow/././././cuda/tensor_gpu-inl.cuh(109): 
here
   instantiation of "void 
mshadow::cuda::MapPlan(mshadow::expr::Plan, const mshadow::expr::Plan &, mshadow::Shape<2>, cudaStream_t) 
[with Saver=mshadow::sv::saveto, DstExp=mshadow::Tensor, E=mshadow::expr::BinaryMapExp, 
mshadow::expr::BinaryMapExp, mshadow::expr::ScalarExp, 
uint8_t, 1>, uint8_t, 1>, DType=uint8_t]"
   /home/hanfeng/zyh/build/mshadow/mshadow/./tensor_gpu-inl.h(115): here
   instantiation of "void 
mshadow::MapExp(mshadow::TRValue *, const mshadow::expr::Exp &) [with 
Saver=mshadow::sv::saveto, R=mshadow::Tensor, dim=1, 
DType=uint8_t, E=mshadow::expr::BinaryMapExp, 
mshadow::expr::BinaryMapExp, mshadow::expr::ScalarExp, 
uint8_t, 1>,