[GitHub] szha closed issue #11501: Tracking issue for tests with fixed random seed

2018-07-12 Thread GitBox
szha closed issue #11501: Tracking issue for tests with fixed random seed
URL: https://github.com/apache/incubator-mxnet/issues/11501
 
 
   


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] szha opened a new issue #11741: test_sparse_ndarray.test_sparse_nd_save_load has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11741: test_sparse_ndarray.test_sparse_nd_save_load 
has fixed seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11741
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.


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] szha opened a new issue #11740: test_sparse_ndarray.test_sparse_nd_pickle has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11740: test_sparse_ndarray.test_sparse_nd_pickle has 
fixed seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11740
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11739: test_quantization.test_get_optimal_thresholds has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11739: test_quantization.test_get_optimal_thresholds 
has fixed seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11739
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11738: test_optimizer.test_signum has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11738: test_optimizer.test_signum has fixed seed that 
can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11738
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11736: test_optimizer.test_nadam has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11736: test_optimizer.test_nadam has fixed seed that 
can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11736
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11737: test_optimizer.test_nag has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11737: test_optimizer.test_nag has fixed seed that can 
mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11737
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11734: test_operator_gpu.test_psroipooling_with_type has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11734: test_operator_gpu.test_psroipooling_with_type 
has fixed seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11734
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11735: test_optimizer.test_ftml has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11735: test_optimizer.test_ftml has fixed seed that 
can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11735
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11733: test_operator_gpu.test_pooling_with_type has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11733: test_operator_gpu.test_pooling_with_type has 
fixed seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11733
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11732: test_operator_gpu.test_pooling_versions has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11732: test_operator_gpu.test_pooling_versions has 
fixed seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11732
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11731: test_operator_gpu.test_ifft has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11731: test_operator_gpu.test_ifft has fixed seed that 
can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11731
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11730: test_operator_gpu.test_fft has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11730: test_operator_gpu.test_fft has fixed seed that 
can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11730
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] NEWPLAN edited a comment on issue #11684: fix compile error in debug model

2018-07-12 Thread GitBox
NEWPLAN edited a comment on issue #11684: fix compile error in debug model
URL: https://github.com/apache/incubator-mxnet/pull/11684#issuecomment-404730258
 
 
   Hi,
   My environment:
   OS:  CentOS Linux release 7.3.1611 (Core)
   GCC: gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-28)
   CPU: Intel(R) Xeon(R) CPU E5-2620 v4 @ 2.10GHz
   DRAM:  64GB 2133MHZ
   G++: g++ (GCC) 4.8.5 20150623 (Red Hat 4.8.5-28)
   NVCC: Built on Fri_Sep__1_21:08:03_CDT_2017 + Cuda compilation tools, 
release 9.0, V9.0.176
   NCCL: 2.1
   CUDA: 9
   cuDNN: 7.0
   
   command for compiling:
   make -j $(nproc) USE_OPENCV=1 USE_BLAS=openblas USE_CUDA=1 
USE_CUDA_PATH=/usr/local/cuda USE_CUDNN=1 USE_DIST_KVSTORE=1 USE_NCCL=1 
USE_CPP_PACKAGE=1 DEBUG=1
   
   there are two errors:
   1: src/kvstore/./kvstore_nccl.h:345:28: error: ‘struct 
mxnet::kvstore::KVStoreNCCL::BufferEntry’ has no member named ‘ctx’
assert(root == buf.ctx().dev_id);
   When i fixed this error with uncomment the assert code and complie again, 
got the second error:
   
   a - build/src/kvstore/kvstore_utils_gpu.o
   /usr/lib/gcc/x86_64-redhat-linux/4.8.5/../../../../lib64/crti.o: In function 
`_init':
   (.init+0x7): relocation truncated to fit: R_X86_64_GOTPCREL against 
undefined symbol `__gmon_start__'
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::NumResponse(int)':
   
/usr/include/c++/4.8.2/x86_64-redhat-linux/bits/gthr-default.h:747:(.text+0x5): 
relocation truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::AddResponse(int, int)':
   
/usr/include/c++/4.8.2/x86_64-redhat-linux/bits/gthr-default.h:747:(.text+0x7d):
 relocation truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::WaitRequest(int)':
   
/usr/include/c++/4.8.2/x86_64-redhat-linux/bits/gthr-default.h:747:(.text+0xfe):
 relocation truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::Receiving()':
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/src/customer.cc:49:(.text+0x1d2):
 relocation truncated to fit: R_X86_64_GOTPCREL against symbol 
`std::string::_Rep::_S_empty_rep_storage@@GLIBCXX_3.4' defined in .bss section 
in /usr/lib/gcc/x86_64-redhat-linux/4.8.5/libstdc++.so
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::Receiving()':
   
/usr/include/c++/4.8.2/x86_64-redhat-linux/bits/gthr-default.h:747:(.text+0x2dd):
 relocation truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::Receiving()':
   /usr/include/c++/4.8.2/bits/stl_construct.h:102:(.text+0x46f): relocation 
truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   /usr/include/c++/4.8.2/bits/stl_construct.h:102:(.text+0x555): relocation 
truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   /usr/include/c++/4.8.2/bits/stl_construct.h:102:(.text+0x61a): relocation 
truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   /usr/include/c++/4.8.2/bits/stl_construct.h:102:(.text+0x6ce): relocation 
truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::Receiving()':
   
/usr/include/c++/4.8.2/x86_64-redhat-linux/bits/gthr-default.h:747:(.text+0x773):
 additional relocation overflows omitted from the output
   collect2: error: ld returned 1 exit status
   make: *** [lib/libmxnet.so] Error 1
   make: *** Waiting for unfinished jobs
   
/usr/lib/gcc/x86_64-redhat-linux/4.8.5/../../../../lib64/crt1.o:(.eh_frame+0x20):
 relocation truncated to fit: R_X86_64_PC32 against `.text'
   /usr/lib/gcc/x86_64-redhat-linux/4.8.5/../../../../lib64/crti.o: In function 
`_init':
   (.init+0x7): relocation truncated to fit: R_X86_64_GOTPCREL against 
undefined symbol `__gmon_start__'
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In 

[GitHub] NEWPLAN edited a comment on issue #11684: fix compile error in debug model

2018-07-12 Thread GitBox
NEWPLAN edited a comment on issue #11684: fix compile error in debug model
URL: https://github.com/apache/incubator-mxnet/pull/11684#issuecomment-404730258
 
 
   Hi,
   My environment:
   OS:  CentOS Linux release 7.3.1611 (Core)
   GCC: gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-28)
   CPU: Intel(R) Xeon(R) CPU E5-2620 v4 @ 2.10GHz
   DRAM:  64GB 2133MHZ
   G++: g++ (GCC) 4.8.5 20150623 (Red Hat 4.8.5-28)
   NVCC: Built on Fri_Sep__1_21:08:03_CDT_2017 + Cuda compilation tools, 
