[GitHub] pengzhao-intel commented on issue #13000: fix quantize_graph pass error when there're multiple outputs from a single node

2018-11-22 Thread GitBox
pengzhao-intel commented on issue #13000: fix quantize_graph pass error when there're multiple outputs from a single node URL: https://github.com/apache/incubator-mxnet/pull/13000#issuecomment-441167340 @anirudh2290 could you help merge this PR?

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2018-11-22 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng 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 5290e50 Bump the publish

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) , I think the first "log_sum_exp" in the red pane may be replaced with "sum" and the second one should be

[GitHub] janelu9 commented on issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 commented on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-441164957 I have resolved this theoritically bug. the new result is numerical stability This

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) , I think the first "log_sum_exp" in the red pane may be replaced with "sum" and the second one should be

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) , I think the first "log_sum_exp" in the red pane may be replaced with "sum" and the second one should be

[GitHub] perdasilva commented on issue #12485: test_ImageRecordIter_seed_augmentation flaky test fix

2018-11-22 Thread GitBox
perdasilva commented on issue #12485: test_ImageRecordIter_seed_augmentation flaky test fix URL: https://github.com/apache/incubator-mxnet/pull/12485#issuecomment-441163348 @stu1130 ready for review This is an automated

[GitHub] coconutyao commented on issue #12923: Deadlock happend while calling MXNDArraySyncCopyToCPU() ?

2018-11-22 Thread GitBox
coconutyao commented on issue #12923: Deadlock happend while calling MXNDArraySyncCopyToCPU() ? URL: https://github.com/apache/incubator-mxnet/issues/12923#issuecomment-441163034 > That sounds good to me. Could you maybe show some minimal example that allows to reproduce the problem?

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) , I think the first "log_sum_exp" in the red pane may be replaced with "sum" and the second one should be

[GitHub] janelu9 opened a new issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 opened a new issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329 ![image](https://user-images.githubusercontent.com/24219258/48750595-d1a31c00-ecba-11e8-8e01-4282994a8daa.png) does this step have any problem? it may not match

[GitHub] anirudhacharya commented on a change in pull request #13357: Add imresize and copyMakeBorder to mx.image

2018-11-22 Thread GitBox
anirudhacharya commented on a change in pull request #13357: Add imresize and copyMakeBorder to mx.image URL: https://github.com/apache/incubator-mxnet/pull/13357#discussion_r235848374 ## File path: tests/python/unittest/test_image.py ## @@ -139,6 +140,26 @@ def

[GitHub] anirudhacharya commented on a change in pull request #13357: Add imresize and copyMakeBorder to mx.image

2018-11-22 Thread GitBox
anirudhacharya commented on a change in pull request #13357: Add imresize and copyMakeBorder to mx.image URL: https://github.com/apache/incubator-mxnet/pull/13357#discussion_r235847823 ## File path: tests/python/unittest/test_image.py ## @@ -139,6 +140,26 @@ def

[GitHub] marcoabreu commented on issue #10951: [MXNET-545] Fix broken cython build

2018-11-22 Thread GitBox
marcoabreu commented on issue #10951: [MXNET-545] Fix broken cython build URL: https://github.com/apache/incubator-mxnet/pull/10951#issuecomment-441152581 I'm very sorry that this PR takes so long to get merged. I will ask some colleagues if they are able to help you reviewing this.

[GitHub] hanzhaogang edited a comment on issue #13220: Many operators don't work for integer type

2018-11-22 Thread GitBox
hanzhaogang edited a comment on issue #13220: Many operators don't work for integer type URL: https://github.com/apache/incubator-mxnet/issues/13220#issuecomment-441151703 Hard to trace. Can't find the gen_op module in the current package while working on this line: `from .gen_op

[GitHub] hanzhaogang commented on issue #13220: Many operators don't work for integer type

2018-11-22 Thread GitBox
hanzhaogang commented on issue #13220: Many operators don't work for integer type URL: https://github.com/apache/incubator-mxnet/issues/13220#issuecomment-441151703 Hard to trace. Can't find the gen_op module in the current package while working on this line: `from .gen_op import *

[GitHub] reminisce closed pull request #13260: Support full convention in quantized pooling

2018-11-22 Thread GitBox
reminisce closed pull request #13260: Support full convention in quantized pooling URL: https://github.com/apache/incubator-mxnet/pull/13260 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

[incubator-mxnet] branch master updated: Support full convention in quantized pooling (#13260)

2018-11-22 Thread reminisce
This is an automated email from the ASF dual-hosted git repository. reminisce 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 cd0ce3b Support full convention in

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) , I think the first "log_sum_exp" in the red pane may be replaced with "sum" ,and the second one is "log_sum"

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) , I think the first "log_sum_exp" in the red pane may be replaced with "sum" in theory .but it may causs a

