[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 ti

[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 "lo

[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 "lo

[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 "lo

[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 messa

[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 "lo

[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 th

[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 test_resiz

[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 test_resiz

[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 imp

[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 this

[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] 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 `kv.create`?

[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 ari

[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 w

[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 us

[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 "log_s

[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 Gi

[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 the

[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 test_resize_short

[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 test_resize_short

[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 Se

[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 forei

[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 = h

[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 pytho

[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 ti

[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 o

[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 o

[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, pleas

[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 NOSE_COVERAGE_ARGUMENTS=

[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 won

[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 protec

[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 NOSE_COVERAGE_ARGUMEN

[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 nnvm::No

[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 automat

[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 Apa

[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 A

[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 Apa

[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 Apache

[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 read

[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 is

[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 t

[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 ti

[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 fr

[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 pr

[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 CMake

[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 messag

[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 i

[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 messa

[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 ti

[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 testLarge

[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 SampleGenNe

[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 p

[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 issu

[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. https://github.com/apache/incubator-mxnet/blob/564e01acd

[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: https://discuss.mxnet.io/t/mxnet-1-2-

[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 A

[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) include(cmake/DownloadMKLML.cmake)

[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 knowl

[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 ti

[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 th

[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 aut

[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 be

[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 funct

[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 - http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-12749/67/pipeline/1

[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 Ap

[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 Apach

[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 Ap

[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   >