[GitHub] lupesko commented on issue #9378: Mxnet C API CPU Mode segmentation fault

2018-01-16 Thread GitBox
lupesko commented on issue #9378: Mxnet C API CPU Mode segmentation fault URL: https://github.com/apache/incubator-mxnet/issues/9378#issuecomment-358222749 @pitLog it is reasonable to assume the problem is in your model. Try building w/o MKL and see if the issue goes away.

[GitHub] eric-haibin-lin commented on a change in pull request #9464: refactor logging in infer storage pass

2018-01-16 Thread GitBox
eric-haibin-lin commented on a change in pull request #9464: refactor logging in infer storage pass URL: https://github.com/apache/incubator-mxnet/pull/9464#discussion_r161969301 ## File path: src/executor/infer_graph_attr_pass.cc ## @@ -50,7 +50,15 @@ bool

[GitHub] eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161939535 ## File path: src/operator/tensor/matrix_op.cc ## @@ -180,6 +182,51 @@ If the argument `reverse` is set

[GitHub] reminisce commented on a change in pull request #9464: refactor logging in infer storage pass

2018-01-16 Thread GitBox
reminisce commented on a change in pull request #9464: refactor logging in infer storage pass URL: https://github.com/apache/incubator-mxnet/pull/9464#discussion_r161967144 ## File path: src/executor/infer_graph_attr_pass.cc ## @@ -50,7 +50,15 @@ bool

[GitHub] reminisce commented on a change in pull request #9464: refactor logging in infer storage pass

2018-01-16 Thread GitBox
reminisce commented on a change in pull request #9464: refactor logging in infer storage pass URL: https://github.com/apache/incubator-mxnet/pull/9464#discussion_r161968768 ## File path: src/operator/operator_common.h ## @@ -479,67 +480,11 @@ inline void

[GitHub] eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161939145 ## File path: tests/python/unittest/test_executor.py ## @@ -140,22 +140,20 @@ def test_dot(): def

[GitHub] reminisce commented on a change in pull request #9464: refactor logging in infer storage pass

2018-01-16 Thread GitBox
reminisce commented on a change in pull request #9464: refactor logging in infer storage pass URL: https://github.com/apache/incubator-mxnet/pull/9464#discussion_r161967547 ## File path: src/imperative/imperative_utils.h ## @@ -138,15 +138,23 @@ inline void

[GitHub] eric-haibin-lin commented on a change in pull request #9464: refactor logging in infer storage pass

2018-01-16 Thread GitBox
eric-haibin-lin commented on a change in pull request #9464: refactor logging in infer storage pass URL: https://github.com/apache/incubator-mxnet/pull/9464#discussion_r161964292 ## File path: src/common/utils.h ## @@ -374,6 +416,25 @@ inline void LogOnce(const

[GitHub] cjolivier01 commented on a change in pull request #9464: refactor logging in infer storage pass

2018-01-16 Thread GitBox
cjolivier01 commented on a change in pull request #9464: refactor logging in infer storage pass URL: https://github.com/apache/incubator-mxnet/pull/9464#discussion_r161963306 ## File path: src/common/utils.h ## @@ -364,6 +364,48 @@ inline std::string dev_type_string(const

[GitHub] cjolivier01 commented on a change in pull request #9464: refactor logging in infer storage pass

2018-01-16 Thread GitBox
cjolivier01 commented on a change in pull request #9464: refactor logging in infer storage pass URL: https://github.com/apache/incubator-mxnet/pull/9464#discussion_r161963534 ## File path: src/common/utils.h ## @@ -374,6 +416,25 @@ inline void LogOnce(const std::string&

[GitHub] eric-haibin-lin opened a new pull request #9464: refactor logging in infer storage pass

2018-01-16 Thread GitBox
eric-haibin-lin opened a new pull request #9464: refactor logging in infer storage pass URL: https://github.com/apache/incubator-mxnet/pull/9464 ## Description ## This PR moves the logging of storage fallback message and other error messages from individual operator's FInferStorage to

[GitHub] jeremiedb commented on issue #9399: error: unable to load shared object /mxnet/libs/libmxnet.so