[GitHub] YaoC commented on issue #12363: distributed training notebook tests

2018-11-22 Thread GitBox
YaoC commented on issue #12363: distributed training notebook tests URL: https://github.com/apache/incubator-mxnet/issues/12363#issuecomment-441149369 ```Python # Create a distributed key-value store store = kv.create('dist') ``` hi, is it `dist` the proper param of

[GitHub] DuCheng2018 edited a comment on issue #13373: How to release GPU memory by CPP interface?

2018-11-22 Thread GitBox
DuCheng2018 edited a comment on issue #13373: How to release GPU memory by CPP interface? URL: https://github.com/apache/incubator-mxnet/issues/13373#issuecomment-441148707 I find two interface functions named MXNDArrayFree( ) and MXSymbolFree( ) in c_api.h. But when I use them, they do

[GitHub] DuCheng2018 commented on issue #13373: How to release GPU memory by CPP interface?

2018-11-22 Thread GitBox
DuCheng2018 commented on issue #13373: How to release GPU memory by CPP interface? URL: https://github.com/apache/incubator-mxnet/issues/13373#issuecomment-441148707 I find a interface named MXNDArrayFree( ) and MXSymbolFree( ) in c_api.h of mxnet. But when I use them, they do not work

[GitHub] janelu9 closed issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 closed issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[GitHub] janelu9 edited a comment on issue #13329: problem about lstm_crf

2018-11-22 Thread GitBox
janelu9 edited a comment on issue #13329: problem about lstm_crf URL: https://github.com/apache/incubator-mxnet/issues/13329#issuecomment-440304581 it's difficult to compute Z(t) , I think the first "log_sum_exp" in the red pane may be replaced with "sum_exp" ,and the second one is

[GitHub] marcoabreu opened a new pull request #13379: DO NOT MERGE: Test performance of MKLDNN without Debug

2018-11-22 Thread GitBox
marcoabreu opened a new pull request #13379: DO NOT MERGE: Test performance of MKLDNN without Debug URL: https://github.com/apache/incubator-mxnet/pull/13379 Testing performance This is an automated message from the Apache

[GitHub] hanzhaogang commented on issue #13327: Cannot find custom operator proposal Error whiling trying to export a Deformable-ConvNets which is based onmxnet into onnx

2018-11-22 Thread GitBox
hanzhaogang commented on issue #13327: Cannot find custom operator proposal Error whiling trying to export a Deformable-ConvNets which is based onmxnet into onnx URL: https://github.com/apache/incubator-mxnet/issues/13327#issuecomment-441144486 @vandanavk I am really sorry that due to

[GitHub] XiaotaoChen commented on a change in pull request #12926: parallelize NDArray::Copy when data size is large

2018-11-22 Thread GitBox
XiaotaoChen commented on a change in pull request #12926: parallelize NDArray::Copy when data size is large URL: https://github.com/apache/incubator-mxnet/pull/12926#discussion_r235836681 ## File path: src/ndarray/ndarray_function.cc ## @@ -38,13 +38,14 @@ void Copy(const

[GitHub] stu1130 commented on a change in pull request #13357: Add imresize and copyMakeBorder to mx.image

2018-11-22 Thread GitBox
stu1130 commented on a change in pull request #13357: Add imresize and copyMakeBorder to mx.image URL: https://github.com/apache/incubator-mxnet/pull/13357#discussion_r235835214 ## File path: tests/python/unittest/test_image.py ## @@ -139,6 +140,26 @@ def

[GitHub] stu1130 commented on a change in pull request #13357: Add imresize and copyMakeBorder to mx.image

2018-11-22 Thread GitBox
stu1130 commented on a change in pull request #13357: Add imresize and copyMakeBorder to mx.image URL: https://github.com/apache/incubator-mxnet/pull/13357#discussion_r235835226 ## File path: tests/python/unittest/test_image.py ## @@ -139,6 +140,26 @@ def