release 9.0, V9.0.176
   NCCL: 2.1
   CUDA: 9
   cuDNN: 7.0
   
   command for compiling:
   make -j $(nproc) USE_OPENCV=1 USE_BLAS=openblas USE_CUDA=1 
USE_CUDA_PATH=/usr/local/cuda USE_CUDNN=1 USE_DIST_KVSTORE=1 USE_NCCL=1 
USE_CPP_PACKAGE=1 DEBUG=1
   
   there are two errors:
   1: src/kvstore/./kvstore_nccl.h:345:28: error: ‘struct 
mxnet::kvstore::KVStoreNCCL::BufferEntry’ has no member named ‘ctx’
assert(root == buf.ctx().dev_id);
   When i fixed this error as uncommend and complie again, got the second error:
   
   a - build/src/kvstore/kvstore_utils_gpu.o
   /usr/lib/gcc/x86_64-redhat-linux/4.8.5/../../../../lib64/crti.o: In function 
`_init':
   (.init+0x7): relocation truncated to fit: R_X86_64_GOTPCREL against 
undefined symbol `__gmon_start__'
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::NumResponse(int)':
   
/usr/include/c++/4.8.2/x86_64-redhat-linux/bits/gthr-default.h:747:(.text+0x5): 
relocation truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::AddResponse(int, int)':
   
/usr/include/c++/4.8.2/x86_64-redhat-linux/bits/gthr-default.h:747:(.text+0x7d):
 relocation truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::WaitRequest(int)':
   
/usr/include/c++/4.8.2/x86_64-redhat-linux/bits/gthr-default.h:747:(.text+0xfe):
 relocation truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::Receiving()':
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/src/customer.cc:49:(.text+0x1d2):
 relocation truncated to fit: R_X86_64_GOTPCREL against symbol 
`std::string::_Rep::_S_empty_rep_storage@@GLIBCXX_3.4' defined in .bss section 
in /usr/lib/gcc/x86_64-redhat-linux/4.8.5/libstdc++.so
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::Receiving()':
   
/usr/include/c++/4.8.2/x86_64-redhat-linux/bits/gthr-default.h:747:(.text+0x2dd):
 relocation truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::Receiving()':
   /usr/include/c++/4.8.2/bits/stl_construct.h:102:(.text+0x46f): relocation 
truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   /usr/include/c++/4.8.2/bits/stl_construct.h:102:(.text+0x555): relocation 
truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   /usr/include/c++/4.8.2/bits/stl_construct.h:102:(.text+0x61a): relocation 
truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   /usr/include/c++/4.8.2/bits/stl_construct.h:102:(.text+0x6ce): relocation 
truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::Receiving()':
   
/usr/include/c++/4.8.2/x86_64-redhat-linux/bits/gthr-default.h:747:(.text+0x773):
 additional relocation overflows omitted from the output
   collect2: error: ld returned 1 exit status
   make: *** [lib/libmxnet.so] Error 1
   make: *** Waiting for unfinished jobs
   
/usr/lib/gcc/x86_64-redhat-linux/4.8.5/../../../../lib64/crt1.o:(.eh_frame+0x20):
 relocation truncated to fit: R_X86_64_PC32 against `.text'
   /usr/lib/gcc/x86_64-redhat-linux/4.8.5/../../../../lib64/crti.o: In function 
`_init':
   (.init+0x7): relocation truncated to fit: R_X86_64_GOTPCREL against 
undefined symbol `__gmon_start__'
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function 

[GitHub] szha opened a new issue #11728: test_operator_gpu.test_deformable_convolution_with_type has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11728: 
test_operator_gpu.test_deformable_convolution_with_type has fixed seed that can 
mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11728
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11729: test_operator_gpu.test_deformable_psroipooling_with_type has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11729: 
test_operator_gpu.test_deformable_psroipooling_with_type has fixed seed that 
can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11729
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11727: test_operator_gpu.test_convolution_with_type has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11727: test_operator_gpu.test_convolution_with_type 
has fixed seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11727
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] NEWPLAN commented on issue #11684: fix compile error in debug model

2018-07-12 Thread GitBox
NEWPLAN commented on issue #11684: fix compile error in debug model
URL: https://github.com/apache/incubator-mxnet/pull/11684#issuecomment-404730258
 
 
   Hi,
   My environment:
   OS:  CentOS Linux release 7.3.1611 (Core)
   GCC: gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-28)
   CPU: Intel(R) Xeon(R) CPU E5-2620 v4 @ 2.10GHz
   DRAM:  64GB 2133MHZ
   G++: g++ (GCC) 4.8.5 20150623 (Red Hat 4.8.5-28)
   NVCC: Built on Fri_Sep__1_21:08:03_CDT_2017 + Cuda compilation tools, 
release 9.0, V9.0.176
   
   command for compiling:
   make -j $(nproc) USE_OPENCV=1 USE_BLAS=openblas USE_CUDA=1 
USE_CUDA_PATH=/usr/local/cuda USE_CUDNN=1 USE_DIST_KVSTORE=1 USE_NCCL=1 
USE_CPP_PACKAGE=1 DEBUG=1
   
   there are two errors:
   1: src/kvstore/./kvstore_nccl.h:345:28: error: ‘struct 
mxnet::kvstore::KVStoreNCCL::BufferEntry’ has no member named ‘ctx’
assert(root == buf.ctx().dev_id);
   When i fixed this error as uncommend and complie again, got the second error:
   
   a - build/src/kvstore/kvstore_utils_gpu.o
   /usr/lib/gcc/x86_64-redhat-linux/4.8.5/../../../../lib64/crti.o: In function 
`_init':
   (.init+0x7): relocation truncated to fit: R_X86_64_GOTPCREL against 
undefined symbol `__gmon_start__'
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::NumResponse(int)':
   
/usr/include/c++/4.8.2/x86_64-redhat-linux/bits/gthr-default.h:747:(.text+0x5): 
relocation truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::AddResponse(int, int)':
   
/usr/include/c++/4.8.2/x86_64-redhat-linux/bits/gthr-default.h:747:(.text+0x7d):
 relocation truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::WaitRequest(int)':
   
/usr/include/c++/4.8.2/x86_64-redhat-linux/bits/gthr-default.h:747:(.text+0xfe):
 relocation truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::Receiving()':
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/src/customer.cc:49:(.text+0x1d2):
 relocation truncated to fit: R_X86_64_GOTPCREL against symbol 
`std::string::_Rep::_S_empty_rep_storage@@GLIBCXX_3.4' defined in .bss section 
in /usr/lib/gcc/x86_64-redhat-linux/4.8.5/libstdc++.so
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::Receiving()':
   
/usr/include/c++/4.8.2/x86_64-redhat-linux/bits/gthr-default.h:747:(.text+0x2dd):
 relocation truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::Receiving()':
   /usr/include/c++/4.8.2/bits/stl_construct.h:102:(.text+0x46f): relocation 
truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   /usr/include/c++/4.8.2/bits/stl_construct.h:102:(.text+0x555): relocation 
truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   /usr/include/c++/4.8.2/bits/stl_construct.h:102:(.text+0x61a): relocation 
truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   /usr/include/c++/4.8.2/bits/stl_construct.h:102:(.text+0x6ce): relocation 
truncated to fit: R_X86_64_GOTPCREL against symbol 
`__pthread_key_create@@GLIBC_2.2.5' defined in .text section in 
/lib64/libpthread.so.0
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::Receiving()':
   
