[GitHub] ankkhedia closed pull request #13271: [Test] [Example] adding tests for examples

2018-12-19 Thread GitBox
ankkhedia closed pull request #13271: [Test] [Example] adding tests for examples URL: https://github.com/apache/incubator-mxnet/pull/13271 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] ankkhedia commented on issue #13271: [Test] [Example] adding tests for examples

2018-12-19 Thread GitBox
ankkhedia commented on issue #13271: [Test] [Example] adding tests for examples URL: https://github.com/apache/incubator-mxnet/pull/13271#issuecomment-448686180 @kalyc Thanks for taking it forward. This is an automated

[GitHub] KellenSunderland closed pull request #13659: [MXNET-1252][1 of 2] Decouple NNVM to ONNX from NNVM to TenosrRT conv…

2018-12-19 Thread GitBox
KellenSunderland closed pull request #13659: [MXNET-1252][1 of 2] Decouple NNVM to ONNX from NNVM to TenosrRT conv… URL: https://github.com/apache/incubator-mxnet/pull/13659 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below

[GitHub] KellenSunderland commented on issue #13689: (WIP): Jetson build with CMake and TensorRT

2018-12-19 Thread GitBox
KellenSunderland commented on issue #13689: (WIP): Jetson build with CMake and TensorRT URL: https://github.com/apache/incubator-mxnet/pull/13689#issuecomment-448700364 This would be awesome if possible. This is an automated

[GitHub] srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r243042709 ## File path: NEWS.md ## @@ -1,6 +1,595 @@ -MXNet Change Log +Apache MXNet (incubating) Change

[GitHub] Roshrini commented on issue #13413: fix for params with no dims in onnx

2018-12-19 Thread GitBox
Roshrini commented on issue #13413: fix for params with no dims in onnx URL: https://github.com/apache/incubator-mxnet/pull/13413#issuecomment-448717083 @mxnet-label-bot [pr-work-in-progress] This is an automated message from

[GitHub] Roshrini commented on issue #13460: onnx export slope for prelu operator corrected

2018-12-19 Thread GitBox
Roshrini commented on issue #13460: onnx export slope for prelu operator corrected URL: https://github.com/apache/incubator-mxnet/pull/13460#issuecomment-448717379 @mxnet-label-bot Update [pr-work-in-progress] This is an

[GitHub] aaronmarkham commented on issue #12977: [MXNET-1164] Generate the document for cpp-package using Doxygen

2018-12-19 Thread GitBox
aaronmarkham commented on issue #12977: [MXNET-1164] Generate the document for cpp-package using Doxygen URL: https://github.com/apache/incubator-mxnet/pull/12977#issuecomment-448719848 @leleamol Please add `3rdparty` to the `EXCLUDE` directive in `Doxyfile`. That should help retrigger CI

[GitHub] vandanavk commented on a change in pull request #13500: [MXNET-898] ONNX import/export: Sample_multinomial, ONNX export: GlobalLpPool, LpPool

2018-12-19 Thread GitBox
vandanavk commented on a change in pull request #13500: [MXNET-898] ONNX import/export: Sample_multinomial, ONNX export: GlobalLpPool, LpPool URL: https://github.com/apache/incubator-mxnet/pull/13500#discussion_r243023655 ## File path:

[GitHub] zhreshold commented on issue #13593: Low CPU usage of MXNet in subprocesses

2018-12-19 Thread GitBox
zhreshold commented on issue #13593: Low CPU usage of MXNet in subprocesses URL: https://github.com/apache/incubator-mxnet/issues/13593#issuecomment-448702230 @YutingZhang MXNET_MP_WORKER_NTHREADS can only control how many mxnet operators run in parallel, in the case of some