[GitHub] marcoabreu commented on issue #13372: Support sparse as return in topK?

2018-11-22 Thread GitBox
marcoabreu commented on issue #13372: Support sparse as return in topK? URL: https://github.com/apache/incubator-mxnet/issues/13372#issuecomment-441141273 @eric-haibin-lin This is an automated message from the Apache Git

[GitHub] marcoabreu closed pull request #13377: Add timing output to python tests

2018-11-22 Thread GitBox
marcoabreu closed pull request #13377: Add timing output to python tests URL: https://github.com/apache/incubator-mxnet/pull/13377 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

[GitHub] marcoabreu commented on a change in pull request #13378: [DO NOT MERGE] Testing subrepo by branch

2018-11-22 Thread GitBox
marcoabreu commented on a change in pull request #13378: [DO NOT MERGE] Testing subrepo by branch URL: https://github.com/apache/incubator-mxnet/pull/13378#discussion_r235834853 ## File path: .gitmodules ## @@ -20,12 +20,13 @@ path = 3rdparty/mkldnn url =

[incubator-mxnet] branch master updated: Add timing output to python tests (#13377)

2018-11-22 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu 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 be26104 Add timing output to

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2018-11-22 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng 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 f76a104 Bump the publish

[GitHub] stu1130 commented on issue #13376: Fix incorrect delete in MXExecutorReshape exception handling

2018-11-22 Thread GitBox
stu1130 commented on issue #13376: Fix incorrect delete in MXExecutorReshape exception handling URL: https://github.com/apache/incubator-mxnet/pull/13376#issuecomment-441138241 @mxnet-label-bot add [pr-awaiting-review] Thanks for your contribution @taliesinb

[GitHub] stu1130 commented on issue #13378: [DO NOT MERGE] Testing subrepo by branch

2018-11-22 Thread GitBox
stu1130 commented on issue #13378: [DO NOT MERGE] Testing subrepo by branch URL: https://github.com/apache/incubator-mxnet/pull/13378#issuecomment-441138105 @mxnet-label-bot add [pr-work-in-progress] Thanks for your contribution @KellenSunderland

[GitHub] stu1130 commented on issue #13371: Add dtype when copy parameters

2018-11-22 Thread GitBox
stu1130 commented on issue #13371: Add dtype when copy parameters URL: https://github.com/apache/incubator-mxnet/pull/13371#issuecomment-441137683 @mxnet-label-bot add [pr-awaiting-review] Thanks for your contribution @kice Could you please change the description to "Fixes #13335"?

[incubator-mxnet] branch master updated: adding test for softmax operator for inputs with large magnitude (#13328)

2018-11-22 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 be76f85 adding test for softmax

[GitHub] anirudh2290 closed pull request #13328: adding test for softmax operator for inputs with large magnitude

2018-11-22 Thread GitBox
anirudh2290 closed pull request #13328: adding test for softmax operator for inputs with large magnitude URL: https://github.com/apache/incubator-mxnet/pull/13328 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] anirudh2290 closed issue #13141: MKLDNN softmax outputs NaN in mkldnn 0.14

2018-11-22 Thread GitBox
anirudh2290 closed issue #13141: MKLDNN softmax outputs NaN in mkldnn 0.14 URL: https://github.com/apache/incubator-mxnet/issues/13141 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] marcoabreu commented on a change in pull request #13377: Add timing output to python tests

2018-11-22 Thread GitBox
marcoabreu commented on a change in pull request #13377: Add timing output to python tests URL: https://github.com/apache/incubator-mxnet/pull/13377#discussion_r235827979 ## File path: ci/docker/runtime_functions.sh ## @@ -23,6 +23,7 @@ set -ex

[GitHub] KellenSunderland commented on issue #13376: Fix incorrect delete in MXExecutorReshape exception handling

2018-11-22 Thread GitBox
KellenSunderland commented on issue #13376: Fix incorrect delete in MXExecutorReshape exception handling URL: https://github.com/apache/incubator-mxnet/pull/13376#issuecomment-441132134 So I see we potentially are deleting memory we shouldn't whenever an error is encountered. What I'm

[GitHub] lebeg commented on issue #13221: Flaky tests: test_gluon_model_zoo_gpu.test_training

2018-11-22 Thread GitBox
lebeg commented on issue #13221: Flaky tests: test_gluon_model_zoo_gpu.test_training URL: https://github.com/apache/incubator-mxnet/issues/13221#issuecomment-441130281 Maybe it is not an issue anymore, feel free to introduce a PR that reverts the disabling of the test.

[GitHub] KellenSunderland commented on issue #13322: Change CUB submodule to track Nvidia CUB project.

2018-11-22 Thread GitBox
KellenSunderland commented on issue #13322: Change CUB submodule to track Nvidia CUB project. URL: https://github.com/apache/incubator-mxnet/pull/13322#issuecomment-441129901 I've looked into this a little further and I suspect that tracking a branch is still not going to get us any

[GitHub] KellenSunderland opened a new pull request #13378: [DO NOT MERGE] Testing subrepo by branch

2018-11-22 Thread GitBox
KellenSunderland opened a new pull request #13378: [DO NOT MERGE] Testing subrepo by branch URL: https://github.com/apache/incubator-mxnet/pull/13378 [DO NOT MERGE] Testing subrepo by branch This is an automated message from

[GitHub] KellenSunderland commented on issue #13322: Change CUB submodule to track Nvidia CUB project.

2018-11-22 Thread GitBox
KellenSunderland commented on issue #13322: Change CUB submodule to track Nvidia CUB project. URL: https://github.com/apache/incubator-mxnet/pull/13322#issuecomment-441127263 @frankfliu Thanks for the contribution. Your description indicates why we're using the name '3rdparty/nvidia_cub'

[GitHub] ThomasDelteil commented on a change in pull request #13377: Add timing output to python tests

2018-11-22 Thread GitBox
ThomasDelteil commented on a change in pull request #13377: Add timing output to python tests URL: https://github.com/apache/incubator-mxnet/pull/13377#discussion_r235822428 ## File path: ci/docker/runtime_functions.sh ## @@ -23,6 +23,7 @@ set -ex

[GitHub] anirudh2290 commented on a change in pull request #13324: Fix/env disable mkldnn cache map

2018-11-22 Thread GitBox
anirudh2290 commented on a change in pull request #13324: Fix/env disable mkldnn cache map URL: https://github.com/apache/incubator-mxnet/pull/13324#discussion_r235822121 ## File path: src/common/utils.h ## @@ -474,6 +474,10 @@ inline void LogStorageFallback(const

[GitHub] KellenSunderland commented on issue #13376: Fix incorrect delete in MXExecutorReshape exception handling

2018-11-22 Thread GitBox
KellenSunderland commented on issue #13376: Fix incorrect delete in MXExecutorReshape exception handling URL: https://github.com/apache/incubator-mxnet/pull/13376#issuecomment-441123751 Taking a look ... This is an automated

[GitHub] marcoabreu commented on issue #13376: Fix incorrect delete in MXExecutorReshape exception handling

2018-11-22 Thread GitBox
marcoabreu commented on issue #13376: Fix incorrect delete in MXExecutorReshape exception handling URL: https://github.com/apache/incubator-mxnet/pull/13376#issuecomment-441123567 @mkolod @KellenSunderland This is an

[GitHub] reminisce commented on a change in pull request #13297: [MKLDNN]Add quantized concat

2018-11-22 Thread GitBox
reminisce commented on a change in pull request #13297: [MKLDNN]Add quantized concat URL: https://github.com/apache/incubator-mxnet/pull/13297#discussion_r235820130 ## File path: src/operator/quantization/quantized_concat.cc ## @@ -0,0 +1,148 @@ +/* + * Licensed to the

[GitHub] gaurav-gireesh commented on a change in pull request #13325: [MXNET-1210 ] Gluon Audio - Example

2018-11-22 Thread GitBox
gaurav-gireesh commented on a change in pull request #13325: [MXNET-1210 ] Gluon Audio - Example URL: https://github.com/apache/incubator-mxnet/pull/13325#discussion_r235818261 ## File path: example/gluon/urban_sounds/datasets.py ## @@ -0,0 +1,174 @@ +# Licensed to the

[GitHub] gaurav-gireesh commented on a change in pull request #13325: [MXNET-1210 ] Gluon Audio - Example

2018-11-22 Thread GitBox
gaurav-gireesh commented on a change in pull request #13325: [MXNET-1210 ] Gluon Audio - Example URL: https://github.com/apache/incubator-mxnet/pull/13325#discussion_r235818224 ## File path: example/gluon/urban_sounds/transforms.py ## @@ -0,0 +1,204 @@ +# Licensed to the

[GitHub] gaurav-gireesh commented on a change in pull request #13325: [MXNET-1210 ] Gluon Audio - Example

2018-11-22 Thread GitBox
gaurav-gireesh commented on a change in pull request #13325: [MXNET-1210 ] Gluon Audio - Example URL: https://github.com/apache/incubator-mxnet/pull/13325#discussion_r235818249 ## File path: example/gluon/urban_sounds/train.py ## @@ -0,0 +1,161 @@ +# Licensed to the

[incubator-mxnet] branch master updated: Replace fileread with reading from stdout (#13355)

2018-11-22 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu 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 cfabc6b Replace fileread with

[GitHub] marcoabreu closed pull request #13355: Replace fileread with reading from stdout

2018-11-22 Thread GitBox
marcoabreu closed pull request #13355: Replace fileread with reading from stdout URL: https://github.com/apache/incubator-mxnet/pull/13355 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] marcoabreu opened a new pull request #13377: Add timing output to python tests

2018-11-22 Thread GitBox
marcoabreu opened a new pull request #13377: Add timing output to python tests URL: https://github.com/apache/incubator-mxnet/pull/13377 This PR adds some timing output to our python tests. Recently I have noticed an increase in the duration of Python tests - especially in relation

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2018-11-22 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng 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 6fe3278 Bump the publish

[GitHub] vdantu commented on issue #13372: Support sparse as return in topK?

2018-11-22 Thread GitBox
vdantu commented on issue #13372: Support sparse as return in topK? URL: https://github.com/apache/incubator-mxnet/issues/13372#issuecomment-44566 @mxnet-label-bot add [Operator, Question] This is an automated message

[GitHub] perdasilva commented on issue #12485: [WIP] test_ImageRecordIter_seed_augmentation flaky test fix

2018-11-22 Thread GitBox
perdasilva commented on issue #12485: [WIP] test_ImageRecordIter_seed_augmentation flaky test fix URL: https://github.com/apache/incubator-mxnet/pull/12485#issuecomment-44459 @stu1130 I will polish it tomorrow and submit it for review. I'm sorry for the delay.

[GitHub] vdantu commented on issue #13370: the official image-classification predict-cpp demo crashed if...

2018-11-22 Thread GitBox
vdantu commented on issue #13370: the official image-classification predict-cpp demo crashed if... URL: https://github.com/apache/incubator-mxnet/issues/13370#issuecomment-44461 @mxnet-label-bot add [C++, Example, Question]

[GitHub] azai91 closed pull request #12746: Add env flag to disable MKLDNN cache (MXNET_MKLDNN_CACHE_ENABLED)

2018-11-22 Thread GitBox
azai91 closed pull request #12746: Add env flag to disable MKLDNN cache (MXNET_MKLDNN_CACHE_ENABLED) URL: https://github.com/apache/incubator-mxnet/pull/12746 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] azai91 commented on issue #12746: Add env flag to disable MKLDNN cache (MXNET_MKLDNN_CACHE_ENABLED)

2018-11-22 Thread GitBox
azai91 commented on issue #12746: Add env flag to disable MKLDNN cache (MXNET_MKLDNN_CACHE_ENABLED) URL: https://github.com/apache/incubator-mxnet/pull/12746#issuecomment-44074 closing this PR because it is a dup of https://github.com/apache/incubator-mxnet/pull/13324.

[GitHub] azai91 commented on a change in pull request #13324: Fix/env disable mkldnn cache map

2018-11-22 Thread GitBox
azai91 commented on a change in pull request #13324: Fix/env disable mkldnn cache map URL: https://github.com/apache/incubator-mxnet/pull/13324#discussion_r235809955 ## File path: docs/faq/env_var.md ## @@ -195,6 +195,10 @@ When USE_PROFILER is enabled in Makefile or

[GitHub] vdantu commented on issue #13369: MKLDNN LSTM Regression in MKLDNN 0.17

2018-11-22 Thread GitBox
vdantu commented on issue #13369: MKLDNN LSTM Regression in MKLDNN 0.17 URL: https://github.com/apache/incubator-mxnet/issues/13369#issuecomment-441110240 @mxnet-label-bot add [MKLDNN, Performance] This is an automated

[GitHub] vdantu commented on issue #13363: foreach operator produces corrupt gradients for nodes external to the subgraph when some grad reqs are 'null'

2018-11-22 Thread GitBox
vdantu commented on issue #13363: foreach operator produces corrupt gradients for nodes external to the subgraph when some grad reqs are 'null' URL: https://github.com/apache/incubator-mxnet/issues/13363#issuecomment-441110134 @mxnet-label-bot add [Operator, question]

[GitHub] vdantu commented on issue #13366: Missing libgomp.so.1 on fresh ubuntu 18.04

2018-11-22 Thread GitBox
vdantu commented on issue #13366: Missing libgomp.so.1 on fresh ubuntu 18.04 URL: https://github.com/apache/incubator-mxnet/issues/13366#issuecomment-441109977 @mxnet-label-bot add [Python, pip, installation, question] This

[GitHub] vdantu commented on issue #13365: Missing MKLDNN implementation for backward softmax operator

2018-11-22 Thread GitBox
vdantu commented on issue #13365: Missing MKLDNN implementation for backward softmax operator URL: https://github.com/apache/incubator-mxnet/issues/13365#issuecomment-441109824 @mxnet-label-bot add [MKLDNN, operator, question]

[GitHub] vdantu commented on issue #13374: [Bug] ValueError: There are multiple outputs with name "resnetv1b0_layers1_relu0_fwd_output".

2018-11-22 Thread GitBox
vdantu commented on issue #13374: [Bug] ValueError: There are multiple outputs with name "resnetv1b0_layers1_relu0_fwd_output". URL: https://github.com/apache/incubator-mxnet/issues/13374#issuecomment-441109367 @mxnet-label-bot add [Gluon, Question, Bug]

[GitHub] vdantu commented on issue #13373: How to release GPU memory by CPP interface?

2018-11-22 Thread GitBox
vdantu commented on issue #13373: How to release GPU memory by CPP interface? URL: https://github.com/apache/incubator-mxnet/issues/13373#issuecomment-441109414 @mxnet-label-bot add [C++, Question] This is an automated

[GitHub] vdantu commented on issue #13375: ..

2018-11-22 Thread GitBox
vdantu commented on issue #13375: .. URL: https://github.com/apache/incubator-mxnet/issues/13375#issuecomment-441109161 @qilei123 : What is this created for? If you have suggestions on how to log an issue, could you create a PR for this template? This should otherwise be closed.

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2018-11-22 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng 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 8c279b1 Bump the publish

[GitHub] anirudh2290 commented on a change in pull request #13324: Fix/env disable mkldnn cache map

2018-11-22 Thread GitBox
anirudh2290 commented on a change in pull request #13324: Fix/env disable mkldnn cache map URL: https://github.com/apache/incubator-mxnet/pull/13324#discussion_r235795525 ## File path: docs/faq/env_var.md ## @@ -195,6 +195,10 @@ When USE_PROFILER is enabled in Makefile or

[GitHub] anirudh2290 commented on a change in pull request #12749: [MXNET-1029] Feature request: randint operator

2018-11-22 Thread GitBox
anirudh2290 commented on a change in pull request #12749: [MXNET-1029] Feature request: randint operator URL: https://github.com/apache/incubator-mxnet/pull/12749#discussion_r235789783 ## File path: tests/python/unittest/test_random.py ## @@ -844,6 +836,48 @@ def

[GitHub] anirudh2290 commented on a change in pull request #12749: [MXNET-1029] Feature request: randint operator

2018-11-22 Thread GitBox
anirudh2290 commented on a change in pull request #12749: [MXNET-1029] Feature request: randint operator URL: https://github.com/apache/incubator-mxnet/pull/12749#discussion_r235789708 ## File path: src/operator/random/sample_op.h ## @@ -249,6 +254,30 @@ struct

[GitHub] taliesinb opened a new pull request #13376: Fix incorrect delete in MXExecutorReshape exception handling

2018-11-22 Thread GitBox
taliesinb opened a new pull request #13376: Fix incorrect delete in MXExecutorReshape exception handling URL: https://github.com/apache/incubator-mxnet/pull/13376 MXExecutorReshape makes a very obvious and simple error in its cleanup code, which is that it deletes the location of the out

[GitHub] qilei123 opened a new issue #13375: ..

2018-11-22 Thread GitBox
qilei123 opened a new issue #13375: .. URL: https://github.com/apache/incubator-mxnet/issues/13375 Note: Providing complete information in the most concise form is the best way to get help. This issue template serves as the checklist for essential information to most of the technical

[GitHub] kohillyang opened a new issue #13374: [Bug] ValueError: There are multiple outputs with name "resnetv1b0_layers1_relu0_fwd_output".

2018-11-22 Thread GitBox
kohillyang opened a new issue #13374: [Bug] ValueError: There are multiple outputs with name "resnetv1b0_layers1_relu0_fwd_output". URL: https://github.com/apache/incubator-mxnet/issues/13374 I think here name should not be given.

[GitHub] TaoLv commented on a change in pull request #12926: parallelize NDArray::Copy when data size is large

2018-11-22 Thread GitBox
TaoLv commented on a change in pull request #12926: parallelize NDArray::Copy when data size is large URL: https://github.com/apache/incubator-mxnet/pull/12926#discussion_r235732685 ## File path: src/ndarray/ndarray_function.cc ## @@ -38,13 +38,14 @@ void Copy(const TBlob

[GitHub] adrianloy commented on issue #11005: Handle MKLDNN fallback correct in custom operator

2018-11-22 Thread GitBox
adrianloy commented on issue #11005: Handle MKLDNN fallback correct in custom operator URL: https://github.com/apache/incubator-mxnet/pull/11005#issuecomment-441037386 Was this ever merged? If not, why not? I am having the same issue as described here:

[GitHub] TaoLv commented on issue #13324: Fix/env disable mkldnn cache map

2018-11-22 Thread GitBox
TaoLv commented on issue #13324: Fix/env disable mkldnn cache map URL: https://github.com/apache/incubator-mxnet/pull/13324#issuecomment-441035085 LGTM. Let's wait to see the CI results. This is an automated message from the

[GitHub] TaoLv commented on a change in pull request #13197: build mkldnn as static lib

2018-11-22 Thread GitBox
TaoLv commented on a change in pull request #13197: build mkldnn as static lib URL: https://github.com/apache/incubator-mxnet/pull/13197#discussion_r235719737 ## File path: CMakeLists.txt ## @@ -225,6 +225,7 @@ endif() if(USE_MKLDNN)

[GitHub] pengzhao-intel commented on issue #12160: Remove conflicting llvm OpenMP from cmake builds

2018-11-22 Thread GitBox
pengzhao-intel commented on issue #12160: Remove conflicting llvm OpenMP from cmake builds URL: https://github.com/apache/incubator-mxnet/pull/12160#issuecomment-441023266 Also @xinyu-intel @ZhennanQin they have the experience of different platform, MacOS, Windows and compiler-level

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2018-11-22 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng 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 17c4117 Bump the publish

[GitHub] pengzhao-intel commented on issue #12160: Remove conflicting llvm OpenMP from cmake builds

2018-11-22 Thread GitBox
pengzhao-intel commented on issue #12160: Remove conflicting llvm OpenMP from cmake builds URL: https://github.com/apache/incubator-mxnet/pull/12160#issuecomment-441021610 Thanks @lebeg it's a long thread and I need to go through the change and issue first. Will back to you soon.

[GitHub] lebeg commented on issue #12160: Remove conflicting llvm OpenMP from cmake builds

2018-11-22 Thread GitBox
lebeg commented on issue #12160: Remove conflicting llvm OpenMP from cmake builds URL: https://github.com/apache/incubator-mxnet/pull/12160#issuecomment-441019573 I cross post the things I already sent to the @dev list: The issue was closed, but I am strong in my opinion that it's

[GitHub] lebeg commented on issue #12160: Remove conflicting llvm OpenMP from cmake builds

2018-11-22 Thread GitBox
lebeg commented on issue #12160: Remove conflicting llvm OpenMP from cmake builds URL: https://github.com/apache/incubator-mxnet/pull/12160#issuecomment-441018242 @pengzhao-intel can we have your input on this? This is an

[GitHub] lebeg commented on issue #8532: mxnet-mkl (v0.12.0) crash when using (conda-installed) numpy with MKL

2018-11-22 Thread GitBox
lebeg commented on issue #8532: mxnet-mkl (v0.12.0) crash when using (conda-installed) numpy with MKL URL: https://github.com/apache/incubator-mxnet/issues/8532#issuecomment-441018112 Possibly related https://github.com/apache/incubator-mxnet/pull/12160

[GitHub] lebeg commented on issue #9744: Performance regression when OMP_NUM_THREADS environment variable is not set

2018-11-22 Thread GitBox
lebeg commented on issue #9744: Performance regression when OMP_NUM_THREADS environment variable is not set URL: https://github.com/apache/incubator-mxnet/issues/9744#issuecomment-441018019 Possibly related https://github.com/apache/incubator-mxnet/pull/12160

[GitHub] lebeg commented on issue #11905: Poor concat CPU performance on CUDA builds

2018-11-22 Thread GitBox
lebeg commented on issue #11905: Poor concat CPU performance on CUDA builds URL: https://github.com/apache/incubator-mxnet/issues/11905#issuecomment-441017974 Possibly related https://github.com/apache/incubator-mxnet/pull/12160

[GitHub] DuCheng2018 opened a new issue #13373: How to release GPU memory by CPP interface?

2018-11-22 Thread GitBox
DuCheng2018 opened a new issue #13373: How to release GPU memory by CPP interface? URL: https://github.com/apache/incubator-mxnet/issues/13373 I need to do a forward propagation at first, and then do another one in one process. If gpu memory malloced by the first propagation can not

[GitHub] marcoabreu commented on issue #13367: [Example] add cpp inception in Jenkins and nightly test

2018-11-22 Thread GitBox
marcoabreu commented on issue #13367: [Example] add cpp inception in Jenkins and nightly test URL: https://github.com/apache/incubator-mxnet/pull/13367#issuecomment-440994128 please wait with changes to the Jenkinsfile until further notice

[GitHub] lebeg closed pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

2018-11-22 Thread GitBox
lebeg closed pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148 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:

[GitHub] lebeg commented on issue #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

2018-11-22 Thread GitBox
lebeg commented on issue #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148#issuecomment-440990159 This PR needs to be separated and, as I have stated in on the @dev list, refactored to used as much built-in cmake

[GitHub] ChaiBapchya commented on issue #12453: GPU tests are unstable

2018-11-22 Thread GitBox
ChaiBapchya commented on issue #12453: GPU tests are unstable URL: https://github.com/apache/incubator-mxnet/issues/12453#issuecomment-440989859 CI failed with similar error -

[GitHub] zerolxf opened a new issue #13372: Support sparse as return in topK?

2018-11-22 Thread GitBox
zerolxf opened a new issue #13372: Support sparse as return in topK? URL: https://github.com/apache/incubator-mxnet/issues/13372 ## Description why not support sparse as return in [topK](https://mxnet.incubator.apache.org/api/python/symbol/symbol.html#mxnet.symbol.topk)?

[GitHub] ZhennanQin commented on a change in pull request #13297: [MKLDNN]Add quantized concat

2018-11-22 Thread GitBox
ZhennanQin commented on a change in pull request #13297: [MKLDNN]Add quantized concat URL: https://github.com/apache/incubator-mxnet/pull/13297#discussion_r235635363 ## File path: src/operator/quantization/quantized_concat.cc ## @@ -0,0 +1,148 @@ +/* + * Licensed to the

[GitHub] ZhennanQin commented on a change in pull request #13297: [MKLDNN]Add quantized concat

2018-11-22 Thread GitBox
ZhennanQin commented on a change in pull request #13297: [MKLDNN]Add quantized concat URL: https://github.com/apache/incubator-mxnet/pull/13297#discussion_r235640008 ## File path: src/operator/nn/mkldnn/mkldnn_concat-inl.h ## @@ -0,0 +1,108 @@ +/* + * Licensed to the

[GitHub] ZhennanQin commented on a change in pull request #13297: [MKLDNN]Add quantized concat

2018-11-22 Thread GitBox
ZhennanQin commented on a change in pull request #13297: [MKLDNN]Add quantized concat URL: https://github.com/apache/incubator-mxnet/pull/13297#discussion_r235636478 ## File path: src/operator/quantization/quantized_concat.cc ## @@ -0,0 +1,148 @@ +/* + * Licensed to the

[GitHub] pengzhao-intel commented on a change in pull request #13324: Fix/env disable mkldnn cache map

2018-11-22 Thread GitBox
pengzhao-intel commented on a change in pull request #13324: Fix/env disable mkldnn cache map URL: https://github.com/apache/incubator-mxnet/pull/13324#discussion_r235635700 ## File path: docs/faq/env_var.md ## @@ -195,6 +195,10 @@ When USE_PROFILER is enabled in Makefile

  1   2   >