/usr/include/c++/4.8.2/x86_64-redhat-linux/bits/gthr-default.h:747:(.text+0x773):
 additional relocation overflows omitted from the output
   collect2: error: ld returned 1 exit status
   make: *** [lib/libmxnet.so] Error 1
   make: *** Waiting for unfinished jobs
   
/usr/lib/gcc/x86_64-redhat-linux/4.8.5/../../../../lib64/crt1.o:(.eh_frame+0x20):
 relocation truncated to fit: R_X86_64_PC32 against `.text'
   /usr/lib/gcc/x86_64-redhat-linux/4.8.5/../../../../lib64/crti.o: In function 
`_init':
   (.init+0x7): relocation truncated to fit: R_X86_64_GOTPCREL against 
undefined symbol `__gmon_start__'
   
/home/yang/software/incubator-mxnet/3rdparty/ps-lite/build/libps.a(customer.o): 
In function `ps::Customer::NumResponse(int)':
   

[GitHub] szha opened a new issue #11726: test_operator_gpu.test_bilinear_sampler_with_type has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11726: 
test_operator_gpu.test_bilinear_sampler_with_type has fixed seed that can mask 
flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11726
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11724: test_operator.test_sequence_last has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11724: test_operator.test_sequence_last has fixed seed 
that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11724
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11725: test_operator.test_unary_math_operators has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11725: test_operator.test_unary_math_operators has 
fixed seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11725
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11723: test_operator.test_roipooling has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11723: test_operator.test_roipooling has fixed seed 
that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11723
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11721: test_operator.test_laop_4 has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11721: test_operator.test_laop_4 has fixed seed that 
can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11721
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11722: test_operator.test_rnnrelu_bidirectional has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11722: test_operator.test_rnnrelu_bidirectional has 
fixed seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11722
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11720: test_operator.test_laop_3 has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11720: test_operator.test_laop_3 has fixed seed that 
can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11720
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11718: test_operator.test_laop has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11718: test_operator.test_laop has fixed seed that can 
mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11718
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11719: test_operator.test_laop_2 has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11719: test_operator.test_laop_2 has fixed seed that 
can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11719
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11716: test_operator.test_elementwise_sum has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11716: test_operator.test_elementwise_sum has fixed 
seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11716
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11717: test_operator.test_l2_normalization has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11717: test_operator.test_l2_normalization has fixed 
seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11717
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11715: test_operator.test_dropout has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11715: test_operator.test_dropout has fixed seed that 
can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11715
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] haojin2 commented on issue #10387: Flaky test(scala): test_arange

2018-07-12 Thread GitBox
haojin2 commented on issue #10387: Flaky test(scala): test_arange
URL: 
https://github.com/apache/incubator-mxnet/issues/10387#issuecomment-404729617
 
 
   Plus, seems like a duplicate issue for #8834


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] szha opened a new issue #11714: test_operator.test_dot has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11714: test_operator.test_dot has fixed seed that can 
mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11714
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11713: test_operator.test_deformable_psroipooling has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11713: test_operator.test_deformable_psroipooling has 
fixed seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11713
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11712: test_operator.test_binary_math_operators has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11712: test_operator.test_binary_math_operators has 
fixed seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11712
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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


