[GitHub] eric-haibin-lin commented on a change in pull request #11357: [MXNET-331] Single machine All Reduce Topology-aware Communication

2018-06-28 Thread GitBox
eric-haibin-lin commented on a change in pull request #11357: [MXNET-331] Single machine All Reduce Topology-aware Communication URL: https://github.com/apache/incubator-mxnet/pull/11357#discussion_r199057710 ## File path: src/kvstore/gpu_topology.h ## @@ -0,0 +1,1043 @@

[GitHub] eric-haibin-lin commented on a change in pull request #11357: [MXNET-331] Single machine All Reduce Topology-aware Communication

2018-06-28 Thread GitBox
eric-haibin-lin commented on a change in pull request #11357: [MXNET-331] Single machine All Reduce Topology-aware Communication URL: https://github.com/apache/incubator-mxnet/pull/11357#discussion_r199058554 ## File path: tests/python/gpu/test_kvstore_gpu.py ## @@ -88,34

[GitHub] eric-haibin-lin commented on a change in pull request #11357: [MXNET-331] Single machine All Reduce Topology-aware Communication

2018-06-28 Thread GitBox
eric-haibin-lin commented on a change in pull request #11357: [MXNET-331] Single machine All Reduce Topology-aware Communication URL: https://github.com/apache/incubator-mxnet/pull/11357#discussion_r199057204 ## File path: src/kvstore/comm.h ## @@ -750,6 +762,8 @@ class

[GitHub] hxhxhx88 opened a new issue #11480: Image classfication example has wrong accuracy metric.

2018-06-28 Thread GitBox
hxhxhx88 opened a new issue #11480: Image classfication example has wrong accuracy metric. URL: https://github.com/apache/incubator-mxnet/issues/11480 In file

[GitHub] zheng-da commented on issue #10921: [MXNET-500]Test cases improvement for MKLDNN on Gluon

2018-06-28 Thread GitBox
zheng-da commented on issue #10921: [MXNET-500]Test cases improvement for MKLDNN on Gluon URL: https://github.com/apache/incubator-mxnet/pull/10921#issuecomment-401248185 can you try rebase and see if it works? This is an

[GitHub] leezu commented on issue #11314: Embedding Backward (AddTakeGradLargeBatchCaller) non-deterministic nan values

2018-06-28 Thread GitBox
leezu commented on issue #11314: Embedding Backward (AddTakeGradLargeBatchCaller) non-deterministic nan values URL: https://github.com/apache/incubator-mxnet/issues/11314#issuecomment-401248199 There is some evidence that `nan` values can still occur with above workaround applied, though

[GitHub] Hui-Li commented on issue #9267: compiling from source failed

2018-06-28 Thread GitBox
Hui-Li commented on issue #9267: compiling from source failed URL: https://github.com/apache/incubator-mxnet/issues/9267#issuecomment-401245124 In case you are using Mac OS, it is probably that you use GCC as the compiler. Change to clang in **config.mk**: ``` export CC = clang

[GitHub] szha commented on issue #11287: [MXNET-548] fixed path for auto_module_index.js

2018-06-28 Thread GitBox
szha commented on issue #11287: [MXNET-548] fixed path for auto_module_index.js URL: https://github.com/apache/incubator-mxnet/pull/11287#issuecomment-401239592 @aaronmarkham could you do a rebase? some tests are already fixed recently.

[GitHub] szha commented on issue #11152: updating Scala IntelliJ tutorial & installation instructions

2018-06-28 Thread GitBox
szha commented on issue #11152: updating Scala IntelliJ tutorial & installation instructions URL: https://github.com/apache/incubator-mxnet/pull/11152#issuecomment-401239498 @aaronmarkham lol. you can use interactive rebase or squash to squash/fix-up all commits into one before rebasing,

[GitHub] szha commented on issue #11442: No module named 'dmlc_tracker'