2018-01-16 Thread GitBox
jeremiedb commented on issue #9399: error: unable to load shared object /mxnet/libs/libmxnet.so URL: https://github.com/apache/incubator-mxnet/issues/9399#issuecomment-358201606 Seems like an issue with OpenCV. Have you tried compiled with USE_OPENCV = 0? Did the `libopencv-dev` was

[GitHub] jeremiedb commented on issue #9453: [R] CNN Memory Leak - Needs to somehow call Garbage Collector

2018-01-16 Thread GitBox
jeremiedb commented on issue #9453: [R] CNN Memory Leak - Needs to somehow call Garbage Collector URL: https://github.com/apache/incubator-mxnet/issues/9453#issuecomment-358200492 Far from ideal, but as a dirty patch you can add a gc() call within the train.model function training loop

[GitHub] cjolivier01 commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0

2018-01-16 Thread GitBox
cjolivier01 commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0 URL: https://github.com/apache/incubator-mxnet/issues/9396#issuecomment-358196577 Apparently not -- was checking if something cpu-related was bottleneck but it seems not.

[GitHub] nicklhy commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0

2018-01-16 Thread GitBox
nicklhy commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0 URL: https://github.com/apache/incubator-mxnet/issues/9396#issuecomment-358193712 @cjolivier01 Same result after setting it. The GPU usage is still 93~95% when testing ResNet101 or

[GitHub] cbalioglu opened a new pull request #9463: Rename kvstore/utils.* to kvstore/kvstore_utils.*

2018-01-16 Thread GitBox
cbalioglu opened a new pull request #9463: Rename kvstore/utils.* to kvstore/kvstore_utils.* URL: https://github.com/apache/incubator-mxnet/pull/9463 Using an old version of GCC (e.g. 4.9) as the host compiler for nvcc causes compilation failures in multi-threaded builds when there are