[incubator-mxnet] branch master updated: fix unpicklable transform_first on windows (#13686)

2018-12-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 61744b5 fix unpicklable

[GitHub] eric-haibin-lin closed pull request #13686: [gluon][transform]fix unpicklable transform_first on windows

2018-12-19 Thread GitBox
eric-haibin-lin closed pull request #13686: [gluon][transform]fix unpicklable transform_first on windows URL: https://github.com/apache/incubator-mxnet/pull/13686 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] anirudhacharya commented on a change in pull request #13683: fix bug in nag optimizer

2018-12-19 Thread GitBox
anirudhacharya commented on a change in pull request #13683: fix bug in nag optimizer URL: https://github.com/apache/incubator-mxnet/pull/13683#discussion_r243040908 ## File path: python/mxnet/optimizer/optimizer.py ## @@ -974,8 +974,7 @@ def update(self, index, weight,

[GitHub] srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r243042963 ## File path: NEWS.md ## @@ -1,6 +1,595 @@ -MXNet Change Log +Apache MXNet (incubating) Change

[GitHub] srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r243042812 ## File path: NEWS.md ## @@ -1,6 +1,595 @@ -MXNet Change Log +Apache MXNet (incubating) Change

[GitHub] anirudhacharya commented on a change in pull request #13683: fix bug in nag optimizer

2018-12-19 Thread GitBox
anirudhacharya commented on a change in pull request #13683: fix bug in nag optimizer URL: https://github.com/apache/incubator-mxnet/pull/13683#discussion_r243035067 ## File path: python/mxnet/optimizer/optimizer.py ## @@ -974,8 +974,7 @@ def update(self, index, weight,

[GitHub] srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r243042891 ## File path: NEWS.md ## @@ -1,6 +1,595 @@ -MXNet Change Log +Apache MXNet (incubating) Change

[GitHub] Roshrini commented on issue #13682: Add a docs build (with error checking) option to the dev_menu.py script

2018-12-19 Thread GitBox
Roshrini commented on issue #13682: Add a docs build (with error checking) option to the dev_menu.py script URL: https://github.com/apache/incubator-mxnet/pull/13682#issuecomment-448715312 @mxnet-label-bot Add [Doc, pr-awaiting-review]

[GitHub] Roshrini commented on issue #13683: fix bug in nag optimizer

2018-12-19 Thread GitBox
Roshrini commented on issue #13683: fix bug in nag optimizer URL: https://github.com/apache/incubator-mxnet/pull/13683#issuecomment-448715473 @mxnet-label-bot Add [pr-awaiting-response] This is an automated message from the

[GitHub] aaronmarkham commented on issue #13682: Add a docs build (with error checking) option to the dev_menu.py script

2018-12-19 Thread GitBox
aaronmarkham commented on issue #13682: Add a docs build (with error checking) option to the dev_menu.py script URL: https://github.com/apache/incubator-mxnet/pull/13682#issuecomment-448716340 @larroy - added this feature per your suggestion. Also, I have some comments about the script

[GitHub] Roshrini commented on issue #13083: [MXNET-345] Hardmax operator

2018-12-19 Thread GitBox
Roshrini commented on issue #13083: [MXNET-345] Hardmax operator URL: https://github.com/apache/incubator-mxnet/pull/13083#issuecomment-448716362 @eric-haibin-lin Can you check take review/merge this PR? @mxnet-label-bot Update [pr-awaiting-response]

[GitHub] Roshrini commented on issue #12977: [MXNET-1164] Generate the document for cpp-package using Doxygen

2018-12-19 Thread GitBox
Roshrini commented on issue #12977: [MXNET-1164] Generate the document for cpp-package using Doxygen URL: https://github.com/apache/incubator-mxnet/pull/12977#issuecomment-448716926 @leleamol Can you retrigger the CI for this PR? @mxnet-label-bot Update [pr-awaiting-testing]

[GitHub] Roshrini commented on issue #13310: [MXNET-703] Update to TensorRT 5, ONNX IR 3. Fix inference bugs.

2018-12-19 Thread GitBox
Roshrini commented on issue #13310: [MXNET-703] Update to TensorRT 5, ONNX IR 3. Fix inference bugs. URL: https://github.com/apache/incubator-mxnet/pull/13310#issuecomment-448716703 @marcoabreu Is jenking refactor finalized now? @KellenSunderland Can you rebase this PR?

[GitHub] ifeherva commented on issue #13632: Gradient reversal (contrib) operator

2018-12-19 Thread GitBox
ifeherva commented on issue #13632: Gradient reversal (contrib) operator URL: https://github.com/apache/incubator-mxnet/pull/13632#issuecomment-448726478 That is certainly possible, shall I rewrite it? This is an automated

[GitHub] apeforest commented on a change in pull request #12440: Add stable nrm2 for L2 normalization

2018-12-19 Thread GitBox
apeforest commented on a change in pull request #12440: Add stable nrm2 for L2 normalization URL: https://github.com/apache/incubator-mxnet/pull/12440#discussion_r243026743 ## File path: src/operator/l2_normalization-inl.h ## @@ -314,6 +309,11 @@ class L2NormalizationProp

[GitHub] apeforest commented on a change in pull request #12440: Add stable nrm2 for L2 normalization

2018-12-19 Thread GitBox
apeforest commented on a change in pull request #12440: Add stable nrm2 for L2 normalization URL: https://github.com/apache/incubator-mxnet/pull/12440#discussion_r243026514 ## File path: src/operator/l2_normalization-inl.h ## @@ -87,22 +88,24 @@ class L2NormalizationOp :

[incubator-mxnet] branch master updated: [MXNET-1252][1 of 2] Decouple NNVM to ONNX from NNVM to TenosrRT conversion (#13659)

2018-12-19 Thread kellen
This is an automated email from the ASF dual-hosted git repository. kellen 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 f85b17b [MXNET-1252][1 of 2] Decouple

[GitHub] zhreshold commented on issue #13686: [gluon][transform]fix unpicklable transform_first on windows

2018-12-19 Thread GitBox
zhreshold commented on issue #13686: [gluon][transform]fix unpicklable transform_first on windows URL: https://github.com/apache/incubator-mxnet/pull/13686#issuecomment-448700806 @eric-haibin-lin Can you help review and merge? Aston is eager to include it in 1.4 so readers using windows

[GitHub] srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r243046167 ## File path: NEWS.md ## @@ -169,52 +199,56 @@ More details can be found here: [Topology-aware

[GitHub] srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r243046281 ## File path: NEWS.md ## @@ -169,52 +199,56 @@ More details can be found here: [Topology-aware

[GitHub] srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r243046341 ## File path: NEWS.md ## @@ -169,52 +199,56 @@ More details can be found here: [Topology-aware

[GitHub] srochel commented on issue #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
srochel commented on issue #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#issuecomment-448716411 addressed all feedback as of 12/19 11.30am. This is an automated

[GitHub] roywei commented on a change in pull request #13597: [MXNET-1255] update hybridize documentation

2018-12-19 Thread GitBox
roywei commented on a change in pull request #13597: [MXNET-1255] update hybridize documentation URL: https://github.com/apache/incubator-mxnet/pull/13597#discussion_r243058785 ## File path: docs/tutorials/gluon/hybrid.md ## @@ -137,4 +137,94 @@ to gluon with

[GitHub] gigasquid commented on a change in pull request #13626: Re-organize Scala maven build

2018-12-19 Thread GitBox
gigasquid commented on a change in pull request #13626: Re-organize Scala maven build URL: https://github.com/apache/incubator-mxnet/pull/13626#discussion_r243092900 ## File path: scala-package/core/pom.xml ## @@ -5,55 +5,71 @@ 4.0.0 org.apache.mxnet -

[GitHub] aaronmarkham closed pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
aaronmarkham closed pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657 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 v1.4.x updated: update with release notes for 1.4.0 release (#13657)

2018-12-19 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham pushed a commit to branch v1.4.x in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/v1.4.x by this push: new e413709 update with release

[GitHub] szha commented on issue #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
szha commented on issue #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#issuecomment-448766221 @aaronmarkham there was open comment by @astonzhang which I think should be addressed. What's your plan?

[GitHub] astonzhang opened a new pull request #13691: Add bug fix #13686

2018-12-19 Thread GitBox
astonzhang opened a new pull request #13691: Add bug fix #13686 URL: https://github.com/apache/incubator-mxnet/pull/13691 ## Description ## Add bug fix #13686 ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. - [x] The PR title

[GitHub] srochel opened a new pull request #13690: added Fix unpicklable transform_first on windows

2018-12-19 Thread GitBox
srochel opened a new pull request #13690: added Fix unpicklable transform_first on windows URL: https://github.com/apache/incubator-mxnet/pull/13690 added * Fix unpicklable transform_first on windows (#13686 ) @eric-haibin-lin still needs to merge this PR and PR 13686 to 1.4.x branch

[incubator-mxnet] branch v1.4.x updated: Add bug fix #13686 to release note (#13691)

2018-12-19 Thread haibin
This is an automated email from the ASF dual-hosted git repository. haibin pushed a commit to branch v1.4.x in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/v1.4.x by this push: new c84bb78 Add bug fix #13686 to release

[GitHub] eric-haibin-lin closed pull request #13691: Add bug fix #13686

2018-12-19 Thread GitBox
eric-haibin-lin closed pull request #13691: Add bug fix #13686 URL: https://github.com/apache/incubator-mxnet/pull/13691 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] gigasquid commented on issue #13385: [Clojure] - Turn examples into integration tests

2018-12-19 Thread GitBox
gigasquid commented on issue #13385: [Clojure] - Turn examples into integration tests URL: https://github.com/apache/incubator-mxnet/issues/13385#issuecomment-448770134 This is now in place! Thanks @hellonico This is an

[GitHub] astonzhang commented on issue #13684: CUDA error when increasing number of training epochs

2018-12-19 Thread GitBox
astonzhang commented on issue #13684: CUDA error when increasing number of training epochs URL: https://github.com/apache/incubator-mxnet/issues/13684#issuecomment-448769778 @mxnet-label-bot add [Cuda, Windows] This is an

[GitHub] astonzhang commented on issue #13690: added Fix unpicklable transform_first on windows

2018-12-19 Thread GitBox
astonzhang commented on issue #13690: added Fix unpicklable transform_first on windows URL: https://github.com/apache/incubator-mxnet/pull/13690#issuecomment-448770206 Thanks! The PR with the same change has been merged: https://github.com/apache/incubator-mxnet/pull/13691

[GitHub] gigasquid closed issue #13385: [Clojure] - Turn examples into integration tests

2018-12-19 Thread GitBox
gigasquid closed issue #13385: [Clojure] - Turn examples into integration tests URL: https://github.com/apache/incubator-mxnet/issues/13385 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] lanking520 commented on a change in pull request #13626: Re-organize Scala maven build

2018-12-19 Thread GitBox
lanking520 commented on a change in pull request #13626: Re-organize Scala maven build URL: https://github.com/apache/incubator-mxnet/pull/13626#discussion_r243100435 ## File path: scala-package/deploy/pom.xml ## @@ -0,0 +1,137 @@ + +http://maven.apache.org/POM/4.0.0; +

[GitHub] ThomasDelteil commented on issue #13632: Gradient reversal (contrib) operator

2018-12-19 Thread GitBox
ThomasDelteil commented on issue #13632: Gradient reversal (contrib) operator URL: https://github.com/apache/incubator-mxnet/pull/13632#issuecomment-448906739 @szha @zheng-da what do you think? This is an automated message

[GitHub] astonzhang commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
astonzhang commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r243083834 ## File path: NEWS.md ## @@ -1,6 +1,580 @@ -MXNet Change Log +Apache MXNet (incubating)

[GitHub] apeforest removed a comment on issue #13083: [MXNET-345] Hardmax operator

2018-12-19 Thread GitBox
apeforest removed a comment on issue #13083: [MXNET-345] Hardmax operator URL: https://github.com/apache/incubator-mxnet/pull/13083#issuecomment-448753600 LGTM. I think we can merge this frontend implementation first so ONNX can leverage this. If there is a strong demand of this operator.

[GitHub] apeforest commented on issue #13083: [MXNET-345] Hardmax operator

2018-12-19 Thread GitBox
apeforest commented on issue #13083: [MXNET-345] Hardmax operator URL: https://github.com/apache/incubator-mxnet/pull/13083#issuecomment-448753600 LGTM. I think we can merge this frontend implementation first so ONNX can leverage this. If there is a strong demand of this operator. We can

[GitHub] apeforest commented on issue #13083: [MXNET-345] Hardmax operator

2018-12-19 Thread GitBox
apeforest commented on issue #13083: [MXNET-345] Hardmax operator URL: https://github.com/apache/incubator-mxnet/pull/13083#issuecomment-448753895 @mxnet-label-bot Update [pr-awaiting-merge] This is an automated message from

[GitHub] Roshrini commented on issue #13689: (WIP): Jetson build with CMake and TensorRT

2018-12-19 Thread GitBox
Roshrini commented on issue #13689: (WIP): Jetson build with CMake and TensorRT URL: https://github.com/apache/incubator-mxnet/pull/13689#issuecomment-448706484 @mxnet-label-bot Add [pr-work-in-progress] This is an automated

[GitHub] Roshrini commented on issue #13687: split_v2 operator

2018-12-19 Thread GitBox
Roshrini commented on issue #13687: split_v2 operator URL: https://github.com/apache/incubator-mxnet/pull/13687#issuecomment-448706730 @HyperZealot Thanks for working on this. @mxnet-label-bot Add [pr-awaiting-review, Operator]

[GitHub] Roshrini commented on issue #13571: Add pixelshuffle layers

2018-12-19 Thread GitBox
Roshrini commented on issue #13571: Add pixelshuffle layers URL: https://github.com/apache/incubator-mxnet/pull/13571#issuecomment-448717730 @mxnet-label-bot Update [Operator, pr-awaiting-response] This is an automated

[GitHub] Roshrini commented on issue #13450: [WIP][MXNET-862] Basic maven jenkins pipeline

2018-12-19 Thread GitBox
Roshrini commented on issue #13450: [WIP][MXNET-862] Basic maven jenkins pipeline URL: https://github.com/apache/incubator-mxnet/pull/13450#issuecomment-448717257 @zachgk Can you rebase this PR? @mxnet-label-bot Update [pr-work-in-progress]

[GitHub] frankfliu commented on a change in pull request #13626: Re-organize Scala maven build

2018-12-19 Thread GitBox
frankfliu commented on a change in pull request #13626: Re-organize Scala maven build URL: https://github.com/apache/incubator-mxnet/pull/13626#discussion_r243048879 ## File path: scala-package/core/pom.xml ## @@ -5,55 +5,71 @@ 4.0.0 org.apache.mxnet -

[incubator-mxnet] branch v1.4.x updated (e27e3e9 -> af7905f)

2018-12-19 Thread haibin
This is an automated email from the ASF dual-hosted git repository. haibin pushed a change to branch v1.4.x in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from e27e3e9 Complimentary gluon DataLoader improvements (#13606) add af7905f fix unpicklable

[GitHub] ThomasDelteil commented on issue #13632: Gradient reversal (contrib) operator

2018-12-19 Thread GitBox
ThomasDelteil commented on issue #13632: Gradient reversal (contrib) operator URL: https://github.com/apache/incubator-mxnet/pull/13632#issuecomment-448724370 Shouldn't we have a more generic gradient multiplier operator? What d you think?

[GitHub] Roshrini commented on issue #13339: CMake: Enable installation of cpp-package headers

2018-12-19 Thread GitBox
Roshrini commented on issue #13339: CMake: Enable installation of cpp-package headers URL: https://github.com/apache/incubator-mxnet/pull/13339#issuecomment-448732116 @marcoabreu @KellenSunderland to review/merge this PR? @mxnet-label-bot Update [pr-awaiting-merge]

[GitHub] szha commented on issue #13681: [WIP] Making MKL-DNN default on MXNet master

2018-12-19 Thread GitBox
szha commented on issue #13681: [WIP] Making MKL-DNN default on MXNet master URL: https://github.com/apache/incubator-mxnet/pull/13681#issuecomment-448736093 The set of changes in this PR looks very different from #13464. What's the plan for the two PRs?

[GitHub] sergeykolychev commented on issue #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
sergeykolychev commented on issue #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#issuecomment-448744023 @srochel after I get this merged into the 1.4.x branch, what's procedure to get it added to the master branch ?

[GitHub] eric-haibin-lin commented on a change in pull request #13599: fallback to dense version for grad(reshape), grad(expand_dims)

2018-12-19 Thread GitBox
eric-haibin-lin commented on a change in pull request #13599: fallback to dense version for grad(reshape), grad(expand_dims) URL: https://github.com/apache/incubator-mxnet/pull/13599#discussion_r242793381 ## File path: src/operator/tensor/elemwise_unary_op_basic.cc ## @@

[GitHub] roywei edited a comment on issue #13597: [MXNET-1255] update hybridize documentation

2018-12-19 Thread GitBox
roywei edited a comment on issue #13597: [MXNET-1255] update hybridize documentation URL: https://github.com/apache/incubator-mxnet/pull/13597#issuecomment-448731047 @aaronmarkham Thanks for the review, I have addressed your comments. 1. I have also added [dive into deeplearning book

[GitHub] roywei commented on issue #13597: [MXNET-1255] update hybridize documentation

2018-12-19 Thread GitBox
roywei commented on issue #13597: [MXNET-1255] update hybridize documentation URL: https://github.com/apache/incubator-mxnet/pull/13597#issuecomment-448731047 @aaronmarkham Thanks for the review, I have addressed your comments. 1. I have also added [dive into deeplearning book

[GitHub] mseth10 commented on a change in pull request #13681: [WIP] Making MKL-DNN default on MXNet master

2018-12-19 Thread GitBox
mseth10 commented on a change in pull request #13681: [WIP] Making MKL-DNN default on MXNet master URL: https://github.com/apache/incubator-mxnet/pull/13681#discussion_r243064386 ## File path: ci/docker/runtime_functions.sh ## @@ -287,6 +287,12 @@ build_centos7_cpu() {

[GitHub] mseth10 commented on a change in pull request #13681: [WIP] Making MKL-DNN default on MXNet master

2018-12-19 Thread GitBox
mseth10 commented on a change in pull request #13681: [WIP] Making MKL-DNN default on MXNet master URL: https://github.com/apache/incubator-mxnet/pull/13681#discussion_r243064434 ## File path: ci/docker/runtime_functions.sh ## @@ -342,6 +347,12 @@ build_centos7_gpu() {

[GitHub] mseth10 commented on a change in pull request #13681: [WIP] Making MKL-DNN default on MXNet master

2018-12-19 Thread GitBox
mseth10 commented on a change in pull request #13681: [WIP] Making MKL-DNN default on MXNet master URL: https://github.com/apache/incubator-mxnet/pull/13681#discussion_r243064943 ## File path: ci/jenkins/Jenkins_steps.groovy ## @@ -23,22 +23,22 @@ utils =

[GitHub] roywei edited a comment on issue #13597: [MXNET-1255] update hybridize documentation

2018-12-19 Thread GitBox
roywei edited a comment on issue #13597: [MXNET-1255] update hybridize documentation URL: https://github.com/apache/incubator-mxnet/pull/13597#issuecomment-448731047 @aaronmarkham @ChaiBapchya Thanks for the review, I have addressed your comments. 1. I have also added [dive into

[GitHub] szha commented on issue #11911: Illegal instruction (core dumped)

2018-12-19 Thread GitBox
szha commented on issue #11911: Illegal instruction (core dumped) URL: https://github.com/apache/incubator-mxnet/issues/11911#issuecomment-448734537 @antoniosap what are the supported instruction sets of the CPU you're using?

[GitHub] HyperZealot opened a new pull request #13687: split_v2 operator

2018-12-19 Thread GitBox
HyperZealot opened a new pull request #13687: split_v2 operator URL: https://github.com/apache/incubator-mxnet/pull/13687 ## Description ## New version of split operator to match the behavior of [numpy.split](https://docs.scipy.org/doc/numpy-1.15.1/reference/generated/numpy.split.html)

[GitHub] eric-haibin-lin closed pull request #13588: Accelerate DGL csr neighbor sampling

2018-12-19 Thread GitBox
eric-haibin-lin closed pull request #13588: Accelerate DGL csr neighbor sampling URL: https://github.com/apache/incubator-mxnet/pull/13588 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] astonzhang commented on issue #13686: [gluon][transform]fix unpicklable transform_first on windows

2018-12-19 Thread GitBox
astonzhang commented on issue #13686: [gluon][transform]fix unpicklable transform_first on windows URL: https://github.com/apache/incubator-mxnet/pull/13686#issuecomment-448516534 Thanks for the fix! This is an automated

[GitHub] aaronmarkham commented on issue #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
aaronmarkham commented on issue #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#issuecomment-448774399 Ah I didn't see Aston's comment. I can add that myself.

[GitHub] astonzhang commented on issue #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
astonzhang commented on issue #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#issuecomment-448774926 Don't worry, it has already been added: https://github.com/apache/incubator-mxnet/pull/13691

[GitHub] eric-haibin-lin closed issue #13372: Support sparse as return in topK?

2018-12-19 Thread GitBox
eric-haibin-lin closed issue #13372: Support sparse as return in topK? URL: https://github.com/apache/incubator-mxnet/issues/13372 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] srochel closed pull request #13690: added Fix unpicklable transform_first on windows

2018-12-19 Thread GitBox
srochel closed pull request #13690: added Fix unpicklable transform_first on windows URL: https://github.com/apache/incubator-mxnet/pull/13690 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] anirudh2290 commented on issue #12440: Add stable nrm2 for L2 normalization

2018-12-19 Thread GitBox
anirudh2290 commented on issue #12440: Add stable nrm2 for L2 normalization URL: https://github.com/apache/incubator-mxnet/pull/12440#issuecomment-448793556 There are pros and cons to both approaches. If @TD has to implement with Kernel::Launch approach he has to rely on mshadow

[incubator-mxnet] branch master updated: Accelerate DGL csr neighbor sampling (#13588)

2018-12-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 defa614 Accelerate DGL csr neighbor

[GitHub] SSE4 commented on issue #13400: [MXNET-1229] use OpenBLAS, lapack & OpenCV from conan

2018-12-19 Thread GitBox
SSE4 commented on issue #13400: [MXNET-1229] use OpenBLAS, lapack & OpenCV from conan URL: https://github.com/apache/incubator-mxnet/pull/13400#issuecomment-448562679 @samskalicky so build is green, feel free to review it further

[GitHub] samskalicky commented on issue #13400: [MXNET-1229] use OpenBLAS, lapack & OpenCV from conan

2018-12-19 Thread GitBox
samskalicky commented on issue #13400: [MXNET-1229] use OpenBLAS, lapack & OpenCV from conan URL: https://github.com/apache/incubator-mxnet/pull/13400#issuecomment-448599220 @SSE4 Just to confirm, this change enables conan builds using the example in the description. And can still be

[GitHub] srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r242957208 ## File path: NEWS.md ## @@ -1,6 +1,565 @@ MXNet Change Log +## 1.4.0

[GitHub] srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r242956564 ## File path: NEWS.md ## @@ -1,6 +1,565 @@ MXNet Change Log +## 1.4.0

[GitHub] SSE4 commented on issue #13400: [MXNET-1229] use OpenBLAS, lapack & OpenCV from conan

2018-12-19 Thread GitBox
SSE4 commented on issue #13400: [MXNET-1229] use OpenBLAS, lapack & OpenCV from conan URL: https://github.com/apache/incubator-mxnet/pull/13400#issuecomment-448599893 @samskalicky yes, you're absolutely right, change adds conan builds which are optional, and you still can build things

[GitHub] srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r242955630 ## File path: NEWS.md ## @@ -1,6 +1,565 @@ MXNet Change Log +## 1.4.0

[GitHub] kice opened a new issue #13688: How to feed MXNet with data already on GPU memory?

2018-12-19 Thread GitBox
kice opened a new issue #13688: How to feed MXNet with data already on GPU memory? URL: https://github.com/apache/incubator-mxnet/issues/13688 I want to feed the image data already on GPU (in fact, it is a ID3D11Texture2D object) to MXNet; and if possiable, after processing, I can

[GitHub] antoniosap commented on issue #11911: Illegal instruction (core dumped)

2018-12-19 Thread GitBox
antoniosap commented on issue #11911: Illegal instruction (core dumped) URL: https://github.com/apache/incubator-mxnet/issues/11911#issuecomment-448615683 1.3.1 core dumped ``` pip install mxnet ipython Python 3.6.5 |Anaconda, Inc.| (default, Apr 29 2018, 16:14:56)

[GitHub] srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
srochel commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r242955299 ## File path: NEWS.md ## @@ -1,6 +1,565 @@ MXNet Change Log +## 1.4.0

[GitHub] aakashpatel25 commented on issue #13674: Multibox Target function returns 0 for Box Mask and Box Target for all Anchors proposed

2018-12-19 Thread GitBox
aakashpatel25 commented on issue #13674: Multibox Target function returns 0 for Box Mask and Box Target for all Anchors proposed URL: https://github.com/apache/incubator-mxnet/issues/13674#issuecomment-448654081 Just wanted to ask people are we suppose to normalize the labels from 0.0 to

[GitHub] aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r242993299 ## File path: NEWS.md ## @@ -169,52 +199,56 @@ More details can be found here:

[GitHub] jermainewang commented on issue #13658: Converting MX array to DLPack crashes when MX array goes out-of-scope

2018-12-19 Thread GitBox
jermainewang commented on issue #13658: Converting MX array to DLPack crashes when MX array goes out-of-scope URL: https://github.com/apache/incubator-mxnet/issues/13658#issuecomment-448675854 That's strange. I thought the mutable is for the data pointer while the shape array should not

[GitHub] aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r242997199 ## File path: NEWS.md ## @@ -1,6 +1,595 @@ -MXNet Change Log +Apache MXNet (incubating)

[GitHub] aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r242992134 ## File path: NEWS.md ## @@ -1,6 +1,565 @@ MXNet Change Log +##

[GitHub] aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r242995296 ## File path: NEWS.md ## @@ -169,52 +199,56 @@ More details can be found here:

[GitHub] MoritzMaxeiner commented on issue #13339: CMake: Enable installation of cpp-package headers

2018-12-19 Thread GitBox
MoritzMaxeiner commented on issue #13339: CMake: Enable installation of cpp-package headers URL: https://github.com/apache/incubator-mxnet/pull/13339#issuecomment-448667104 @Roshrini All checks seem to have passed, now. This

[GitHub] lebeg opened a new pull request #13689: (WIP): Jetson build with CMake and TensorRT

2018-12-19 Thread GitBox
lebeg opened a new pull request #13689: (WIP): Jetson build with CMake and TensorRT URL: https://github.com/apache/incubator-mxnet/pull/13689 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove

[GitHub] aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r242993430 ## File path: NEWS.md ## @@ -169,52 +199,56 @@ More details can be found here:

[GitHub] aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r242999683 ## File path: NEWS.md ## @@ -1,6 +1,595 @@ -MXNet Change Log +Apache MXNet (incubating)

[GitHub] aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r243009478 ## File path: NEWS.md ## @@ -1,6 +1,595 @@ -MXNet Change Log +Apache MXNet (incubating)

[GitHub] aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r243001643 ## File path: NEWS.md ## @@ -1,6 +1,595 @@ -MXNet Change Log +Apache MXNet (incubating)

[GitHub] aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release

2018-12-19 Thread GitBox
aaronmarkham commented on a change in pull request #13657: update with release notes for 1.4.0 release URL: https://github.com/apache/incubator-mxnet/pull/13657#discussion_r242997502 ## File path: NEWS.md ## @@ -1,6 +1,595 @@ -MXNet Change Log +Apache MXNet (incubating)

  1   2   >