[incubator-mxnet] branch master updated: [MXNET-323] Improve performance of broadcast ops backward pass (#11252)

2018-07-12 Thread jxie
This is an automated email from the ASF dual-hosted git repository.

jxie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git


The following commit(s) were added to refs/heads/master by this push:
 new 32d298b  [MXNET-323] Improve performance of broadcast ops backward 
pass (#11252)
32d298b is described below

commit 32d298bd35a1b8513c51fbe505c2ee878480cb0f
Author: Anirudh Subramanian 
AuthorDate: Thu Jul 12 22:27:30 2018 -0700

[MXNET-323] Improve performance of broadcast ops backward pass (#11252)

* Fix cached broadcast

* Fix

* Use seq_reduce_compute logic for stable sum

* Fix lint

* Add declarations

* Add elemwise binary broadcast op cuh file

* Add license for elemwise_binary_broadcast_op-inl.cuh

* Fix broadcast

* Fix indentation

* Use cpu and gpu instead of xpu
---
 src/operator/tensor/broadcast_reduce-inl.cuh   |  4 ++
 src/operator/tensor/broadcast_reduce-inl.h | 48 +++-
 .../tensor/elemwise_binary_broadcast_op-inl.cuh| 64 ++
 src/operator/tensor/elemwise_binary_broadcast_op.h | 35 +++-
 4 files changed, 135 insertions(+), 16 deletions(-)

diff --git a/src/operator/tensor/broadcast_reduce-inl.cuh 
b/src/operator/tensor/broadcast_reduce-inl.cuh
index 5c9b45f..be3d1f9 100644
--- a/src/operator/tensor/broadcast_reduce-inl.cuh
+++ b/src/operator/tensor/broadcast_reduce-inl.cuh
@@ -613,6 +613,10 @@ void Reduce(Stream *s, const TBlob& small, const 
OpReqType req,
   ReduceImpl(stream, small, req, big, workspace, 
config);
 }
 
+template 
+void ReduceWithExtraMem(Stream* s, const TBlob& small, const OpReqType 
req,
+const Tensor& workspace, const TBlob& 
big) {};
+
 template
 void Reduce(Stream *s, const TBlob& small, const OpReqType req,
 const Tensor& workspace, const TBlob& big,
diff --git a/src/operator/tensor/broadcast_reduce-inl.h 
b/src/operator/tensor/broadcast_reduce-inl.h
index 713e3f1..39c68bd 100644
--- a/src/operator/tensor/broadcast_reduce-inl.h
+++ b/src/operator/tensor/broadcast_reduce-inl.h
@@ -31,6 +31,7 @@
 #include 
 #include 
 #include "../mshadow_op.h"
+#include "../operator_common.h"
 
 namespace mxnet {
 namespace op {
@@ -205,16 +206,57 @@ void seq_reduce_compute(const int N, const int M, const 
bool addto,
   }
 }
 
-template
-void Reduce(Stream *s, const TBlob& small, const OpReqType req,
+template 
+void seq_reduce_compute_extra_mem(const int N, const int M, const bool addto,
+  const DType* big, DType* small,
+  const Shape bshape,
+  const Shape sshape,
+  const Shape rshape,
+  const Shape rstride,
+  const index_t* ws_dptr) {
+  #pragma omp parallel for 
num_threads(engine::OpenMP::Get()->GetRecommendedOMPThreadCount())
+  for (int idx = 0; idx < N; ++idx) {
+Shape coord = unravel(idx, sshape);
+int j = ravel(coord, bshape);
+DType val, residual;
+Reducer::SetInitValue(val, residual);
+for (int k = 0; k < M; ++k) {
+  Reducer::Reduce(val, OP::Map(big[j + ws_dptr[k]]), residual);
+}
+assign([idx], addto, val);
+  }
+}
+
+template 
+void Reduce(Stream* s, const TBlob& small, const OpReqType req,
 const Tensor& workspace, const TBlob& big) {
   if (req == kNullOp) return;
   Shape rshape, rstride;
   diff(small.shape_.get(), big.shape_.get(), , );
   int N = small.shape_.Size(), M = rshape.Size();
   seq_reduce_compute(
+N, M, req == kAddTo, big.dptr(), small.dptr(),
+big.shape_.get(), small.shape_.get(), rshape, rstride);
+}
+
+template 
+void ReduceWithExtraMem(Stream* s, const TBlob& small, const OpReqType 
req,
+const Tensor& workspace, const TBlob& 
big) {
+  using namespace mxnet_op;
+  if (req == kNullOp) return;
+  Shape rshape, rstride;
+  diff(small.shape_.get(), big.shape_.get(), , );
+  index_t* ws_dptr = reinterpret_cast(workspace.dptr_);
+  int N = small.shape_.Size(), M = rshape.Size();
+  #pragma omp parallel for 
num_threads(engine::OpenMP::Get()->GetRecommendedOMPThreadCount())
+  for (int k = 0; k < M; k++) {
+Shape coord = unravel(k, rshape);
+ws_dptr[k] = dot(coord, rstride);
+  }
+
+  seq_reduce_compute_extra_mem(
 N, M, req == kAddTo, big.dptr(), small.dptr(), 
big.shape_.get(),
-small.shape_.get(), rshape, rstride);
+small.shape_.get(), rshape, rstride, ws_dptr);
 }
 
 template
diff --git a/src/operator/tensor/elemwise_binary_broadcast_op-inl.cuh 
b/src/operator/tensor/elemwise_binary_broadcast_op-inl.cuh
new file mode 100644
index 000..1998576
--- /dev/null
+++ b/src/operator/tensor/elemwise_binary_broadcast_op-inl.cuh
@@ -0,0 +1,64 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor 

[GitHub] szha opened a new issue #11711: test_operator.test_bilinear_sampler has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11711: test_operator.test_bilinear_sampler has fixed 
seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11711
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] piiswrong closed pull request #11252: [MXNET-323] Improve performance of broadcast ops backward pass

2018-07-12 Thread GitBox
piiswrong closed pull request #11252: [MXNET-323] Improve performance of 
broadcast ops backward pass
URL: https://github.com/apache/incubator-mxnet/pull/11252
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/src/operator/tensor/broadcast_reduce-inl.cuh 
b/src/operator/tensor/broadcast_reduce-inl.cuh
index b6bb39a1984..f1a55315da7 100644
--- a/src/operator/tensor/broadcast_reduce-inl.cuh
+++ b/src/operator/tensor/broadcast_reduce-inl.cuh
@@ -602,6 +602,10 @@ void Reduce(Stream *s, const TBlob& small, const 
OpReqType req,
   ReduceImpl(stream, small, req, big, workspace, 
config);
 }
 
+template 
+void ReduceWithExtraMem(Stream* s, const TBlob& small, const OpReqType 
req,
+const Tensor& workspace, const TBlob& 
big) {};
+
 template
 void Reduce(Stream *s, const TBlob& small, const OpReqType req,
 const Tensor& workspace, const TBlob& big,
diff --git a/src/operator/tensor/broadcast_reduce-inl.h 
b/src/operator/tensor/broadcast_reduce-inl.h
index 76ec92a9e72..c969a219e64 100644
--- a/src/operator/tensor/broadcast_reduce-inl.h
+++ b/src/operator/tensor/broadcast_reduce-inl.h
@@ -31,6 +31,7 @@
 #include 
 #include 
 #include "../mshadow_op.h"
+#include "../operator_common.h"
 
 namespace mxnet {
 namespace op {
@@ -204,16 +205,57 @@ void seq_reduce_compute(const int N, const int M, const 
bool addto,
   }
 }
 
-template
-void Reduce(Stream *s, const TBlob& small, const OpReqType req,
+template 
+void seq_reduce_compute_extra_mem(const int N, const int M, const bool addto,
+  const DType* big, DType* small,
+  const Shape bshape,
+  const Shape sshape,
+  const Shape rshape,
+  const Shape rstride,
+  const index_t* ws_dptr) {
+  #pragma omp parallel for 
num_threads(engine::OpenMP::Get()->GetRecommendedOMPThreadCount())
+  for (int idx = 0; idx < N; ++idx) {
+Shape coord = unravel(idx, sshape);
+int j = ravel(coord, bshape);
+DType val, residual;
+Reducer::SetInitValue(val, residual);
+for (int k = 0; k < M; ++k) {
+  Reducer::Reduce(val, OP::Map(big[j + ws_dptr[k]]), residual);
+}
+assign([idx], addto, val);
+  }
+}
+
+template 
+void Reduce(Stream* s, const TBlob& small, const OpReqType req,
 const Tensor& workspace, const TBlob& big) {
   if (req == kNullOp) return;
   Shape rshape, rstride;
   diff(small.shape_.get(), big.shape_.get(), , );
   int N = small.shape_.Size(), M = rshape.Size();
   seq_reduce_compute(
+N, M, req == kAddTo, big.dptr(), small.dptr(),
+big.shape_.get(), small.shape_.get(), rshape, rstride);
+}
+
+template 
+void ReduceWithExtraMem(Stream* s, const TBlob& small, const OpReqType 
req,
+const Tensor& workspace, const TBlob& 
big) {
+  using namespace mxnet_op;
+  if (req == kNullOp) return;
+  Shape rshape, rstride;
+  diff(small.shape_.get(), big.shape_.get(), , );
+  index_t* ws_dptr = reinterpret_cast(workspace.dptr_);
+  int N = small.shape_.Size(), M = rshape.Size();
+  #pragma omp parallel for 
num_threads(engine::OpenMP::Get()->GetRecommendedOMPThreadCount())
+  for (int k = 0; k < M; k++) {
+Shape coord = unravel(k, rshape);
+ws_dptr[k] = dot(coord, rstride);
+  }
+
+  seq_reduce_compute_extra_mem(
 N, M, req == kAddTo, big.dptr(), small.dptr(), 
big.shape_.get(),
-small.shape_.get(), rshape, rstride);
+small.shape_.get(), rshape, rstride, ws_dptr);
 }
 
 template
diff --git a/src/operator/tensor/elemwise_binary_broadcast_op-inl.cuh 
b/src/operator/tensor/elemwise_binary_broadcast_op-inl.cuh
new file mode 100644
index 000..1998576172b
--- /dev/null
+++ b/src/operator/tensor/elemwise_binary_broadcast_op-inl.cuh
@@ -0,0 +1,64 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+#ifndef MXNET_OPERATOR_TENSOR_ELEMWISE_BINARY_BROADCAST_OP_CUH_
+#define 

[GitHub] haojin2 commented on issue #10274: test_ndarray.test_reduce fails in v1.0.0

2018-07-12 Thread GitBox
haojin2 commented on issue #10274: test_ndarray.test_reduce fails in v1.0.0
URL: 
https://github.com/apache/incubator-mxnet/issues/10274#issuecomment-404729481
 
 
   Is this a duplicate issue of #8780?


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] szha opened a new issue #11710: test_ndarray.test_ndarray_pickle has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11710: test_ndarray.test_ndarray_pickle has fixed seed 
that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11710
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11709: test_ndarray.test_ndarray_elementwise has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11709: test_ndarray.test_ndarray_elementwise has fixed 
seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11709
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11708: test_ndarray.test_cached has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11708: test_ndarray.test_cached has fixed seed that 
can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11708
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11707: test_nccl.test_nccl_pushpull has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11707: test_nccl.test_nccl_pushpull has fixed seed 
that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11707
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] haojin2 commented on issue #11501: Tracking issue for tests with fixed random seed

2018-07-12 Thread GitBox
haojin2 commented on issue #11501: Tracking issue for tests with fixed random 
seed
URL: 
https://github.com/apache/incubator-mxnet/issues/11501#issuecomment-404729204
 
 
   @szha So shall I close this issue for now?


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] szha opened a new issue #11706: test_module.test_monitor has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11706: test_module.test_monitor has fixed seed that 
can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11706
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11704: test_module.test_factorization_machine_module has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11704: test_module.test_factorization_machine_module 
has fixed seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11704
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11705: test_module.test_module_set_params has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11705: test_module.test_module_set_params has fixed 
seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11705
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11703: test_loss.test_triplet_loss has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11703: test_loss.test_triplet_loss has fixed seed that 
can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11703
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11701: test_loss.test_saveload has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11701: test_loss.test_saveload has fixed seed that can 
mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11701
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11702: test_loss.test_squared_hinge_loss has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11702: test_loss.test_squared_hinge_loss has fixed 
seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11702
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11699: test_loss.test_l2_loss has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11699: test_loss.test_l2_loss has fixed seed that can 
mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11699
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11698: test_loss.test_l1_loss has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11698: test_loss.test_l1_loss has fixed seed that can 
mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11698
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11700: test_loss.test_sample_weight_loss has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11700: test_loss.test_sample_weight_loss has fixed 
seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11700
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11697: test_loss.test_kl_loss has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11697: test_loss.test_kl_loss has fixed seed that can 
mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11697
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11695: test_loss.test_hinge_loss has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11695: test_loss.test_hinge_loss has fixed seed that 
can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11695
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11696: test_loss.test_huber_loss has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11696: test_loss.test_huber_loss has fixed seed that 
can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11696
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11694: test_loss.test_ctc_loss_train has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11694: test_loss.test_ctc_loss_train has fixed seed 
that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11694
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11693: test_loss.test_ctc_loss has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11693: test_loss.test_ctc_loss has fixed seed that can 
mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11693
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11692: test_loss.test_ce_loss has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11692: test_loss.test_ce_loss has fixed seed that can 
mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11692
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] haojin2 commented on issue #8834: Flaky test: Scala:OperatorSuite.arange

2018-07-12 Thread GitBox
haojin2 commented on issue #8834: Flaky test: Scala:OperatorSuite.arange
URL: 
https://github.com/apache/incubator-mxnet/issues/8834#issuecomment-404728554
 
 
   @marcoabreu If this is no longer happening again would you please close the 
issue? Thanks!


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] szha opened a new issue #11691: test_loss.test_bce_loss has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11691: test_loss.test_bce_loss has fixed seed that can 
mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11691
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11689: test_gluon_data.test_multi_worker has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11689: test_gluon_data.test_multi_worker has fixed 
seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11689
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] lanking520 commented on issue #8834: Flaky test: Scala:OperatorSuite.arange