2018-06-28 Thread GitBox
szha commented on issue #11442: No module named 'dmlc_tracker' URL: https://github.com/apache/incubator-mxnet/issues/11442#issuecomment-401239133 dmlc_tracker will be included in nightly builds starting from tonight. This is

[GitHub] zhulihuacmss commented on issue #7146: How to compile Amalgamation for android?

2018-06-28 Thread GitBox
zhulihuacmss commented on issue #7146: How to compile Amalgamation for android? URL: https://github.com/apache/incubator-mxnet/issues/7146#issuecomment-401237164 @Corea i have met the same problem 'can not find openblas.so', how did you solve it? thanks

[GitHub] marcoabreu closed pull request #11452: [Nightly Tests] Disable some nightly tests for installation guide

2018-06-28 Thread GitBox
marcoabreu closed pull request #11452: [Nightly Tests] Disable some nightly tests for installation guide URL: https://github.com/apache/incubator-mxnet/pull/11452 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[incubator-mxnet] branch master updated: [Nightly Tests] Disable some nightly tests for installation guide (#11452)

2018-06-28 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 a0669d0 [Nightly Tests] Disable

[incubator-mxnet] branch master updated: [MXNET-607] Fix the broken reported by the new BLC (#11465)

2018-06-28 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 a4054cd [MXNET-607] Fix the

[GitHub] marcoabreu closed pull request #11465: [MXNET-607] Fix the broken reported by the new BLC

2018-06-28 Thread GitBox
marcoabreu closed pull request #11465: [MXNET-607] Fix the broken reported by the new BLC URL: https://github.com/apache/incubator-mxnet/pull/11465 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:

[incubator-mxnet] branch master updated: fix regex which processes RAT check output (#11476)

2018-06-28 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 f02e627 fix regex which processes

[GitHub] marcoabreu closed pull request #11476: [Nightly Tests] Fix regex which processes Apache RAT check output

2018-06-28 Thread GitBox
marcoabreu closed pull request #11476: [Nightly Tests] Fix regex which processes Apache RAT check output URL: https://github.com/apache/incubator-mxnet/pull/11476 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] freshcoderman closed issue #11479: gamma require 2d

2018-06-28 Thread GitBox
freshcoderman closed issue #11479: gamma require 2d URL: https://github.com/apache/incubator-mxnet/issues/11479 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[GitHub] freshcoderman opened a new issue #11479: gamma require 2d

2018-06-28 Thread GitBox
freshcoderman opened a new issue #11479: gamma require 2d URL: https://github.com/apache/incubator-mxnet/issues/11479 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

[GitHub] rahul003 closed issue #11469: Performance regression in augmentation

2018-06-28 Thread GitBox
rahul003 closed issue #11469: Performance regression in augmentation URL: https://github.com/apache/incubator-mxnet/issues/11469 This is an automated message from the Apache Git Service. To respond to the message, please log

[incubator-mxnet] branch master updated: fix record io augmentation speed (#11474)

2018-06-28 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new b344d89 fix record io augmentation

[GitHub] szha closed pull request #11474: Fix record io augmentation speed

2018-06-28 Thread GitBox
szha closed pull request #11474: Fix record io augmentation speed URL: https://github.com/apache/incubator-mxnet/pull/11474 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

[GitHub] szha closed pull request #11379: support shared parameter in summary

2018-06-28 Thread GitBox
szha closed pull request #11379: support shared parameter in summary URL: https://github.com/apache/incubator-mxnet/pull/11379 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

[incubator-mxnet] branch master updated: support shared parameter in summary (#11379)

2018-06-28 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new 36f2aae support shared parameter in

[GitHub] szha commented on issue #11003: rnn_cell little bug fixed

2018-06-28 Thread GitBox
szha commented on issue #11003: rnn_cell little bug fixed URL: https://github.com/apache/incubator-mxnet/pull/11003#issuecomment-401226598 suppose you have a git remote called "upstream" (which you can get by doing `git remote add upstream https://github.com/apache/incubator-mxnet`), you

[GitHub] pengzhao-intel commented on issue #11448: fail to fall back when sparse arrays are passed to MKLDNN-enabled operators.

2018-06-28 Thread GitBox
pengzhao-intel commented on issue #11448: fail to fall back when sparse arrays are passed to MKLDNN-enabled operators. URL: https://github.com/apache/incubator-mxnet/issues/11448#issuecomment-401225029 what else need to cover? @zouluobao can help

[GitHub] anirudh2290 commented on issue #11478: 1.2.1 release notes

2018-06-28 Thread GitBox
anirudh2290 commented on issue #11478: 1.2.1 release notes URL: https://github.com/apache/incubator-mxnet/pull/11478#issuecomment-401224690 @aaronmarkham can you please take a look. This is an automated message from the

[GitHub] haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199034382 ## File path: tests/python/unittest/test_operator.py ## @@ -3009,6 +3009,49 @@ def npy_layer_norm(data,

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

2018-06-28 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 35b9788 Bump the publish

[GitHub] aaronmarkham commented on issue #11404: [MXNET-595] Install page not loading properly in Internet Explorer

2018-06-28 Thread GitBox
aaronmarkham commented on issue #11404: [MXNET-595] Install page not loading properly in Internet Explorer URL: https://github.com/apache/incubator-mxnet/pull/11404#issuecomment-401219131 @indhub this should be the fix for the issue you reported... can you merge?

[GitHub] anirudhacharya commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
anirudhacharya commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199029890 ## File path: src/operator/tensor/broadcast_reduce_op.h ## @@ -880,27 +880,24 @@ inline bool

[GitHub] leezu commented on issue #11403: L2 norm for fp16 goes to nan, needs to be fixed

2018-06-28 Thread GitBox
leezu commented on issue #11403: L2 norm for fp16 goes to nan, needs to be fixed URL: https://github.com/apache/incubator-mxnet/issues/11403#issuecomment-401217193 How about using a numerically stable L2 norm algorithm by scaling the elements? Ie. [BLAS

[GitHub] chinakook commented on issue #11003: rnn_cell little bug fixed

2018-06-28 Thread GitBox
chinakook commented on issue #11003: rnn_cell little bug fixed URL: https://github.com/apache/incubator-mxnet/pull/11003#issuecomment-401216200 Rebasing like this? I've not very familiar with rebasing. This is an automated

[GitHub] haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199028398 ## File path: tests/python/unittest/test_operator.py ## @@ -3009,6 +3009,49 @@ def npy_layer_norm(data,

[GitHub] anirudhacharya commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
anirudhacharya commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199028391 ## File path: src/operator/tensor/broadcast_reduce_op_value.cc ## @@ -284,10 +284,16 @@ array.

[GitHub] anirudhacharya commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
anirudhacharya commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199028119 ## File path: tests/python/unittest/test_operator.py ## @@ -3009,6 +3009,49 @@ def

[GitHub] haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199028037 ## File path: src/operator/tensor/broadcast_reduce_op_value.cc ## @@ -284,10 +284,16 @@ array. Examples::

[GitHub] anirudhacharya commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
anirudhacharya commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199027961 ## File path: tests/python/unittest/test_operator.py ## @@ -3009,6 +3009,49 @@ def

[GitHub] haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199027684 ## File path: src/operator/tensor/broadcast_reduce_op_value.cc ## @@ -284,10 +284,16 @@ array. Examples::

[GitHub] haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199027435 ## File path: src/operator/tensor/broadcast_reduce_op.h ## @@ -1021,8 +1023,38 @@ void

[GitHub] haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199027220 ## File path: src/operator/tensor/broadcast_reduce_op.h ## @@ -880,27 +880,24 @@ inline bool

[GitHub] anirudhacharya commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
anirudhacharya commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199026763 ## File path: src/operator/tensor/broadcast_reduce_op_value.cc ## @@ -284,10 +284,16 @@ array.

[GitHub] anirudhacharya commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
anirudhacharya commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199026810 ## File path: src/operator/tensor/broadcast_reduce_op_value.cc ## @@ -322,7 +328,7 @@

[GitHub] haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199026778 ## File path: tests/python/unittest/test_operator.py ## @@ -3009,6 +3009,49 @@ def npy_layer_norm(data,

[GitHub] eric-haibin-lin commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
eric-haibin-lin commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199026685 ## File path: src/operator/tensor/broadcast_reduce_op.h ## @@ -880,27 +880,24 @@ inline bool

[GitHub] andrewfayres commented on issue #11477: Copy profiler examples to new location for CI

2018-06-28 Thread GitBox
andrewfayres commented on issue #11477: Copy profiler examples to new location for CI URL: https://github.com/apache/incubator-mxnet/pull/11477#issuecomment-401214009 Odd, I know I updated these earlier. I'll take a look when I get home.

[GitHub] haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199026636 ## File path: tests/python/unittest/test_operator.py ## @@ -3009,6 +3009,49 @@ def npy_layer_norm(data,

[GitHub] threeleafzerg commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce

2018-06-28 Thread GitBox
threeleafzerg commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-401213560 Hi yang, I checked your error txt. Your build failure should be unrelated with my PR. e.g.

[GitHub] haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199025996 ## File path: src/operator/tensor/broadcast_reduce_op_value.cc ## @@ -284,10 +284,16 @@ array. Examples::

[GitHub] haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199025950 ## File path: src/operator/tensor/broadcast_reduce_op_value.cc ## @@ -284,10 +284,16 @@ array. Examples::

[GitHub] haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199025732 ## File path: src/operator/tensor/broadcast_reduce_op.h ## @@ -1021,8 +1023,38 @@ void

[GitHub] haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199025529 ## File path: src/operator/tensor/broadcast_reduce_op.h ## @@ -1021,8 +1023,38 @@ void

[GitHub] haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
haojin2 commented on a change in pull request #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#discussion_r199025529 ## File path: src/operator/tensor/broadcast_reduce_op.h ## @@ -1021,8 +1023,38 @@ void

[GitHub] anirudhacharya edited a comment on issue #11229: [MXNET-379] L1 Normalization

2018-06-28 Thread GitBox
anirudhacharya edited a comment on issue #11229: [MXNET-379] L1 Normalization URL: https://github.com/apache/incubator-mxnet/pull/11229#issuecomment-400887360 @haojin2 @eric-haibin-lin @piiswrong ping for review. This is an

[GitHub] anirudhacharya commented on issue #11475: Amending ONNX importer/exporter #11213

2018-06-28 Thread GitBox
anirudhacharya commented on issue #11475: Amending ONNX importer/exporter #11213 URL: https://github.com/apache/incubator-mxnet/issues/11475#issuecomment-401211603 Hi @zhreshold thanks for the feedback - I will make a PR to change ``python-pytest`` to ``onnx``. But the other

[GitHub] ArmageddonKnight commented on a change in pull request #11364: [MXNET-490] Added OpenLSTMRNN together with benchmarks and Tensorboard callback routines.

2018-06-28 Thread GitBox
ArmageddonKnight commented on a change in pull request #11364: [MXNET-490] Added OpenLSTMRNN together with benchmarks and Tensorboard callback routines. URL: https://github.com/apache/incubator-mxnet/pull/11364#discussion_r199024323 ## File path:

[incubator-mxnet] branch master updated: Document AdaGrad eps as initial history accumulator value (#11392)

2018-06-28 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new 2594fca Document AdaGrad eps as

[GitHub] szha closed pull request #11392: Document AdaGrad eps as initial history accumulator value

2018-06-28 Thread GitBox
szha closed pull request #11392: Document AdaGrad eps as initial history accumulator value URL: https://github.com/apache/incubator-mxnet/pull/11392 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] szha commented on issue #11003: rnn_cell little bug fixed

2018-06-28 Thread GitBox
szha commented on issue #11003: rnn_cell little bug fixed URL: https://github.com/apache/incubator-mxnet/pull/11003#issuecomment-401210997 @chinakook would you mind doing a rebase? we have a couple of flaky tests that are already fixed in the upstream.

[GitHub] azai91 commented on issue #11118: [MXNET-431] Get physical cores

2018-06-28 Thread GitBox
azai91 commented on issue #8: [MXNET-431] Get physical cores URL: https://github.com/apache/incubator-mxnet/pull/8#issuecomment-401210597 just chatted with @szha and the plan is to by default use cmake to build this dependency in the makefile (similar to what we do with MKLDNN). if

[GitHub] lanking520 commented on a change in pull request #11477: Copy profiler examples to new location for CI

2018-06-28 Thread GitBox
lanking520 commented on a change in pull request #11477: Copy profiler examples to new location for CI URL: https://github.com/apache/incubator-mxnet/pull/11477#discussion_r199023063 ## File path:

[GitHub] lanking520 commented on a change in pull request #11477: Copy profiler examples to new location for CI

2018-06-28 Thread GitBox
lanking520 commented on a change in pull request #11477: Copy profiler examples to new location for CI URL: https://github.com/apache/incubator-mxnet/pull/11477#discussion_r199022241 ## File path:

[GitHub] lanking520 commented on a change in pull request #11477: Copy profiler examples to new location for CI

2018-06-28 Thread GitBox
lanking520 commented on a change in pull request #11477: Copy profiler examples to new location for CI URL: https://github.com/apache/incubator-mxnet/pull/11477#discussion_r199023103 ## File path:

[GitHub] lanking520 commented on a change in pull request #11477: Copy profiler examples to new location for CI

2018-06-28 Thread GitBox
lanking520 commented on a change in pull request #11477: Copy profiler examples to new location for CI URL: https://github.com/apache/incubator-mxnet/pull/11477#discussion_r199023126 ## File path:

[GitHub] lanking520 commented on a change in pull request #11477: Copy profiler examples to new location for CI

2018-06-28 Thread GitBox
lanking520 commented on a change in pull request #11477: Copy profiler examples to new location for CI URL: https://github.com/apache/incubator-mxnet/pull/11477#discussion_r199022352 ## File path:

[GitHub] anirudh2290 opened a new pull request #11478: 1.2.1 release notes

2018-06-28 Thread GitBox
anirudh2290 opened a new pull request #11478: 1.2.1 release notes URL: https://github.com/apache/incubator-mxnet/pull/11478 ## Description ## I have added more info to 1.2.1 release notes from what @mli sent on the mailing list. Please help review: Once this is approved , I will

[GitHub] haojin2 commented on issue #11473: batchnorm falls back for sparse arrays.

2018-06-28 Thread GitBox
haojin2 commented on issue #11473: batchnorm falls back for sparse arrays. URL: https://github.com/apache/incubator-mxnet/pull/11473#issuecomment-401208891 I think we do need some extra checks for this to verify that this works

[GitHub] andrewfayres opened a new pull request #11477: Copy profiler examples to new location for CI

2018-06-28 Thread GitBox
andrewfayres opened a new pull request #11477: Copy profiler examples to new location for CI URL: https://github.com/apache/incubator-mxnet/pull/11477 ## Description ## As part of the effort to update models to the new inference API we're also moving to a new folder. This example isn't

[GitHub] mbaijal opened a new pull request #11476: [Nightly Tests] Fix regex which processes Apache RAT check output

2018-06-28 Thread GitBox
mbaijal opened a new pull request #11476: [Nightly Tests] Fix regex which processes Apache RAT check output URL: https://github.com/apache/incubator-mxnet/pull/11476 ## Description ## I had recently updated the `license_check.sh` script to process the output correctly for an edge case

[GitHub] ctcyang edited a comment on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce

2018-06-28 Thread GitBox
ctcyang edited a comment on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-401201724 I am having trouble building 1444a84 with GPU support, so I was hoping you could help me. `make` seems to

[GitHub] toddsundsted edited a comment on issue #11398: Floating Point Exception after Array Creation

2018-06-28 Thread GitBox
toddsundsted edited a comment on issue #11398: Floating Point Exception after Array Creation URL: https://github.com/apache/incubator-mxnet/issues/11398#issuecomment-400872858 @frankfliu I have a proposed fix here: https://github.com/apache/incubator-mxnet/pull/11397 this small

[GitHub] hetong007 commented on issue #11474: Fix record io augmentation speed

2018-06-28 Thread GitBox
hetong007 commented on issue #11474: Fix record io augmentation speed URL: https://github.com/apache/incubator-mxnet/pull/11474#issuecomment-401206035 Confirmed that this brings back the speed on both Symbol and Gluon training.

[GitHub] szha commented on a change in pull request #11364: [MXNET-490] Added OpenLSTMRNN together with benchmarks and Tensorboard callback routines.

2018-06-28 Thread GitBox
szha commented on a change in pull request #11364: [MXNET-490] Added OpenLSTMRNN together with benchmarks and Tensorboard callback routines. URL: https://github.com/apache/incubator-mxnet/pull/11364#discussion_r199017711 ## File path:

[GitHub] piiswrong closed pull request #10524: [MXNET-312] Added Matthew's Correlation Coefficient to metrics

2018-06-28 Thread GitBox
piiswrong closed pull request #10524: [MXNET-312] Added Matthew's Correlation Coefficient to metrics URL: https://github.com/apache/incubator-mxnet/pull/10524 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] anirudhacharya commented on issue #11475: Amending ONNX importer/exporter #11213

2018-06-28 Thread GitBox
anirudhacharya commented on issue #11475: Amending ONNX importer/exporter #11213 URL: https://github.com/apache/incubator-mxnet/issues/11475#issuecomment-401203574 @spidyDev @Roshrini This is an automated message from the

[GitHub] szha closed pull request #11447: [documentation] Update doc for contrib.rand_zipfian

2018-06-28 Thread GitBox
szha closed pull request #11447: [documentation] Update doc for contrib.rand_zipfian URL: https://github.com/apache/incubator-mxnet/pull/11447 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] szha commented on a change in pull request #10524: [MXNET-312] Added Matthew's Correlation Coefficient to metrics

2018-06-28 Thread GitBox
szha commented on a change in pull request #10524: [MXNET-312] Added Matthew's Correlation Coefficient to metrics URL: https://github.com/apache/incubator-mxnet/pull/10524#discussion_r199016678 ## File path: tests/python/unittest/test_metric.py ## @@ -122,6 +123,54 @@ def

[GitHub] ctcyang commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce

2018-06-28 Thread GitBox
ctcyang commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-401201724 I am having trouble building 1444a84 with GPU support, so I was hoping your could help me. `make` seems to cause

[incubator-mxnet] branch master updated: fix. (#11471)

2018-06-28 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new 4cdc150 fix. (#11471) 4cdc150 is

[GitHub] szha closed pull request #11471: Fix MKLDNN build for Mac.

2018-06-28 Thread GitBox
szha closed pull request #11471: Fix MKLDNN build for Mac. URL: https://github.com/apache/incubator-mxnet/pull/11471 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] szha closed pull request #11444: Fix flaky test test_operator_gpu.test_spatial_transformer_with_type (…

2018-06-28 Thread GitBox
szha closed pull request #11444: Fix flaky test test_operator_gpu.test_spatial_transformer_with_type (… URL: https://github.com/apache/incubator-mxnet/pull/11444 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[incubator-mxnet] branch master updated: Fix flaky test test_operator_gpu.test_spatial_transformer_with_type (#7645) (#11444)

2018-06-28 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new f7a0025 Fix flaky test

[GitHub] szha commented on issue #11213: [MXNET-533] MXNet-ONNX export

2018-06-28 Thread GitBox
szha commented on issue #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#issuecomment-401200760 @sandeep-krishnamurthy thanks for the efforts. Please respect "request changes" as vetos nonetheless and try and reach @zhreshold, especially given

[GitHub] Caenorst commented on a change in pull request #11325: Added TensorRT runtime integration

2018-06-28 Thread GitBox
Caenorst commented on a change in pull request #11325: Added TensorRT runtime integration URL: https://github.com/apache/incubator-mxnet/pull/11325#discussion_r199014063 ## File path: src/executor/tensorrt_pass.cc ## @@ -0,0 +1,583 @@ +/* + * Licensed to the Apache

[GitHub] zhreshold commented on issue #11213: [MXNET-533] MXNet-ONNX export

2018-06-28 Thread GitBox
zhreshold commented on issue #11213: [MXNET-533] MXNet-ONNX export URL: https://github.com/apache/incubator-mxnet/pull/11213#issuecomment-401197110 I opened a new issue regarding my concerns in #11475 This is an automated

[GitHub] zheng-da commented on a change in pull request #11371: [MXNET-486] Create CPP test for concat MKLDNN operator

2018-06-28 Thread GitBox
zheng-da commented on a change in pull request #11371: [MXNET-486] Create CPP test for concat MKLDNN operator URL: https://github.com/apache/incubator-mxnet/pull/11371#discussion_r199010760 ## File path: tests/cpp/operator/mkldnn.cc ## @@ -804,6 +1012,24 @@

[GitHub] aaronmarkham commented on a change in pull request #11304: [MXNET-594] Added Learning Rate Finder tutorial

2018-06-28 Thread GitBox
aaronmarkham commented on a change in pull request #11304: [MXNET-594] Added Learning Rate Finder tutorial URL: https://github.com/apache/incubator-mxnet/pull/11304#discussion_r199007868 ## File path: docs/tutorials/gluon/learning_rate_finder.md ## @@ -0,0 +1,322 @@ + +#

[GitHub] aaronmarkham commented on a change in pull request #11304: [MXNET-594] Added Learning Rate Finder tutorial

2018-06-28 Thread GitBox
aaronmarkham commented on a change in pull request #11304: [MXNET-594] Added Learning Rate Finder tutorial URL: https://github.com/apache/incubator-mxnet/pull/11304#discussion_r199008464 ## File path: docs/tutorials/gluon/learning_rate_finder.md ## @@ -0,0 +1,322 @@ + +#

[GitHub] zheng-da commented on issue #11448: fail to fall back when sparse arrays are passed to MKLDNN-enabled operators.

2018-06-28 Thread GitBox
zheng-da commented on issue #11448: fail to fall back when sparse arrays are passed to MKLDNN-enabled operators. URL: https://github.com/apache/incubator-mxnet/issues/11448#issuecomment-401195601 i created a PR for batchnorm.

[GitHub] hetong007 opened a new pull request #11474: Fix record io augmentation speed

2018-06-28 Thread GitBox
hetong007 opened a new pull request #11474: Fix record io augmentation speed URL: https://github.com/apache/incubator-mxnet/pull/11474 ## Description ## 1. Fixing https://github.com/apache/incubator-mxnet/issues/11469 2. Fixing default augmentation setting in current image

[GitHub] zheng-da opened a new pull request #11473: batchnorm falls back for sparse arrays.

2018-06-28 Thread GitBox
zheng-da opened a new pull request #11473: batchnorm falls back for sparse arrays. URL: https://github.com/apache/incubator-mxnet/pull/11473 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove

[GitHub] rahul003 commented on issue #11469: Performance regression in augmentation

2018-06-28 Thread GitBox
rahul003 commented on issue #11469: Performance regression in augmentation URL: https://github.com/apache/incubator-mxnet/issues/11469#issuecomment-401194743 That was it. Was accidentally performing an unnecessary affine transform before cropping. Tong is sending out a PR to fix it.

[GitHub] rahul003 edited a comment on issue #11469: Performance regression in augmentation

2018-06-28 Thread GitBox
rahul003 edited a comment on issue #11469: Performance regression in augmentation URL: https://github.com/apache/incubator-mxnet/issues/11469#issuecomment-401194743 That was it. Was accidentally performing an unnecessary affine transform before cropping. Tong is sending out a PR to

[GitHub] gigasquid commented on a change in pull request #11205: Clojure Contrib Package

2018-06-28 Thread GitBox
gigasquid commented on a change in pull request #11205: Clojure Contrib Package URL: https://github.com/apache/incubator-mxnet/pull/11205#discussion_r199008534 ## File path: contrib/clojure-package/README.md ## @@ -0,0 +1,204 @@ +# Clojure MXNet + +A clojure package to the

[GitHub] sandeep-krishnamurthy edited a comment on issue #11406: [MXNET-599] Partial shape infer for Slice

2018-06-28 Thread GitBox
sandeep-krishnamurthy edited a comment on issue #11406: [MXNET-599] Partial shape infer for Slice URL: https://github.com/apache/incubator-mxnet/pull/11406#issuecomment-401194229 Thanks a lot @rahul003 - This is very useful fix for all Keras-MXNet users using slice operation. Overall

[GitHub] sandeep-krishnamurthy commented on issue #11406: [MXNET-599] Partial shape infer for Slice

2018-06-28 Thread GitBox
sandeep-krishnamurthy commented on issue #11406: [MXNET-599] Partial shape infer for Slice URL: https://github.com/apache/incubator-mxnet/pull/11406#issuecomment-401194229 Thanks a lot @rahul003 - This is very useful fix for all Keras-MXNet users using slice operation.

[GitHub] mkolod commented on a change in pull request #11325: Added TensorRT runtime integration

2018-06-28 Thread GitBox
mkolod commented on a change in pull request #11325: Added TensorRT runtime integration URL: https://github.com/apache/incubator-mxnet/pull/11325#discussion_r199008230 ## File path: src/executor/tensorrt_pass.cc ## @@ -0,0 +1,583 @@ +/* + * Licensed to the Apache Software

[GitHub] sandeep-krishnamurthy commented on a change in pull request #11406: [MXNET-599] Partial shape infer for Slice

2018-06-28 Thread GitBox
sandeep-krishnamurthy commented on a change in pull request #11406: [MXNET-599] Partial shape infer for Slice URL: https://github.com/apache/incubator-mxnet/pull/11406#discussion_r199007699 ## File path: src/operator/tensor/matrix_op-inl.h ## @@ -674,13 +675,23 @@ inline

[GitHub] mkolod commented on a change in pull request #11325: Added TensorRT runtime integration

2018-06-28 Thread GitBox
mkolod commented on a change in pull request #11325: Added TensorRT runtime integration URL: https://github.com/apache/incubator-mxnet/pull/11325#discussion_r199006909 ## File path: src/executor/graph_executor.cc ## @@ -940,6 +968,91 @@ void

[GitHub] aaronmarkham commented on issue #11287: [MXNET-548] fixed path for auto_module_index.js

2018-06-28 Thread GitBox
aaronmarkham commented on issue #11287: [MXNET-548] fixed path for auto_module_index.js URL: https://github.com/apache/incubator-mxnet/pull/11287#issuecomment-401192228 5th restart due to flaky CI... This is an automated

  1   2   3   4   >