[GitHub] IrvingShu commented on issue #9920: Asynchronous Issue on CustomOP and mxnet.image.ImageDetIter

2018-04-19 Thread GitBox
IrvingShu commented on issue #9920: Asynchronous Issue on CustomOP and mxnet.image.ImageDetIter URL: https://github.com/apache/incubator-mxnet/issues/9920#issuecomment-382997920 @wkcn thanks for sharing your ways. I added ctx = context.cpu(), however, the new problem happens: /usr/l

[GitHub] lionel92 commented on issue #10599: [MKLDNN Bug] MKLDNN eats lots of memory and then crash down.

2018-04-19 Thread GitBox
lionel92 commented on issue #10599: [MKLDNN Bug] MKLDNN eats lots of memory and then crash down. URL: https://github.com/apache/incubator-mxnet/issues/10599#issuecomment-382997495 @zheng-da @pengzhao-intel I reproduced this case, and per the memory allocation profiling (via the conten

[GitHub] qxin commented on issue #9275: "bad" mAP score on COCO

2018-04-19 Thread GitBox
qxin commented on issue #9275: "bad" mAP score on COCO URL: https://github.com/apache/incubator-mxnet/issues/9275#issuecomment-382994431 @sandeep-krishnamurthy Thanks ~ (and sorry, for the inactivity) This is an automated mes

[GitHub] zheng-da commented on issue #10599: [MKLDNN Bug] MKLDNN eats lots of memory and then crash down.

2018-04-19 Thread GitBox
zheng-da commented on issue #10599: [MKLDNN Bug] MKLDNN eats lots of memory and then crash down. URL: https://github.com/apache/incubator-mxnet/issues/10599#issuecomment-382989367 The memory used by MKLDNN is also allocated by the MXNet memory allocator. MKLDNN NDArrays are reused in the

[GitHub] liangfu commented on issue #10595: Update mobilenetv2 symbol definition

2018-04-19 Thread GitBox
liangfu commented on issue #10595: Update mobilenetv2 symbol definition URL: https://github.com/apache/incubator-mxnet/pull/10595#issuecomment-382986585 add a link to pretrained model ? This is an automated message from the Ap

[GitHub] hongtao12310 commented on issue #10579: NNPACK can't be compiled at 1.1.0 1.2.0 and master branch

2018-04-19 Thread GitBox
hongtao12310 commented on issue #10579: NNPACK can't be compiled at 1.1.0 1.2.0 and master branch URL: https://github.com/apache/incubator-mxnet/issues/10579#issuecomment-382985784 mkldnn can be built. but nnpack was still can't be built , i reviewed the code, sounds like there are some i

[GitHub] hetong007 commented on issue #10595: Update mobilenetv2 symbol definition

2018-04-19 Thread GitBox
hetong007 commented on issue #10595: Update mobilenetv2 symbol definition URL: https://github.com/apache/incubator-mxnet/pull/10595#issuecomment-382981055 LGTM This is an automated message from the Apache Git Service. To respo

[GitHub] chinakook commented on issue #10599: [MKLDNN Bug] MKLDNN eats lots of memory and then crash down.

2018-04-19 Thread GitBox
chinakook commented on issue #10599: [MKLDNN Bug] MKLDNN eats lots of memory and then crash down. URL: https://github.com/apache/incubator-mxnet/issues/10599#issuecomment-382980794 I think the MKLDNN may use its own allocation engine, other than MXNet allocation engine. The former may not

[GitHub] chinakook commented on issue #10599: [MKLDNN Bug] MKLDNN eats lots of memory and then crash down.

2018-04-19 Thread GitBox
chinakook commented on issue #10599: [MKLDNN Bug] MKLDNN eats lots of memory and then crash down. URL: https://github.com/apache/incubator-mxnet/issues/10599#issuecomment-382980794 I think the MKLDNN may use its own allocation engine, other than MXNet allocation engine. The former may not

[GitHub] zhreshold commented on issue #10595: Update mobilenetv2 symbol definition

2018-04-19 Thread GitBox
zhreshold commented on issue #10595: Update mobilenetv2 symbol definition URL: https://github.com/apache/incubator-mxnet/pull/10595#issuecomment-382980813 Any other comments? @szha @hetong007 This is an automated message from

[GitHub] szha commented on issue #10618: [MXNET-338] Fix symbol boolean evaluation