2018-07-12 Thread GitBox
lanking520 commented on issue #8834: Flaky test: Scala:OperatorSuite.arange
URL: 
https://github.com/apache/incubator-mxnet/issues/8834#issuecomment-404728275
 
 
   Yep, @haojin2 kill this one


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] szha opened a new issue #11690: test_io.test_ImageRecordIter_seed_augmentation has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11690: test_io.test_ImageRecordIter_seed_augmentation 
has fixed seed that can mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11690
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11688: test_gluon.test_export has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11688: test_gluon.test_export has fixed seed that can 
mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11688
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.


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] szha opened a new issue #11687: test_gluon.test_conv has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11687: test_gluon.test_conv has fixed seed that can 
mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11687
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.


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] szha opened a new issue #11686: test_executor.test_dot has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11686: test_executor.test_dot has fixed seed that can 
mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11686
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.
   


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] szha opened a new issue #11685: test_executor.test_bind has fixed seed that can mask flakiness

2018-07-12 Thread GitBox
szha opened a new issue #11685: test_executor.test_bind has fixed seed that can 
mask flakiness
URL: https://github.com/apache/incubator-mxnet/issues/11685
 
 
   The unit test in title have been using fixed seed to mask flakiness. 
Suggested action:
   1. Evaluate whether the test is flaky without fixed seed. If not, remove 
seed. Else move to 2
   2. If test is flaky, determine whether it's an actual uncaught edge case. If 
so, fix the operator. Else move to 3
   3. If numerical instability is inevitable, adjust tolerance level 
appropriately.


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] cclauss edited a comment on issue #11669: Fix TabError for Python 3

2018-07-12 Thread GitBox
cclauss edited a comment on issue #11669: Fix TabError for Python 3
URL: https://github.com/apache/incubator-mxnet/pull/11669#issuecomment-404726234
 
 
   @marcoabreu @ThomasDelteil Can you please review ("Files changed" --> 
"Review changes") this change because the goal of this PR is to make a **simple 
whitespace-only change** that is easy to review.  However, much text has been 
written above that will cause reviewers to pause.  


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] szha commented on issue #11501: Tracking issue for tests with fixed random seed

2018-07-12 Thread GitBox
szha commented on issue #11501: Tracking issue for tests with fixed random seed
URL: 
https://github.com/apache/incubator-mxnet/issues/11501#issuecomment-404726406
 
 
   Thanks for summarizing the issues @haojin2. Fixed seed indeed has the effect 
of hiding flaky tests so we should treat these as flaky tests until fixed. On 
the other hand, I think it's probably better to track them as individual items 
so that people can contribute to the fix one at a time. I will go ahead and 
split these into individual issues.


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] cclauss commented on issue #11669: Fix TabError for Python 3

2018-07-12 Thread GitBox
cclauss commented on issue #11669: Fix TabError for Python 3
URL: https://github.com/apache/incubator-mxnet/pull/11669#issuecomment-404726234
 
 
   @marcoabreu @ThomasDelteil Can you please review ("Files changed" --> 
"Review changes") this change because the goal of this PR was to make a 
**simple whitespace-only change** that was easy to review.  However, much text 
has been written above that will cause reviewers to pause.  


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] haojin2 commented on issue #10387: Flaky test(scala): test_arange