[incubator-mxnet-site] branch asf-site updated: removed torch.html and references. Fixed Nesterov Momentum education link (#46)

2018-01-16 Thread skm
This is an automated email from the ASF dual-hosted git repository. skm pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new 2075c26 removed torch.html and

[GitHub] piiswrong commented on a change in pull request #9460: Data-iterator tutorial made python3 compatible.

2018-01-16 Thread GitBox
piiswrong commented on a change in pull request #9460: Data-iterator tutorial made python3 compatible. URL: https://github.com/apache/incubator-mxnet/pull/9460#discussion_r161949534 ## File path: docs/tutorials/basic/data.md ## @@ -122,17 +126,17 @@ class

[GitHub] piiswrong commented on a change in pull request #9460: Data-iterator tutorial made python3 compatible.

2018-01-16 Thread GitBox
piiswrong commented on a change in pull request #9460: Data-iterator tutorial made python3 compatible. URL: https://github.com/apache/incubator-mxnet/pull/9460#discussion_r161949442 ## File path: docs/tutorials/basic/data.md ## @@ -180,6 +184,30 @@ mod =

[GitHub] cjolivier01 commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0

2018-01-16 Thread GitBox
cjolivier01 commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0 URL: https://github.com/apache/incubator-mxnet/issues/9396#issuecomment-358187266 (set before running python) This is an

[GitHub] cjolivier01 commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0

2018-01-16 Thread GitBox
cjolivier01 commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0 URL: https://github.com/apache/incubator-mxnet/issues/9396#issuecomment-358187229 can you try setting OMP_NUM_THREADS=1 in the environment and try again? does it speed up?

[GitHub] TaoLv commented on issue #3946: When predicting, does mxnet provide thread-safe interface?

2018-01-16 Thread GitBox
TaoLv commented on issue #3946: When predicting, does mxnet provide thread-safe interface? URL: https://github.com/apache/incubator-mxnet/issues/3946#issuecomment-357873369 @piiswrong @eric-haibin-lin Maybe not related. Can mxnet find two independent ops in a computation graph and

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161940072 ## File path: src/storage/cpu_device_storage.h ## @@ -54,7 +54,11 @@ class CPUDeviceStorage { /*! *

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161939043 ## File path: include/mxnet/ndarray.h ## @@ -611,6 +565,64 @@ class NDArray { <<

[incubator-mxnet] branch master updated: Remove empty lines at the end of file (#9459)

2018-01-16 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 9353c2e Remove empty lines at the end

[GitHub] eric-haibin-lin closed pull request #9459: Remove empty lines at the end of NOTICE file

2018-01-16 Thread GitBox
eric-haibin-lin closed pull request #9459: Remove empty lines at the end of NOTICE file URL: https://github.com/apache/incubator-mxnet/pull/9459 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

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161937980 ## File path: include/mxnet/ndarray.h ## @@ -611,6 +565,64 @@ class NDArray { <<

[GitHub] nicklhy commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0

2018-01-16 Thread GitBox
nicklhy commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0 URL: https://github.com/apache/incubator-mxnet/issues/9396#issuecomment-358157026 @eric-haibin-lin The above results are tested with a ResNet152 json file downloaded from

[GitHub] chinakook commented on issue #9458: Support auto infer input shape in convolution operator

2018-01-16 Thread GitBox
chinakook commented on issue #9458: Support auto infer input shape in convolution operator URL: https://github.com/apache/incubator-mxnet/issues/9458#issuecomment-358166890 Gluon can support this. This is an automated

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161936090 ## File path: include/mxnet/ndarray.h ## @@ -534,15 +491,12 @@ class NDArray {

[GitHub] mbaijal opened a new pull request #9462: [WIP][ReviewRequired] License Fixes post 1.0.0 Release

2018-01-16 Thread GitBox
mbaijal opened a new pull request #9462: [WIP][ReviewRequired] License Fixes post 1.0.0 Release URL: https://github.com/apache/incubator-mxnet/pull/9462 ## Description ## Fixing Licenses based on comments from 1.0.0 Release. Recorded in this wiki -

[GitHub] piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161931886 ## File path: include/mxnet/ndarray.h ## @@ -611,6 +565,64 @@ class NDArray { <<

[GitHub] piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161928578 ## File path: src/kvstore/kvstore_local.h ## @@ -256,7 +256,13 @@ class KVStoreLocal : public KVStore {

[GitHub] piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161932726 ## File path: include/mxnet/ndarray.h ## @@ -611,6 +565,64 @@ class NDArray { <<

[GitHub] piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161928812 ## File path: src/ndarray/ndarray.cc ## @@ -64,14 +166,55 @@ nnvm::Symbol NDArray::get_autograd_symbol() const

[GitHub] piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161926358 ## File path: include/mxnet/ndarray.h ## @@ -534,15 +491,12 @@ class NDArray {

[GitHub] piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161932918 ## File path: include/mxnet/ndarray.h ## @@ -611,6 +565,64 @@ class NDArray { <<

[GitHub] piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161928410 ## File path: src/executor/graph_executor.cc ## @@ -693,7 +701,7 @@ static NDArray ReshapeOrCreate(const

[GitHub] piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161930522 ## File path: src/storage/cpu_device_storage.h ## @@ -54,7 +54,11 @@ class CPUDeviceStorage { /*! *

[GitHub] piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161932822 ## File path: include/mxnet/ndarray.h ## @@ -611,6 +565,64 @@ class NDArray { <<

[GitHub] piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161932131 ## File path: include/mxnet/ndarray.h ## @@ -611,6 +565,64 @@ class NDArray { <<

[GitHub] piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161932398 ## File path: include/mxnet/ndarray.h ## @@ -611,6 +565,64 @@ class NDArray { <<

[GitHub] zhangguotai opened a new issue #9461: The fault in the process of running example

2018-01-16 Thread GitBox
zhangguotai opened a new issue #9461: The fault in the process of running example URL: https://github.com/apache/incubator-mxnet/issues/9461 I had compiled libmxnet.so and image-classification-predict.When running order that ./image-classification-predict apple.jpg,I got a fault that

[GitHub] piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161932783 ## File path: include/mxnet/ndarray.h ## @@ -611,6 +565,64 @@ class NDArray { <<

[GitHub] piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
piiswrong commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161933076 ## File path: include/mxnet/ndarray.h ## @@ -645,6 +657,12 @@ class NDArray { for csr,

[GitHub] nicklhy commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0

2018-01-16 Thread GitBox
nicklhy commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0 URL: https://github.com/apache/incubator-mxnet/issues/9396#issuecomment-358157026 @eric-haibin-lin The above results are tested with a ResNet152 json file downloaded from

[GitHub] nicklhy commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0

2018-01-16 Thread GitBox
nicklhy commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0 URL: https://github.com/apache/incubator-mxnet/issues/9396#issuecomment-358157026 @eric-haibin-lin The above results are tested from a ResNet152 json file downloaded from

[GitHub] nicklhy commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0

2018-01-16 Thread GitBox
nicklhy commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0 URL: https://github.com/apache/incubator-mxnet/issues/9396#issuecomment-358157026 @eric-haibin-lin The above results are tested with a ResNet152 json file downloaded from

[GitHub] szha commented on issue #7987: Per-sample gradients

2018-01-16 Thread GitBox
szha commented on issue #7987: Per-sample gradients URL: https://github.com/apache/incubator-mxnet/issues/7987#issuecomment-358153506 @apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and needs triage. For general "how-to" questions, our

[GitHub] pracheer commented on issue #8339: data iterators tutorial errors

2018-01-16 Thread GitBox
pracheer commented on issue #8339: data iterators tutorial errors URL: https://github.com/apache/incubator-mxnet/issues/8339#issuecomment-358152063 Apologies for the delay. Here is the pull request for it: https://github.com/apache/incubator-mxnet/pull/9460

[GitHub] pracheer commented on issue #9460: Data-iterator tutorial made python3 compatible.

2018-01-16 Thread GitBox
pracheer commented on issue #9460: Data-iterator tutorial made python3 compatible. URL: https://github.com/apache/incubator-mxnet/pull/9460#issuecomment-358151885 @aaronmarkham @eric-haibin-lin @piiswrong This is an

[GitHub] pracheer opened a new pull request #9460: Data-iterator tutorial made python3 compatible.

2018-01-16 Thread GitBox
pracheer opened a new pull request #9460: Data-iterator tutorial made python3 compatible. URL: https://github.com/apache/incubator-mxnet/pull/9460 ## Description ## Faced 2 main issues while executing this http://mxnet.incubator.apache.org/tutorials/basic/data.html tutorial on python3:

[GitHub] thinksanky commented on issue #46: Removed torch.html and all its references. Fixed Nesterov Momentum education ?

2018-01-16 Thread GitBox
thinksanky commented on issue #46: Removed torch.html and all its references. Fixed Nesterov Momentum education ? URL: https://github.com/apache/incubator-mxnet-site/pull/46#issuecomment-358150760 @sandeep-krishnamurthy - please help merge this PR. Also fixing the Master branch to

[GitHub] thinksanky commented on issue #46: Removed torch.html and all its references. Fixed Nesterov Momentum education ?

2018-01-16 Thread GitBox
thinksanky commented on issue #46: Removed torch.html and all its references. Fixed Nesterov Momentum education ? URL: https://github.com/apache/incubator-mxnet-site/pull/46#issuecomment-358150539 @sandeep-krishnamurthy,

[GitHub] thinksanky opened a new pull request #46: Removed torch.html and all its references. Fixed Nesterov Momentum education ?

2018-01-16 Thread GitBox
thinksanky opened a new pull request #46: Removed torch.html and all its references. Fixed Nesterov Momentum education ? URL: https://github.com/apache/incubator-mxnet-site/pull/46 Fixed the broken links for the following - - Torch html is not used anymore. So removed torch.html and all

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161921308 ## File path: src/common/exec_utils.h ## @@ -43,19 +43,61 @@ namespace common { indices are not

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161921308 ## File path: src/common/exec_utils.h ## @@ -43,19 +43,61 @@ namespace common { indices are not

[GitHub] zheng-da commented on issue #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
zheng-da commented on issue #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#issuecomment-358145000 @eric-haibin-lin the illegal instruction issue in CI has been resolved. The solution is in the latest commit.

[GitHub] sandeep-krishnamurthy commented on issue #42: Fix gpu install instructions

2018-01-16 Thread GitBox
sandeep-krishnamurthy commented on issue #42: Fix gpu install instructions URL: https://github.com/apache/incubator-mxnet-site/pull/42#issuecomment-358138951 Yes. Thanks. This is an automated message from the Apache Git

[GitHub] eric-haibin-lin commented on issue #8852: Environment variable MXNET_EXEC_BULK_EXEC_INFERENCE=0 does not work

2018-01-16 Thread GitBox
eric-haibin-lin commented on issue #8852: Environment variable MXNET_EXEC_BULK_EXEC_INFERENCE=0 does not work URL: https://github.com/apache/incubator-mxnet/issues/8852#issuecomment-358133146 Resolved by #9055 This is an

[GitHub] eric-haibin-lin closed issue #8852: Environment variable MXNET_EXEC_BULK_EXEC_INFERENCE=0 does not work

2018-01-16 Thread GitBox
eric-haibin-lin closed issue #8852: Environment variable MXNET_EXEC_BULK_EXEC_INFERENCE=0 does not work URL: https://github.com/apache/incubator-mxnet/issues/8852 This is an automated message from the Apache Git Service. To

[GitHub] eric-haibin-lin commented on issue #8592: Simple bind doesnt infer the provided stype for non default stypes

2018-01-16 Thread GitBox
eric-haibin-lin commented on issue #8592: Simple bind doesnt infer the provided stype for non default stypes URL: https://github.com/apache/incubator-mxnet/issues/8592#issuecomment-358132319 I think that caused by a typo in the test: ``` >>> xv = mx.symbol.Variable('x') >>> y =

[GitHub] eric-haibin-lin closed issue #8592: Simple bind doesnt infer the provided stype for non default stypes

2018-01-16 Thread GitBox
eric-haibin-lin closed issue #8592: Simple bind doesnt infer the provided stype for non default stypes URL: https://github.com/apache/incubator-mxnet/issues/8592 This is an automated message from the Apache Git Service. To

[GitHub] eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161899483 ## File path: include/mxnet/ndarray.h ## @@ -61,6 +62,9 @@ enum NDArrayStorageType { kDefaultStorage,

[GitHub] eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161903292 ## File path: src/common/exec_utils.h ## @@ -43,19 +43,61 @@ namespace common { indices are

[GitHub] eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161900792 ## File path: include/mxnet/ndarray.h ## @@ -789,15 +810,22 @@ class NDArray { // size is the

[GitHub] eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161905366 ## File path: src/ndarray/ndarray.cc ## @@ -64,14 +166,55 @@ nnvm::Symbol NDArray::get_autograd_symbol()

[GitHub] eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161901479 ## File path: include/mxnet/ndarray.h ## @@ -823,20 +851,24 @@ class NDArray { // storage shape is

[GitHub] eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161900341 ## File path: include/mxnet/ndarray.h ## @@ -611,6 +565,64 @@ class NDArray { <<

[GitHub] eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161899924 ## File path: include/mxnet/ndarray.h ## @@ -184,11 +138,17 @@ class NDArray { const TBlob ,

[GitHub] eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161904685 ## File path: src/executor/graph_executor.cc ## @@ -54,6 +54,14 @@ GraphExecutor::~GraphExecutor() {

[GitHub] eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
eric-haibin-lin commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r161902810 ## File path: src/common/utils.cc ## @@ -41,5 +41,21 @@ void CastStorageDispatch(const OpContext& ctx,

[GitHub] eric-haibin-lin commented on issue #9449: Can sequential module be distributed into mulit machines and how?

2018-01-16 Thread GitBox
eric-haibin-lin commented on issue #9449: Can sequential module be distributed into mulit machines and how? URL: https://github.com/apache/incubator-mxnet/issues/9449#issuecomment-358120806 I'm afraid not yet. What's your use case and why are you considering it?

[GitHub] mbaijal opened a new pull request #9459: Remove empty lines at the end of file

2018-01-16 Thread GitBox
mbaijal opened a new pull request #9459: Remove empty lines at the end of file URL: https://github.com/apache/incubator-mxnet/pull/9459 ## Description ## Remove blank lines from end of NOTICE file as requested in the 1.0.0 vote thread. Issue 10 discussed here -

[GitHub] eric-haibin-lin commented on issue #8914: The custom operator not supported for group context?

2018-01-16 Thread GitBox
eric-haibin-lin commented on issue #8914: The custom operator not supported for group context? URL: https://github.com/apache/incubator-mxnet/issues/8914#issuecomment-358111349 @mg0880gm could you try latest version of MXNet and update the `__init__` function of your `XXCustomOpProp`

[GitHub] eric-haibin-lin commented on issue #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
eric-haibin-lin commented on issue #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#issuecomment-358109197 @marcoabreu @zheng-da what's the action item to resolve the illegal instruction issue on CI? Do we need to update the setup in CI?

[GitHub] sandeep-krishnamurthy opened a new issue #9458: Support auto infer input shape in convolution operator

2018-01-16 Thread GitBox
sandeep-krishnamurthy opened a new issue #9458: Support auto infer input shape in convolution operator URL: https://github.com/apache/incubator-mxnet/issues/9458 MXNet does not support auto inferring of input shape for convolution operator. It would be very convenient for users to have

[GitHub] eric-haibin-lin commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0

2018-01-16 Thread GitBox
eric-haibin-lin commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0 URL: https://github.com/apache/incubator-mxnet/issues/9396#issuecomment-358107218 Hi @nicklhy what network are you using? Does it include any custom operators?

[GitHub] zhreshold commented on issue #8582: Yolo2 operator

2018-01-16 Thread GitBox
zhreshold commented on issue #8582: Yolo2 operator URL: https://github.com/apache/incubator-mxnet/pull/8582#issuecomment-358102363 @eric-haibin-lin CI build problem solved, could you have a look? This is an automated message

[GitHub] zheng-da commented on issue #8302: Refactor operators & MKLDNN

2018-01-16 Thread GitBox
zheng-da commented on issue #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#issuecomment-358086101 The problem that mkldnn+GPU can't run on G3 is caused by the fact that mkldnn is compiled with avx512 instructions on C5 but G3 doesn't support

[GitHub] piiswrong closed pull request #9447: Fix broken links in model_parallel_lstm.md

2018-01-16 Thread GitBox
piiswrong closed pull request #9447: Fix broken links in model_parallel_lstm.md URL: https://github.com/apache/incubator-mxnet/pull/9447 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

[incubator-mxnet] branch master updated: Update model_parallel_lstm.md (#9447)

2018-01-16 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 a182aec Update model_parallel_lstm.md

[incubator-mxnet] branch master updated: correct usage of bool arguments from command line (#8880)

2018-01-16 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 89b4fa0 correct usage of bool arguments

[GitHub] piiswrong closed pull request #8880: correct usage of bool arguments from command line

2018-01-16 Thread GitBox
piiswrong closed pull request #8880: correct usage of bool arguments from command line URL: https://github.com/apache/incubator-mxnet/pull/8880 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

[GitHub] szha commented on issue #6257: Merge AMD-HIP port

2018-01-16 Thread GitBox
szha commented on issue #6257: Merge AMD-HIP port URL: https://github.com/apache/incubator-mxnet/issues/6257#issuecomment-358082623 @sriharikarnam submodules are versioned by commit hash, so you may go ahead and make changes in the submodules first, and once merged, update the submodules

[GitHub] cjolivier01 commented on a change in pull request #9369: UT fix for Windows

2018-01-16 Thread GitBox
cjolivier01 commented on a change in pull request #9369: UT fix for Windows URL: https://github.com/apache/incubator-mxnet/pull/9369#discussion_r161864850 ## File path: src/operator/nn/batch_norm-inl.h ## @@ -56,7 +56,7 @@ constexpr int DEFAULT_AXIS = 1; } // namespace

[GitHub] cjolivier01 commented on a change in pull request #9369: UT fix for Windows

2018-01-16 Thread GitBox
cjolivier01 commented on a change in pull request #9369: UT fix for Windows URL: https://github.com/apache/incubator-mxnet/pull/9369#discussion_r161858079 ## File path: tests/CMakeLists.txt ## @@ -21,7 +21,7 @@ if(NOT MSVC) endif() # FIXME MSVC unit test linking issue

[GitHub] cjolivier01 commented on a change in pull request #9369: UT fix for Windows

2018-01-16 Thread GitBox
cjolivier01 commented on a change in pull request #9369: UT fix for Windows URL: https://github.com/apache/incubator-mxnet/pull/9369#discussion_r161857886 ## File path: src/operator/nn/batch_norm-inl.h ## @@ -56,7 +56,7 @@ constexpr int DEFAULT_AXIS = 1; } // namespace

[GitHub] cjolivier01 commented on a change in pull request #9369: UT fix for Windows

2018-01-16 Thread GitBox
cjolivier01 commented on a change in pull request #9369: UT fix for Windows URL: https://github.com/apache/incubator-mxnet/pull/9369#discussion_r161857543 ## File path: src/operator/nn/batch_norm-imp.h ## @@ -0,0 +1,22 @@ + +#ifndef MXNET_OPERATOR_NN_BATCH_NORM_IMP_H_

[GitHub] thinksanky commented on issue #43: fixed version drop down selection menu bar for 0.12.0 and 0.12.1 vers?

2018-01-16 Thread GitBox
thinksanky commented on issue #43: fixed version drop down selection menu bar for 0.12.0 and 0.12.1 vers? URL: https://github.com/apache/incubator-mxnet-site/pull/43#issuecomment-358075434 @szha - This change is only for versioned website (Static) which do not run nightly. So they are

[GitHub] thinksanky commented on issue #42: Fix gpu install instructions

2018-01-16 Thread GitBox
thinksanky commented on issue #42: Fix gpu install instructions URL: https://github.com/apache/incubator-mxnet-site/pull/42#issuecomment-358074588 When you say MXNet Code base you mean master ? Master already had these changes and I didnt have to change this.

[GitHub] cjolivier01 commented on issue #9369: UT fix for Windows

2018-01-16 Thread GitBox
cjolivier01 commented on issue #9369: UT fix for Windows URL: https://github.com/apache/incubator-mxnet/pull/9369#issuecomment-358073203 Would it be better to just have the WIndows unit tests link statically like the UNiex tests do? That way, you wouldn't need to export internal classes

[incubator-mxnet-site] branch asf-site updated (8d0313b -> 942cfb7)

2018-01-16 Thread haibin
This is an automated email from the ASF dual-hosted git repository. haibin pushed a change to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git. from 8d0313b Nightly build add 6ae4ef8 fixed broken links for scala index page new 942cfb7

[incubator-mxnet-site] 01/01: Merge pull request #45 from thinksanky/fix-broken-links-01152018

2018-01-16 Thread haibin
This is an automated email from the ASF dual-hosted git repository. haibin pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git commit 942cfb71559a9f4c645df60ff5b8ceadbe80cc45 Merge: 8d0313b 6ae4ef8 Author: Haibin Lin

[GitHub] eric-haibin-lin closed pull request #45: fixed broken links for scala index page

2018-01-16 Thread GitBox
eric-haibin-lin closed pull request #45: fixed broken links for scala index page URL: https://github.com/apache/incubator-mxnet-site/pull/45 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

[GitHub] thinksanky commented on issue #45: fixed broken links for scala index page

2018-01-16 Thread GitBox
thinksanky commented on issue #45: fixed broken links for scala index page URL: https://github.com/apache/incubator-mxnet-site/pull/45#issuecomment-358068795 @eric-haibin-lin - please have a look into this. Thanks This is an

[GitHub] thinksanky opened a new pull request #45: fixed broken links for scala index page

2018-01-16 Thread GitBox
thinksanky opened a new pull request #45: fixed broken links for scala index page URL: https://github.com/apache/incubator-mxnet-site/pull/45 ## Description ## Fixed Scala API index.html to reference to the right URLs

[incubator-mxnet] branch master updated: text: use _contan, import modules, and keep code style consistent (#9446)

2018-01-16 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 b396bc3 text: use _contan, import

[GitHub] piiswrong closed pull request #9446: text: use _contan, import modules, and keep code style consistent

2018-01-16 Thread GitBox
piiswrong closed pull request #9446: text: use _contan, import modules, and keep code style consistent URL: https://github.com/apache/incubator-mxnet/pull/9446 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

[GitHub] eric-haibin-lin commented on issue #9452: A new deep learning visualization tool from baidu support MXNet!

2018-01-16 Thread GitBox
eric-haibin-lin commented on issue #9452: A new deep learning visualization tool from baidu support MXNet! URL: https://github.com/apache/incubator-mxnet/issues/9452#issuecomment-358062669 @reminisce @zihaolucky This is an

  1   2   >