2018-04-19 Thread GitBox
szha commented on issue #10618: [MXNET-338] Fix symbol boolean evaluation URL: https://github.com/apache/incubator-mxnet/pull/10618#issuecomment-382973926 Thanks for fixing the bug. This is an automated message from the Apache

[incubator-mxnet] branch master updated: [MXNET-338] Fix symbol boolean evaluation (#10618)

2018-04-19 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng 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 6fb7c85 [MXNET-338] Fix symbol boole

[GitHub] szha closed pull request #10618: [MXNET-338] Fix symbol boolean evaluation

2018-04-19 Thread GitBox
szha closed pull request #10618: [MXNET-338] Fix symbol boolean evaluation URL: https://github.com/apache/incubator-mxnet/pull/10618 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 for

[GitHub] zheng-da opened a new pull request #10624: Fix a bug in the MKLDNN integration.

2018-04-19 Thread GitBox
zheng-da opened a new pull request #10624: Fix a bug in the MKLDNN integration. URL: https://github.com/apache/incubator-mxnet/pull/10624 ## Description ## This is to fix a bug in https://github.com/apache/incubator-mxnet/issues/10580 The bug happens when an NDArray already has a MKLD

[GitHub] 2016xjtuzyt closed issue #8443: (err) == (CUDNN_STATUS_SUCCESS) CUDNN_STATUS_NOT_INITIALIZED

2018-04-19 Thread GitBox
2016xjtuzyt closed issue #8443: (err) == (CUDNN_STATUS_SUCCESS) CUDNN_STATUS_NOT_INITIALIZED URL: https://github.com/apache/incubator-mxnet/issues/8443 This is an automated message from the Apache Git Service. To respond to

[GitHub] 2016xjtuzyt commented on issue #8443: (err) == (CUDNN_STATUS_SUCCESS) CUDNN_STATUS_NOT_INITIALIZED

2018-04-19 Thread GitBox
2016xjtuzyt commented on issue #8443: (err) == (CUDNN_STATUS_SUCCESS) CUDNN_STATUS_NOT_INITIALIZED URL: https://github.com/apache/incubator-mxnet/issues/8443#issuecomment-382966981 the problem has been solved by update cuda driver! -

[GitHub] chinakook commented on issue #10611: Reshape network

2018-04-19 Thread GitBox
chinakook commented on issue #10611: Reshape network URL: https://github.com/apache/incubator-mxnet/issues/10611#issuecomment-382966512 You only need to compile the newest mxnet by yourself. Then the MXPredReshape can be used on windows. ---

[GitHub] chinakook commented on issue #10612: MXPredReshape support Windows

2018-04-19 Thread GitBox
chinakook commented on issue #10612: MXPredReshape support Windows URL: https://github.com/apache/incubator-mxnet/pull/10612#issuecomment-382965410 Yes, It's good for product implementation. This is an automated message from t

[GitHub] sergeykolychev commented on issue #10620: MXNET-335 [Perl] Support for image operations on symbols and ndarrays.

2018-04-19 Thread GitBox
sergeykolychev commented on issue #10620: MXNET-335 [Perl] Support for image operations on symbols and ndarrays. URL: https://github.com/apache/incubator-mxnet/pull/10620#issuecomment-382962092 @tlby Thank you. This is an aut

[GitHub] smartadpole commented on issue #10612: MXPredReshape support Windows

2018-04-19 Thread GitBox
smartadpole commented on issue #10612: MXPredReshape support Windows URL: https://github.com/apache/incubator-mxnet/pull/10612#issuecomment-382955510 @zheng-da 我在用C的接口,但是整个检测项目还没跑通,还在做. - I am useing prediction c API, but it has n

[GitHub] smartadpole commented on issue #10612: MXPredReshape support Windows

2018-04-19 Thread GitBox
smartadpole commented on issue #10612: MXPredReshape support Windows URL: https://github.com/apache/incubator-mxnet/pull/10612#issuecomment-382955510 @zheng-da 我在用C的接口,但是整个检测项目还没跑通,还在做. - @zheng-da I am useing prediction c API, but it

[GitHub] smartadpole commented on issue #10611: Reshape network

2018-04-19 Thread GitBox
smartadpole commented on issue #10611: Reshape network URL: https://github.com/apache/incubator-mxnet/issues/10611#issuecomment-382954461 @chinakook 谢谢您的回答。我还有几个问题: “Add MXPredReshape API MXNET_DLL prefix” 的操作是mxnet的开发者做吗,还是我做?我做的话可能要重编源码,是这样吗? 既然现在要导出MXPredReshape ,为什么一开始没有这样做呢?是编码失

[GitHub] ZiyueHuang commented on issue #10615: [MXNET-334] fix dtype of Embedding and add dtype argument to Dense

2018-04-19 Thread GitBox
ZiyueHuang commented on issue #10615: [MXNET-334] fix dtype of Embedding and add dtype argument to Dense URL: https://github.com/apache/incubator-mxnet/pull/10615#issuecomment-382950343 cc @szha This is an automated message

[GitHub] ashokei commented on a change in pull request #10616: ensure same mkldnn engine is used for consistency

2018-04-19 Thread GitBox
ashokei commented on a change in pull request #10616: ensure same mkldnn engine is used for consistency URL: https://github.com/apache/incubator-mxnet/pull/10616#discussion_r182928559 ## File path: src/operator/nn/mkldnn/mkldnn_base-inl.h ## @@ -67,7 +67,8 @@ class CpuEngi

[GitHub] TaoLv commented on a change in pull request #10616: ensure same mkldnn engine is used for consistency

2018-04-19 Thread GitBox
TaoLv commented on a change in pull request #10616: ensure same mkldnn engine is used for consistency URL: https://github.com/apache/incubator-mxnet/pull/10616#discussion_r182928254 ## File path: src/operator/nn/mkldnn/mkldnn_base-inl.h ## @@ -67,7 +67,8 @@ class CpuEngine

[GitHub] ariwaranosai closed issue #10601: load parameters error with the same code exported by HybridBlock

2018-04-19 Thread GitBox
ariwaranosai closed issue #10601: load parameters error with the same code exported by HybridBlock URL: https://github.com/apache/incubator-mxnet/issues/10601 This is an automated message from the Apache Git Service. To res

[GitHub] ariwaranosai closed pull request #10604: fix strip prefix in HybridBlock with parameter to control it.

2018-04-19 Thread GitBox
ariwaranosai closed pull request #10604: fix strip prefix in HybridBlock with parameter to control it. URL: https://github.com/apache/incubator-mxnet/pull/10604 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] ariwaranosai commented on issue #10604: fix strip prefix in HybridBlock with parameter to control it.

2018-04-19 Thread GitBox
ariwaranosai commented on issue #10604: fix strip prefix in HybridBlock with parameter to control it. URL: https://github.com/apache/incubator-mxnet/pull/10604#issuecomment-382937049 thanks,i will try it. This is an automated

[GitHub] ThomasDelteil opened a new pull request #10623: [MXNET-339] Add warning on input data for RNN for layout NTC

2018-04-19 Thread GitBox
ThomasDelteil opened a new pull request #10623: [MXNET-339] Add warning on input data for RNN for layout NTC URL: https://github.com/apache/incubator-mxnet/pull/10623 ## Description ## (Brief description on what this PR is about) Add comment in docs about extra mem copy for NTC layout

[GitHub] zheng-da commented on a change in pull request #10591: handle inplace in mkldnn FallBackCompute

2018-04-19 Thread GitBox
zheng-da commented on a change in pull request #10591: handle inplace in mkldnn FallBackCompute URL: https://github.com/apache/incubator-mxnet/pull/10591#discussion_r182921389 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -294,10 +294,15 @@ void FallBackCompute

[GitHub] thomelane opened a new pull request #10622: Added to API Docs for Python and Gluon

2018-04-19 Thread GitBox
thomelane opened a new pull request #10622: Added to API Docs for Python and Gluon URL: https://github.com/apache/incubator-mxnet/pull/10622 ## Description ## With the aim to removing the main navigation link to Gluon, these changes are to provide extra context around Module vs Gluo

[GitHub] thomelane opened a new pull request #10621: Updated tutorials page.

2018-04-19 Thread GitBox
thomelane opened a new pull request #10621: Updated tutorials page. URL: https://github.com/apache/incubator-mxnet/pull/10621 ## Description ## A refactor of the existing tutorial page. * Reworded the introduction. * Added more structure to the tutorials (nested hierarchy)

[GitHub] ThomasDelteil commented on issue #10387: Flaky test(scala): test_arange

2018-04-19 Thread GitBox
ThomasDelteil commented on issue #10387: Flaky test(scala): test_arange URL: https://github.com/apache/incubator-mxnet/issues/10387#issuecomment-382921869 Bumping, happened again http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-10608/9/pipeline

[GitHub] ThomasDelteil commented on issue #10387: Flaky test(scala): test_arange

2018-04-19 Thread GitBox
ThomasDelteil commented on issue #10387: Flaky test(scala): test_arange URL: https://github.com/apache/incubator-mxnet/issues/10387#issuecomment-382921869 Bumping, happened again This is an automated message from the Apache G

[GitHub] li-haoran commented on issue #10610: memory increase when use adam and rmsprop.

2018-04-19 Thread GitBox
li-haoran commented on issue #10610: memory increase when use adam and rmsprop. URL: https://github.com/apache/incubator-mxnet/issues/10610#issuecomment-382920764 ok, i posted. This is an automated message from the Apache Gi

[GitHub] haojin2 commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-19 Thread GitBox
haojin2 commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r182918193 ## File path: src/operator/tensor/dot.cc ## @@ -51,13 +1

[GitHub] haojin2 commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-19 Thread GitBox
haojin2 commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r182918204 ## File path: src/operator/tensor/dot-inl.cuh ## @@ -442

[GitHub] haojin2 commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-19 Thread GitBox
haojin2 commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r182918133 ## File path: src/operator/tensor/dot-inl.cuh ## @@ -895

[GitHub] haojin2 commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-19 Thread GitBox
haojin2 commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r182918157 ## File path: src/operator/tensor/dot-inl.cuh ## @@ -895

[incubator-mxnet] branch master updated: standard sgd_update (#10614)

2018-04-19 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 9ccd647 standard sgd_update (#10614) 9

[GitHub] eric-haibin-lin closed pull request #10614: [MXNET-333] standard sgd_update for sparse tensors with momentum=0.0

2018-04-19 Thread GitBox
eric-haibin-lin closed pull request #10614: [MXNET-333] standard sgd_update for sparse tensors with momentum=0.0 URL: https://github.com/apache/incubator-mxnet/pull/10614 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for th

[GitHub] eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-19 Thread GitBox
eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r182913804 ## File path: src/operator/tensor/dot.cc ## @@ -

[GitHub] eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-19 Thread GitBox
eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r182915535 ## File path: src/operator/tensor/dot-inl.cuh ##

[GitHub] eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-19 Thread GitBox
eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r182915139 ## File path: src/operator/tensor/dot-inl.cuh ##

[GitHub] eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-19 Thread GitBox
eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r182914225 ## File path: tests/python/unittest/test_sparse_operator.p

[GitHub] eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-19 Thread GitBox
eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r182912981 ## File path: src/operator/tensor/dot-inl.cuh ##

[GitHub] eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-19 Thread GitBox
eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r182914018 ## File path: tests/python/unittest/test_sparse_operator.p

[GitHub] eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-19 Thread GitBox
eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r182914873 ## File path: src/operator/tensor/dot.cu ## @@ -

[GitHub] eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-19 Thread GitBox
eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r182913349 ## File path: src/operator/tensor/dot.cc ## @@ -

[GitHub] eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-19 Thread GitBox
eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r182913680 ## File path: src/operator/tensor/dot.cc ## @@ -

[GitHub] eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-19 Thread GitBox
eric-haibin-lin commented on a change in pull request #10371: [MXNET-263] [WIP] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#discussion_r182914587 ## File path: tests/python/unittest/test_sparse_operator.p

[GitHub] haojin2 commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors

2018-04-19 Thread GitBox
haojin2 commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors URL: https://github.com/apache/incubator-mxnet/pull/10550#discussion_r182915603 ## File path: src/operator/tensor/elemwise_binary_op.h ## @@ -305,6 +316

[GitHub] haojin2 commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors

2018-04-19 Thread GitBox
haojin2 commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors URL: https://github.com/apache/incubator-mxnet/pull/10550#discussion_r182915595 ## File path: src/operator/tensor/elemwise_binary_op-inl.h ## @@ -374,6

[GitHub] haojin2 commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors

2018-04-19 Thread GitBox
haojin2 commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors URL: https://github.com/apache/incubator-mxnet/pull/10550#discussion_r182915599 ## File path: src/operator/tensor/elemwise_binary_op_basic.cc ## @@ -166

[GitHub] zheng-da commented on issue #10619: handle NDArray slice properly for mkldnn layout

2018-04-19 Thread GitBox
zheng-da commented on issue #10619: handle NDArray slice properly for mkldnn layout URL: https://github.com/apache/incubator-mxnet/pull/10619#issuecomment-382915326 can you also add the code snippet as a test? This is an auto

[GitHub] eric-haibin-lin commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors

2018-04-19 Thread GitBox
eric-haibin-lin commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors URL: https://github.com/apache/incubator-mxnet/pull/10550#discussion_r182912098 ## File path: src/operator/tensor/elemwise_binary_op_basic.cc ##

[GitHub] eric-haibin-lin commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors

2018-04-19 Thread GitBox
eric-haibin-lin commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors URL: https://github.com/apache/incubator-mxnet/pull/10550#discussion_r182912264 ## File path: src/operator/tensor/elemwise_binary_op-inl.h ## @@

[GitHub] eric-haibin-lin commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors

2018-04-19 Thread GitBox
eric-haibin-lin commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors URL: https://github.com/apache/incubator-mxnet/pull/10550#discussion_r182912399 ## File path: src/operator/tensor/elemwise_binary_op.h ## @@ -30

[GitHub] sergeykolychev commented on issue #10620: MXNET-335 [Perl] Support for image operations on symbols and ndarrays.

2018-04-19 Thread GitBox
sergeykolychev commented on issue #10620: MXNET-335 [Perl] Support for image operations on symbols and ndarrays. URL: https://github.com/apache/incubator-mxnet/pull/10620#issuecomment-382908274 @tlby Hi Robert, could you please review when you have time. Thanks! ---

[GitHub] sergeykolychev opened a new pull request #10620: MXNET-335 [Perl] Support for image operations on symbols and ndarrays.

2018-04-19 Thread GitBox
sergeykolychev opened a new pull request #10620: MXNET-335 [Perl] Support for image operations on symbols and ndarrays. URL: https://github.com/apache/incubator-mxnet/pull/10620 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ###

[GitHub] leopd closed issue #10158: Assertion fail when creating very large matrix

2018-04-19 Thread GitBox
leopd closed issue #10158: Assertion fail when creating very large matrix URL: https://github.com/apache/incubator-mxnet/issues/10158 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] spidyDev commented on issue #10519: Can too many connection for the symbol make unintended computation?

2018-04-19 Thread GitBox
spidyDev commented on issue #10519: Can too many connection for the symbol make unintended computation? URL: https://github.com/apache/incubator-mxnet/issues/10519#issuecomment-382892116 @eric-haibin-lin : Please label: Question, Operator --

[GitHub] spidyDev commented on issue #10502: MXPredReshape

2018-04-19 Thread GitBox
spidyDev commented on issue #10502: MXPredReshape URL: https://github.com/apache/incubator-mxnet/issues/10502#issuecomment-382891874 @eric-haibin-lin Please label it as: Question This is an automated message from the Apache

[GitHub] ashokei commented on issue #10619: handle NDArray slice properly for mkldnn layout

2018-04-19 Thread GitBox
ashokei commented on issue #10619: handle NDArray slice properly for mkldnn layout URL: https://github.com/apache/incubator-mxnet/pull/10619#issuecomment-382882430 @zheng-da please review, thanks. This is an automated message

[GitHub] ashokei opened a new pull request #10619: handle NDArray slice properly for mkldnn layout

2018-04-19 Thread GitBox
ashokei opened a new pull request #10619: handle NDArray slice properly for mkldnn layout URL: https://github.com/apache/incubator-mxnet/pull/10619 ## Description ## when NDArray ```asnumpy``` is called for a view of NDArray, and if the original NDArray has mkldnn layout, a crash is see

[GitHub] szha commented on a change in pull request #10617: [MXNET-337] Change Model url to S3

2018-04-19 Thread GitBox
szha commented on a change in pull request #10617: [MXNET-337] Change Model url to S3 URL: https://github.com/apache/incubator-mxnet/pull/10617#discussion_r182884567 ## File path: scala-package/examples/src/test/scala/org/apache/mxnetexamples/infer/objectdetector/ObjectDetectorExam

[GitHub] astonzhang opened a new pull request #10618: Fix symbol boolean evaluation

2018-04-19 Thread GitBox
astonzhang opened a new pull request #10618: Fix symbol boolean evaluation URL: https://github.com/apache/incubator-mxnet/pull/10618 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items

[GitHub] zhreshold commented on a change in pull request #10595: Update mobilenetv2 symbol definition

2018-04-19 Thread GitBox
zhreshold commented on a change in pull request #10595: Update mobilenetv2 symbol definition URL: https://github.com/apache/incubator-mxnet/pull/10595#discussion_r182880180 ## File path: example/image-classification/symbols/mobilenetv2.py ## @@ -17,60 +17,202 @@ # -*- c

[GitHub] zhreshold commented on a change in pull request #10595: Update mobilenetv2 symbol definition

2018-04-19 Thread GitBox
zhreshold commented on a change in pull request #10595: Update mobilenetv2 symbol definition URL: https://github.com/apache/incubator-mxnet/pull/10595#discussion_r182880077 ## File path: example/image-classification/symbols/mobilenetv2.py ## @@ -17,60 +17,202 @@ # -*- c

[GitHub] ThomasDelteil commented on a change in pull request #10617: [MXNET-337] Change Model url to S3

2018-04-19 Thread GitBox
ThomasDelteil commented on a change in pull request #10617: [MXNET-337] Change Model url to S3 URL: https://github.com/apache/incubator-mxnet/pull/10617#discussion_r182877436 ## File path: scala-package/examples/src/test/scala/org/apache/mxnetexamples/infer/objectdetector/ObjectDet

[GitHub] aaronmarkham commented on issue #60: Site refresh

2018-04-19 Thread GitBox
aaronmarkham commented on issue #60: Site refresh URL: https://github.com/apache/incubator-mxnet-site/pull/60#issuecomment-382870876 Yes, it's too big to review. I'm going to try to split this up by version branch. This is a

[GitHub] ThomasDelteil commented on issue #9553: test_operator_gpu.test_correlation fails

2018-04-19 Thread GitBox
ThomasDelteil commented on issue #9553: test_operator_gpu.test_correlation fails URL: https://github.com/apache/incubator-mxnet/issues/9553#issuecomment-382870242 Seems to be flaky again, can we reopen?: ``` ==

[GitHub] ThomasDelteil commented on issue #9553: test_operator_gpu.test_correlation fails

2018-04-19 Thread GitBox
ThomasDelteil commented on issue #9553: test_operator_gpu.test_correlation fails URL: https://github.com/apache/incubator-mxnet/issues/9553#issuecomment-382870242 Seems to be flaky again: ``` == FAIL: test_oper

[GitHub] Roshrini commented on issue #10379: When to use SyncCopyFromCPU and SyncCopyToCPU - C++

2018-04-19 Thread GitBox
Roshrini commented on issue #10379: When to use SyncCopyFromCPU and SyncCopyToCPU - C++ URL: https://github.com/apache/incubator-mxnet/issues/10379#issuecomment-382869834 @mrRo8o7 Please close the issue if your query is resolved. Also, Please use the user forum for how-to questions. Yo

[GitHub] Roshrini commented on issue #10395: Android build documentation

2018-04-19 Thread GitBox
Roshrini commented on issue #10395: Android build documentation URL: https://github.com/apache/incubator-mxnet/issues/10395#issuecomment-382868338 @nswamy Can you please add labels- Doc, Installation This is an automated

[GitHub] Roshrini commented on issue #10397: grad_req in multi-task example

2018-04-19 Thread GitBox
Roshrini commented on issue #10397: grad_req in multi-task example URL: https://github.com/apache/incubator-mxnet/issues/10397#issuecomment-382867890 Please use the user forum for how-to questions. You'll get quicker responses there. https://discuss.mxnet.io/ @reminisce Can you please

[GitHub] Roshrini commented on issue #10617: [MXNET-337] Change Model url to S3

2018-04-19 Thread GitBox
Roshrini commented on issue #10617: [MXNET-337] Change Model url to S3 URL: https://github.com/apache/incubator-mxnet/pull/10617#issuecomment-382866102 Thank you! LGTM This is an automated message from the Apache Git Service.

[GitHub] aaronmarkham commented on a change in pull request #60: Site refresh

2018-04-19 Thread GitBox
aaronmarkham commented on a change in pull request #60: Site refresh URL: https://github.com/apache/incubator-mxnet-site/pull/60#discussion_r182867574 ## File path: .htaccess ## @@ -1,5 +0,0 @@ -RewriteEngine on Review comment: No. That's not right. I'll see what's u

[GitHub] lanking520 commented on issue #10432: [scala-infer]move test files for scala infer API to s3

2018-04-19 Thread GitBox
lanking520 commented on issue #10432: [scala-infer]move test files for scala infer API to s3 URL: https://github.com/apache/incubator-mxnet/issues/10432#issuecomment-382860005 Here you go: https://github.com/apache/incubator-mxnet/pull/10617 ---

[GitHub] lanking520 opened a new pull request #10617: [MXNET-337] Change Model url to S3

2018-04-19 Thread GitBox
lanking520 opened a new pull request #10617: [MXNET-337] Change Model url to S3 URL: https://github.com/apache/incubator-mxnet/pull/10617 ## Description ## Change the links for the models to S3 @CodingCat @Roshrini ## Checklist ## ### Essentials ### Please feel free to re

[GitHub] ThomasDelteil commented on a change in pull request #10608: [MXNET-307] Add tutorial tests to the CI

2018-04-19 Thread GitBox
ThomasDelteil commented on a change in pull request #10608: [MXNET-307] Add tutorial tests to the CI URL: https://github.com/apache/incubator-mxnet/pull/10608#discussion_r182786851 ## File path: tests/tutorials/test_tutorials.py ## @@ -0,0 +1,187 @@ +# Licensed to the Apac

[GitHub] ThomasDelteil commented on a change in pull request #10608: [MXNET-307] Add tutorial tests to the CI

2018-04-19 Thread GitBox
ThomasDelteil commented on a change in pull request #10608: [MXNET-307] Add tutorial tests to the CI URL: https://github.com/apache/incubator-mxnet/pull/10608#discussion_r182786851 ## File path: tests/tutorials/test_tutorials.py ## @@ -0,0 +1,187 @@ +# Licensed to the Apac

[GitHub] ThomasDelteil commented on a change in pull request #10608: [MXNET-307] Add tutorial tests to the CI

2018-04-19 Thread GitBox
ThomasDelteil commented on a change in pull request #10608: [MXNET-307] Add tutorial tests to the CI URL: https://github.com/apache/incubator-mxnet/pull/10608#discussion_r182857076 ## File path: tests/tutorials/test_tutorials.py ## @@ -0,0 +1,187 @@ +# Licensed to the Apac

[GitHub] ThomasDelteil commented on issue #10608: [MXNET-307] Add tutorial tests to the CI

2018-04-19 Thread GitBox
ThomasDelteil commented on issue #10608: [MXNET-307] Add tutorial tests to the CI URL: https://github.com/apache/incubator-mxnet/pull/10608#issuecomment-382849815 @marcoabreu thanks for reviewing, I addressed your comments Th

[GitHub] ThomasDelteil commented on issue #10608: [MXNET-307] Add tutorial tests to the CI

2018-04-19 Thread GitBox
ThomasDelteil commented on issue #10608: [MXNET-307] Add tutorial tests to the CI URL: https://github.com/apache/incubator-mxnet/pull/10608#issuecomment-382849815 @marcoabreu addressed your comments This is an automated messa

[GitHub] spidyDev commented on issue #10552: Build error with NNPACK (fix included)

2018-04-19 Thread GitBox
spidyDev commented on issue #10552: Build error with NNPACK (fix included) URL: https://github.com/apache/incubator-mxnet/issues/10552#issuecomment-382849352 @eric-haibin-lin : Please label this as: Build This is an automated

[GitHub] ThomasDelteil commented on a change in pull request #10608: [MXNET-307] Add tutorial tests to the CI

2018-04-19 Thread GitBox
ThomasDelteil commented on a change in pull request #10608: [MXNET-307] Add tutorial tests to the CI URL: https://github.com/apache/incubator-mxnet/pull/10608#discussion_r182854976 ## File path: tests/tutorials/test_tutorials.py ## @@ -0,0 +1,187 @@ +# Licensed to the Apac

[GitHub] spidyDev commented on issue #10553: Build error for OSX with CUDA support

2018-04-19 Thread GitBox
spidyDev commented on issue #10553: Build error for OSX with CUDA support URL: https://github.com/apache/incubator-mxnet/issues/10553#issuecomment-382848717 @KellenSunderland Please label this issue : Build This is an automa

[GitHub] anirudhacharya commented on a change in pull request #10605: [MXNET-310] [ONNX-MXNet] API to import ONNX models into Gluon.

2018-04-19 Thread GitBox
anirudhacharya commented on a change in pull request #10605: [MXNET-310] [ONNX-MXNet] API to import ONNX models into Gluon. URL: https://github.com/apache/incubator-mxnet/pull/10605#discussion_r182850613 ## File path: tests/python-pytest/onnx/gluon_backend.py ## @@ -0,0 +1

[GitHub] spidyDev commented on issue #10567: [Feature Request] Cache the CUDNN convolution optimization result

2018-04-19 Thread GitBox
spidyDev commented on issue #10567: [Feature Request] Cache the CUDNN convolution optimization result URL: https://github.com/apache/incubator-mxnet/issues/10567#issuecomment-382844275 @eric-haibin-lin : Please label : CUDA, Feature

[GitHub] ashokei opened a new pull request #10616: ensure same mkldnn engine is used for consistency

2018-04-19 Thread GitBox
ashokei opened a new pull request #10616: ensure same mkldnn engine is used for consistency URL: https://github.com/apache/incubator-mxnet/pull/10616 ## Description ## Gluon data iterators may trigger different thread for execution context, this causes mkl-dnn engine to be inconsistent.

[GitHub] spidyDev commented on issue #10571: Gluon not mentioned in Symbol API tutorial

2018-04-19 Thread GitBox
spidyDev commented on issue #10571: Gluon not mentioned in Symbol API tutorial URL: https://github.com/apache/incubator-mxnet/issues/10571#issuecomment-382843853 @eric-haibin-lin Please label : Doc This is an automated messa

[GitHub] spidyDev commented on issue #10574: Could't work well sbt compile using jar file

2018-04-19 Thread GitBox
spidyDev commented on issue #10574: Could't work well sbt compile using jar file URL: https://github.com/apache/incubator-mxnet/issues/10574#issuecomment-382843156 @reminisce : Please label this as : Scala, Question This is a

[GitHub] ZiyueHuang opened a new pull request #10615: [MXNET-334] fix dtype of Embedding and add dtype argument to Dense

2018-04-19 Thread GitBox
ZiyueHuang opened a new pull request #10615: [MXNET-334] fix dtype of Embedding and add dtype argument to Dense URL: https://github.com/apache/incubator-mxnet/pull/10615 ## Description ## Output dtype of Embedding layer in gluon is not the same with dtype argument in the constructor. Al

[GitHub] spidyDev commented on issue #10579: NNPACK can't be compiled at 1.1.0 1.2.0 and master branch

2018-04-19 Thread GitBox
spidyDev commented on issue #10579: NNPACK can't be compiled at 1.1.0 1.2.0 and master branch URL: https://github.com/apache/incubator-mxnet/issues/10579#issuecomment-382832779 @hongtao12310 : Do you have answers for your problem. IF yes, please go ahead and close this issue. Thanks

[GitHub] anirudhacharya commented on a change in pull request #10605: [MXNET-310] [ONNX-MXNet] API to import ONNX models into Gluon.

2018-04-19 Thread GitBox
anirudhacharya commented on a change in pull request #10605: [MXNET-310] [ONNX-MXNet] API to import ONNX models into Gluon. URL: https://github.com/apache/incubator-mxnet/pull/10605#discussion_r182839939 ## File path: python/mxnet/contrib/onnx/_import/import_onnx.py ## @@

[GitHub] haojin2 commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors

2018-04-19 Thread GitBox
haojin2 commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors URL: https://github.com/apache/incubator-mxnet/pull/10550#discussion_r182837308 ## File path: src/operator/elemwise_op_common.h ## @@ -102,6 +102,48 @@

[GitHub] haojin2 commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors

2018-04-19 Thread GitBox
haojin2 commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors URL: https://github.com/apache/incubator-mxnet/pull/10550#discussion_r182837408 ## File path: src/operator/elemwise_op_common.h ## @@ -102,6 +102,48 @@

[GitHub] haojin2 commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors

2018-04-19 Thread GitBox
haojin2 commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors URL: https://github.com/apache/incubator-mxnet/pull/10550#discussion_r182837353 ## File path: src/operator/elemwise_op_common.h ## @@ -102,6 +102,48 @@

[GitHub] haojin2 commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors

2018-04-19 Thread GitBox
haojin2 commented on a change in pull request #10550: [MXNET-320] Support elemwise_add/sub between dense and csr tensors URL: https://github.com/apache/incubator-mxnet/pull/10550#discussion_r182837286 ## File path: src/operator/elemwise_op_common.h ## @@ -102,6 +102,48 @@

  1   2   3   >