2018-07-12 Thread GitBox
haojin2 commented on issue #10387: Flaky test(scala): test_arange
URL: 
https://github.com/apache/incubator-mxnet/issues/10387#issuecomment-404723360
 
 
   @marcoabreu Have we observed more occurrence of this?


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] haojin2 commented on issue #8834: Flaky test: Scala:OperatorSuite.arange

2018-07-12 Thread GitBox
haojin2 commented on issue #8834: Flaky test: Scala:OperatorSuite.arange
URL: 
https://github.com/apache/incubator-mxnet/issues/8834#issuecomment-404723208
 
 
   @lanking520 Was this fixed earlier?


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] haojin2 commented on issue #11680: Flaky Perl test: test_loss.test_dynamic

2018-07-12 Thread GitBox
haojin2 commented on issue #11680: Flaky Perl test: test_loss.test_dynamic
URL: 
https://github.com/apache/incubator-mxnet/issues/11680#issuecomment-404722419
 
 
   Thanks a lot for your quick response!


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] szha commented on issue #11684: fix compile error in debug model

2018-07-12 Thread GitBox
szha commented on issue #11684: fix compile error in debug model
URL: https://github.com/apache/incubator-mxnet/pull/11684#issuecomment-404719232
 
 
   @NEWPLAN thanks. What error do you see when compiling in debug mode? What 
operating system and compiler are you using?


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] rahul003 commented on a change in pull request #11633: Fix dist kvstore for trainer and flaky dist kvstore test

2018-07-12 Thread GitBox
rahul003 commented on a change in pull request #11633: Fix dist kvstore for 
trainer and flaky dist kvstore test
URL: https://github.com/apache/incubator-mxnet/pull/11633#discussion_r202236977
 
 

 ##
 File path: tests/nightly/dist_sync_kvstore.py
 ##
 @@ -398,18 +432,25 @@ def check_trainer_kv_type(stype, grad_stype, 
update_on_kv):
 parser.add_argument('--no-gpu', dest='gpu', action='store_false')
 parser.add_argument('--no-multiprecision', dest='multiprecision', 
action='store_false')
 opt = parser.parse_args()
-if opt.type == 'gluon':
-test_gluon_trainer()
-if opt.type == 'invalid':
+if opt.type == 'gluon_type_cpu':
+test_gluon_trainer_type()
+exit()
+if opt.type == 'gluon_step_cpu':
+test_gluon_trainer_step()
+exit()
+if opt.type == 'gluon_sparse_step_cpu':
+test_gluon_trainer_sparse_step()
+exit()
+if opt.type == 'invalid_cpu':
 test_invalid_operations()
+exit()
 if opt.type == 'all' or opt.type == 'init':
 test_sync_init(opt.gpu)
+kv = init_kv()
 if opt.type == 'all' or opt.type == 'default':
-kv = init_kv()
 kv = set_optimizer(use_multiprecision=opt.multiprecision)
 test_sync_push_pull(opt.nrepeat)
-# dont run non compressed tests after this as kvstore compression will be 
set here
+# don't run non compressed tests after this as kvstore compression will be 
set here
 if opt.type == 'all' or opt.type == 'compressed':
 
 Review comment:
   Because in the current script, it no longer represents all tests.


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] eric-haibin-lin commented on a change in pull request #11633: Fix dist kvstore for trainer and flaky dist kvstore test

2018-07-12 Thread GitBox
eric-haibin-lin commented on a change in pull request #11633: Fix dist kvstore 
for trainer and flaky dist kvstore test
URL: https://github.com/apache/incubator-mxnet/pull/11633#discussion_r202236693
 
 

 ##
 File path: tests/nightly/dist_sync_kvstore.py
 ##
 @@ -398,18 +432,25 @@ def check_trainer_kv_type(stype, grad_stype, 
update_on_kv):
 parser.add_argument('--no-gpu', dest='gpu', action='store_false')
 parser.add_argument('--no-multiprecision', dest='multiprecision', 
action='store_false')
 opt = parser.parse_args()
-if opt.type == 'gluon':
-test_gluon_trainer()
-if opt.type == 'invalid':
+if opt.type == 'gluon_type_cpu':
+test_gluon_trainer_type()
+exit()
+if opt.type == 'gluon_step_cpu':
+test_gluon_trainer_step()
+exit()
+if opt.type == 'gluon_sparse_step_cpu':
+test_gluon_trainer_sparse_step()
+exit()
+if opt.type == 'invalid_cpu':
 test_invalid_operations()
+exit()
 if opt.type == 'all' or opt.type == 'init':
 test_sync_init(opt.gpu)
+kv = init_kv()
 if opt.type == 'all' or opt.type == 'default':
-kv = init_kv()
 kv = set_optimizer(use_multiprecision=opt.multiprecision)
 test_sync_push_pull(opt.nrepeat)
-# dont run non compressed tests after this as kvstore compression will be 
set here
+# don't run non compressed tests after this as kvstore compression will be 
set here
 if opt.type == 'all' or opt.type == 'compressed':
 
 Review comment:
   why remove it?
   


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] anirudhacharya commented on issue #10686: Building from source fails

2018-07-12 Thread GitBox
anirudhacharya commented on issue #10686: Building from source fails 
URL: 
https://github.com/apache/incubator-mxnet/issues/10686#issuecomment-404716668
 
 
   @crawforc3 please try these updated instructions and reopen if the issue 
persists.
   
   @nswamy please close this issue.


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] NEWPLAN opened a new pull request #11684: fix compile error in debug model

2018-07-12 Thread GitBox
NEWPLAN opened a new pull request #11684: fix compile error in debug model
URL: https://github.com/apache/incubator-mxnet/pull/11684
 
 
   if in debug model, the generate file is large than default size, increase 
the code size limitation
   
   ## Description ##
   This PR is for fixing compiler error in debug model
   
   ## Checklist ##
   ### Essentials ###
   Please feel free to remove inapplicable items for your PR.
   - [ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to 
the relevant [JIRA issue](https://issues.apache.org/jira/projects/MXNET/issues) 
created (except PRs with tiny changes)
   - [ ] Changes are complete (i.e. I finished coding on this PR)
   - [ ] All changes have test coverage:
   - Unit tests are added for small changes to verify correctness (e.g. adding 
a new operator)
   - Nightly tests are added for complicated/long-running ones (e.g. changing 
distributed kvstore)
   - Build tests will be added for build configuration changes (e.g. adding a 
new build option with NCCL)
   - [ ] Code is well-documented: 
   - For user-facing API changes, API doc string has been updated. 
   - For new C++ functions in header files, their functionalities and arguments 
are documented. 
   - For new examples, README.md is added to explain the what the example does, 
the source of the dataset, expected performance on test set and reference to 
the original paper if applicable
   - Check the API doc at 
http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
   - [ ] To the my best knowledge, examples are either not affected by this 
change, or have been fixed to be compatible with this change
   
   ### Changes ###
   - [ ] Feature1, tests, (and when applicable, API doc)
   - [ ] Feature2, tests, (and when applicable, API doc)
   
   ## Comments ##
   - If this change is a backward incompatible change, why must this change be 
made.
   - Interesting edge cases to note here
   


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] sergeykolychev commented on issue #11680: Flaky Perl test: test_loss.test_dynamic

2018-07-12 Thread GitBox
sergeykolychev commented on issue #11680: Flaky Perl test: 
test_loss.test_dynamic
URL: 
https://github.com/apache/incubator-mxnet/issues/11680#issuecomment-404706340
 
 
   @haojin2 Should be fixed by 
https://github.com/apache/incubator-mxnet/pull/11642/commits/ef96ad3c11abed719951ba70833cb273c47104fa
   I plan to merge in parent PR tomorrow.
   Thanks.


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


[incubator-mxnet] branch master updated: Update README.md (#11677)

2018-07-12 Thread anirudh2290
This is an automated email from the ASF dual-hosted git repository.

anirudh2290 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git


The following commit(s) were added to refs/heads/master by this push:
 new 92366a9  Update README.md (#11677)
92366a9 is described below

commit 92366a99b15395583ec2991e722c679da1e281c5
Author: Kellen Sunderland 
AuthorDate: Fri Jul 13 04:25:18 2018 +0200

Update README.md (#11677)
---
 contrib/clojure-package/README.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/contrib/clojure-package/README.md 
b/contrib/clojure-package/README.md
index 430be4d..5e7356c 100644
--- a/contrib/clojure-package/README.md
+++ b/contrib/clojure-package/README.md
@@ -193,7 +193,7 @@ This command can be very handy too
 timestamp, name, utilization.gpu [%], utilization.memory [%], memory.total 
[MiB], memory.free [MiB], memory.used [MiB]`
 
 **Supported APIs**
-There are 3 high level apis supported in MxNet: (Model/FeedForward), Module, 
and Gluon. The Module api is supported in the Clojure package because of the 
existing support for it in the Scala package. The Module api is very similar to 
the Gluon api and examples of the usage can be found in the examples directory. 
The Model/FeedForward Api is deprected.
+There are 3 high level apis supported in MXNet: (Model/FeedForward), Module, 
and Gluon. The Module api is supported in the Clojure package because of the 
existing support for it in the Scala package. The Module api is very similar to 
the Gluon api and examples of the usage can be found in the examples directory. 
The Model/FeedForward Api is deprected.
 
 Gluon support will come later and may or may not be built on the Scala gluon 
api (when it lands there)
 



[GitHub] anirudh2290 closed pull request #11677: Update README.md to fix MXNet capitalization

2018-07-12 Thread GitBox
anirudh2290 closed pull request #11677: Update README.md to fix MXNet 
capitalization
URL: https://github.com/apache/incubator-mxnet/pull/11677
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/contrib/clojure-package/README.md 
b/contrib/clojure-package/README.md
index 430be4d6a84..5e7356caf64 100644
--- a/contrib/clojure-package/README.md
+++ b/contrib/clojure-package/README.md
@@ -193,7 +193,7 @@ This command can be very handy too
 timestamp, name, utilization.gpu [%], utilization.memory [%], memory.total 
[MiB], memory.free [MiB], memory.used [MiB]`
 
 **Supported APIs**
-There are 3 high level apis supported in MxNet: (Model/FeedForward), Module, 
and Gluon. The Module api is supported in the Clojure package because of the 
existing support for it in the Scala package. The Module api is very similar to 
the Gluon api and examples of the usage can be found in the examples directory. 
The Model/FeedForward Api is deprected.
+There are 3 high level apis supported in MXNet: (Model/FeedForward), Module, 
and Gluon. The Module api is supported in the Clojure package because of the 
existing support for it in the Scala package. The Module api is very similar to 
the Gluon api and examples of the usage can be found in the examples directory. 
The Model/FeedForward Api is deprected.
 
 Gluon support will come later and may or may not be built on the Scala gluon 
api (when it lands there)
 


 


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] sergeykolychev commented on issue #11680: Flaky Perl test: test_loss.test_dynamic

2018-07-12 Thread GitBox
sergeykolychev commented on issue #11680: Flaky Perl test: 
test_loss.test_dynamic
URL: 
https://github.com/apache/incubator-mxnet/issues/11680#issuecomment-404704376
 
 
   @haojin2 I'll take a look ad will fix.


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] yukang2017 opened a new issue #11683: time cost in grad clipping

2018-07-12 Thread GitBox
yukang2017 opened a new issue #11683: time cost in grad clipping
URL: https://github.com/apache/incubator-mxnet/issues/11683
 
 
   I compare the speed of grad clipping between pytorch(0.3.1) and 
mxnet-gluon(1.2.0) on a Titan X. Pytorch gets nearly 10 times faster than 
mxnet-gluon.
   (The network I use is [nasnet](https://arxiv.org/abs/1707.07012).)
   mxnet:
   ```
   grads = [i.grad(ctx) for i in model.collect_params().values() if i._grad is 
not None]
   gluon.utils.clip_global_norm(grads, args.grad_clip)
   ```
   pytorch:
   `nn.utils.clip_grad_norm(model.parameters(), args.grad_clip)
   `
   
   However, when I test 'clip_gradient' in mxnet.gluon.Trainer and torch.optim, 
following thomelane's reply in the 
[issue](https://github.com/apache/incubator-mxnet/issues/11508#issuecomment-404675656).
   
   The time cost is similar.
   ```
   mxnet.gluon.Trainer(net.collect_params(), optimizer='sgd',
   optimizer_params={'learning_rate': 0.1, 
'clip_gradient':5},
   kvstore='device') 
   ```
   


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] yukang2017 commented on issue #11508: 【Question】Are there any examples for gradients clipping in gluon?

2018-07-12 Thread GitBox
yukang2017 commented on issue #11508: 【Question】Are there any examples for 
gradients clipping in gluon?
URL: 
https://github.com/apache/incubator-mxnet/issues/11508#issuecomment-404702227
 
 
   @thomelane I will open a new issue. Thank you!
   I tested 'clip_gradient' in _mxnet.gluon.Trainer_ and torch.optim. The time 
cost is similar.


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] lynneChan commented on issue #11137: memory cost problem for different label width when using ssd example

2018-07-12 Thread GitBox
lynneChan commented on issue #11137: memory cost problem for different label 
width when using ssd example
URL: 
https://github.com/apache/incubator-mxnet/issues/11137#issuecomment-404701765
 
 
   Hi @thomelane, thanks for your replay.
   
I was using the SSD example code to training a face detector, the WIDER 
dataset have about 12880 images, so the `batch_size` is fine. And the number of 
 classes is 2(face or non-face). The code used `DetRecordIter` in 
`dataset/iterator.py` to get image batches which called another API 
`mx.io.ImageDetRecordIter`. This API need a `label_width` parameter. 
   
   I keeped all the settings, and just change the `label_width` from 602 to 
4784, and the GPU memory cost changes from  6991MB to 10367MB. That's pretty 
strange, it should not increase so much.


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] Feywell commented on issue #9186: How to train model with multi machines

2018-07-12 Thread GitBox
Feywell commented on issue #9186: How to train model with multi machines
URL: 
https://github.com/apache/incubator-mxnet/issues/9186#issuecomment-404700463
 
 
   @thomelane Thank you! I will read it carefully.


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] ijkguo commented on issue #8474: How to train variable input shape CNN?

2018-07-12 Thread GitBox
ijkguo commented on issue #8474: How to train variable input shape CNN?
URL: 
https://github.com/apache/incubator-mxnet/issues/8474#issuecomment-404698670
 
 
   Module API had been updated to support variable shape training, c.f. 
https://github.com/apache/incubator-mxnet/pull/6805 before this issue.
   
   MutableModule from rcnn example was introduced before the previous PR to 
avoid breaking module API. It was unfortunate that the example was not updated 
timely since the PR. But if still interested, check out 
https://github.com/apache/incubator-mxnet/blob/master/example/rcnn/train.py#L117.
   
   As for a special tutorial on variable shape training, you don't need one. 
Use `module.fit` or Gluon just as if it was fixed shape will work.


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] ijkguo commented on issue #11240: rcnn example throws CUDNN_STATUS_BAD_PARAM when running under cudnn 6.0

2018-07-12 Thread GitBox
ijkguo commented on issue #11240: rcnn example throws CUDNN_STATUS_BAD_PARAM 
when running under cudnn 6.0
URL: 
https://github.com/apache/incubator-mxnet/issues/11240#issuecomment-404695463
 
 
   It was probably related to old SoftmaxActivation layer. Now changed to 
`mx.sym.softmax` in https://github.com/apache/incubator-mxnet/pull/11373.


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] zhreshold commented on issue #11643: Added the diag() operator

2018-07-12 Thread GitBox
zhreshold commented on issue #11643: Added the diag() operator
URL: https://github.com/apache/incubator-mxnet/pull/11643#issuecomment-404694877
 
 
   @ifeherva Try locally simulate the CI with the docker build script. 
Currently we can get nothing from the log. 


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] solin319 commented on issue #11165: ImageIter much slower than ImageRecordIter

2018-07-12 Thread GitBox
solin319 commented on issue #11165: ImageIter much slower than ImageRecordIter
URL: 
https://github.com/apache/incubator-mxnet/issues/11165#issuecomment-404694859
 
 
   We move some augmenters call in c++, and the speed can achieve 0.21s per 
batch, which is 0.35 smentioned above.
   So we expect community can add an interface can read raw pic with high speed.


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] ifeherva commented on issue #11643: Added the diag() operator

2018-07-12 Thread GitBox
ifeherva commented on issue #11643: Added the diag() operator
URL: https://github.com/apache/incubator-mxnet/pull/11643#issuecomment-404694302
 
 
   Any idea why it does not pass CI?


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


[incubator-mxnet] branch master updated: xrange(10) --> range(10) for Python 3 (#11673)

2018-07-12 Thread haibin
This is an automated email from the ASF dual-hosted git repository.

haibin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git


The following commit(s) were added to refs/heads/master by this push:
 new 6b2cbd4  xrange(10) --> range(10) for Python 3 (#11673)
6b2cbd4 is described below

commit 6b2cbd4ce8fd5088bf4d6c8c6357b3e3305378d6
Author: cclauss 
AuthorDate: Fri Jul 13 02:57:46 2018 +0200

xrange(10) --> range(10) for Python 3 (#11673)

__xrange()__ was removed in Python 3 in favor of __range()__.  For such a 
small number as 10 there is no difference between __xrange()__ and __range()__ 
in Python 2.
---
 benchmark/python/sparse/util.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/benchmark/python/sparse/util.py b/benchmark/python/sparse/util.py
index c20b33a..29000bd 100644
--- a/benchmark/python/sparse/util.py
+++ b/benchmark/python/sparse/util.py
@@ -24,7 +24,7 @@ def estimate_density(DATA_PATH, feature_size):
 raise Exception("Data is not there!")
 density = []
 P = 0.01
-for _ in xrange(10):
+for _ in range(10):
 num_non_zero = 0
 num_sample = 0
 with open(DATA_PATH) as f:



[GitHub] eric-haibin-lin closed pull request #11673: xrange(10) --> range(10) for Python 3

2018-07-12 Thread GitBox
eric-haibin-lin closed pull request #11673: xrange(10) --> range(10) for Python 
3
URL: https://github.com/apache/incubator-mxnet/pull/11673
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/benchmark/python/sparse/util.py b/benchmark/python/sparse/util.py
index c20b33a86d6..29000bdf82e 100644
--- a/benchmark/python/sparse/util.py
+++ b/benchmark/python/sparse/util.py
@@ -24,7 +24,7 @@ def estimate_density(DATA_PATH, feature_size):
 raise Exception("Data is not there!")
 density = []
 P = 0.01
-for _ in xrange(10):
+for _ in range(10):
 num_non_zero = 0
 num_sample = 0
 with open(DATA_PATH) as f:


 


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] zhreshold closed issue #9823: RCNN example fails for using latest mxnet

2018-07-12 Thread GitBox
zhreshold closed issue #9823: RCNN example fails for using latest mxnet
URL: https://github.com/apache/incubator-mxnet/issues/9823
 
 
   


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] zhreshold closed issue #8694: Error when running "/rcnn/script/additional_deps.sh"

2018-07-12 Thread GitBox
zhreshold closed issue #8694: Error when running 
"/rcnn/script/additional_deps.sh"
URL: https://github.com/apache/incubator-mxnet/issues/8694
 
 
   


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] zhreshold closed issue #10019: ModuleNotFoundError

2018-07-12 Thread GitBox
zhreshold closed issue #10019: ModuleNotFoundError
URL: https://github.com/apache/incubator-mxnet/issues/10019
 
 
   


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] zhreshold commented on issue #9823: RCNN example fails for using latest mxnet

2018-07-12 Thread GitBox
zhreshold commented on issue #9823: RCNN example fails for using latest mxnet
URL: 
https://github.com/apache/incubator-mxnet/issues/9823#issuecomment-404693226
 
 
   Closing this after merging #11373, feel free to ping me to reopen it it's 
not fixed.


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] zhreshold commented on issue #8694: Error when running "/rcnn/script/additional_deps.sh"

2018-07-12 Thread GitBox
zhreshold commented on issue #8694: Error when running 
"/rcnn/script/additional_deps.sh"
URL: 
https://github.com/apache/incubator-mxnet/issues/8694#issuecomment-404693156
 
 
   Closing this after merging #11373


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] zhreshold commented on issue #10291: some code in example/rcnn/demo.py is wrong.

2018-07-12 Thread GitBox
zhreshold commented on issue #10291: some code in example/rcnn/demo.py is wrong.
URL: 
https://github.com/apache/incubator-mxnet/issues/10291#issuecomment-404693102
 
 
   Closing this after #11373 is merged


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


  1   2   3   4   >