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

2018-09-29 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 5c45592 Bump the publish ti

[GitHub] Aozorany commented on issue #8850: iOS compilation with amalgamate bash, undefined symbols for architecture arm64

2018-09-29 Thread GitBox
Aozorany commented on issue #8850: iOS compilation with amalgamate bash, undefined symbols for architecture arm64 URL: https://github.com/apache/incubator-mxnet/issues/8850#issuecomment-425698159 [only works on version 1.2.0 and 1.2.1](https://github.com/Aozorany/incubator-mxnet/tree/mast

[GitHub] Aozorany commented on issue #8525: how to make the IOS library?

2018-09-29 Thread GitBox
Aozorany commented on issue #8525: how to make the IOS library? URL: https://github.com/apache/incubator-mxnet/issues/8525#issuecomment-425698102 [see here](https://github.com/Aozorany/incubator-mxnet/tree/master/amalgamation) it only works on version 1.2.0 and 1.2.1 --

[GitHub] eric-haibin-lin commented on issue #9547: Implementing new operators in an external module

2018-09-29 Thread GitBox
eric-haibin-lin commented on issue #9547: Implementing new operators in an external module URL: https://github.com/apache/incubator-mxnet/issues/9547#issuecomment-425694913 Bump. Extensibility is very important This is an a

[GitHub] eric-haibin-lin commented on issue #12230: Update Operator Implementation Tutorial

2018-09-29 Thread GitBox
eric-haibin-lin commented on issue #12230: Update Operator Implementation Tutorial URL: https://github.com/apache/incubator-mxnet/pull/12230#issuecomment-425693827 My comments are addressed This is an automated message from t

[GitHub] eric-haibin-lin commented on issue #12226: Memory optimization in GLUON

2018-09-29 Thread GitBox
eric-haibin-lin commented on issue #12226: Memory optimization in GLUON URL: https://github.com/apache/incubator-mxnet/issues/12226#issuecomment-425693759 @samskalicky This is an automated message from the Apache Git Service

[GitHub] eric-haibin-lin closed pull request #10921: [MXNET-500]Test cases improvement for MKLDNN on Gluon

2018-09-29 Thread GitBox
eric-haibin-lin closed pull request #10921: [MXNET-500]Test cases improvement for MKLDNN on Gluon URL: https://github.com/apache/incubator-mxnet/pull/10921 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 prove

[incubator-mxnet] branch master updated: [MXNET-500]Test cases improvement for MKLDNN on Gluon (#10921)

2018-09-29 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 6522a2c [MXNET-500]Test cases improvem

[GitHub] KellenSunderland commented on issue #12631: [MXNET-953] Fix oob memory read

2018-09-29 Thread GitBox
KellenSunderland commented on issue #12631: [MXNET-953] Fix oob memory read URL: https://github.com/apache/incubator-mxnet/pull/12631#issuecomment-425686158 Thanks for continuing to give feedback @lebeg. The pros I see for that approach: * Handles OOB * Works for various dest

[GitHub] KellenSunderland commented on issue #12706: [MXNET-908] Speed up travis builds to avoid timeouts

2018-09-29 Thread GitBox
KellenSunderland commented on issue #12706: [MXNET-908] Speed up travis builds to avoid timeouts URL: https://github.com/apache/incubator-mxnet/pull/12706#issuecomment-425685949 @vandanavk PR should be ready for review. This

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

2018-09-29 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 f6f9a12 Bump the publish ti

[GitHub] stu1130 commented on a change in pull request #12696: Refine mxnet python installation

2018-09-29 Thread GitBox
stu1130 commented on a change in pull request #12696: Refine mxnet python installation URL: https://github.com/apache/incubator-mxnet/pull/12696#discussion_r221443668 ## File path: docs/install/windows_setup.md ## @@ -141,10 +141,10 @@ To build and install MXNet yourself u

[GitHub] ZhennanQin commented on a change in pull request #12530: Implement mkldnn convolution fusion and quantization.

2018-09-29 Thread GitBox
ZhennanQin commented on a change in pull request #12530: Implement mkldnn convolution fusion and quantization. URL: https://github.com/apache/incubator-mxnet/pull/12530#discussion_r221443657 ## File path: src/operator/subgraph/mkldnn/mkldnn_conv_property.cc ## @@ -0,0 +1,2

[GitHub] stu1130 commented on a change in pull request #12696: Refine mxnet python installation

2018-09-29 Thread GitBox
stu1130 commented on a change in pull request #12696: Refine mxnet python installation URL: https://github.com/apache/incubator-mxnet/pull/12696#discussion_r221443622 ## File path: docs/install/windows_setup.md ## @@ -221,9 +222,6 @@ libgfortran-3.dll (in mingw64_dll) lib

[GitHub] lebeg commented on issue #12382: [WIP] Fixes test_nccl_pushpull flaky test

2018-09-29 Thread GitBox
lebeg commented on issue #12382: [WIP] Fixes test_nccl_pushpull flaky test URL: https://github.com/apache/incubator-mxnet/pull/12382#issuecomment-425684457 @perdasilva Yes, I was. In fact if you look closer to the test than you realise that it would pass only in the case you have 1 GPU :)

[GitHub] lebeg commented on issue #10564: Simplified CUDA language detection in cmake

2018-09-29 Thread GitBox
lebeg commented on issue #10564: Simplified CUDA language detection in cmake URL: https://github.com/apache/incubator-mxnet/pull/10564#issuecomment-425684310 This PR is already a bit old and now it depends on https://github.com/apache/incubator-mxnet/pull/12331. I will take a look at updat

[GitHub] lebeg commented on issue #12331: Moves f16c autodetection to its own cmake module

2018-09-29 Thread GitBox
lebeg commented on issue #12331: Moves f16c autodetection to its own cmake module URL: https://github.com/apache/incubator-mxnet/pull/12331#issuecomment-425684254 @vandanavk This PR is good to go. This is an automated message

[GitHub] lebeg commented on issue #12631: [MXNET-953] Fix oob memory read

2018-09-29 Thread GitBox
lebeg commented on issue #12631: [MXNET-953] Fix oob memory read URL: https://github.com/apache/incubator-mxnet/pull/12631#issuecomment-425684038 I think with the cast it's better since you can avoid the copying. My suggestion would be the following: ``` MSHADOW_TYPE_SWITCH(out_data.t

[GitHub] ankkhedia commented on a change in pull request #12696: Refine mxnet python installation

2018-09-29 Thread GitBox
ankkhedia commented on a change in pull request #12696: Refine mxnet python installation URL: https://github.com/apache/incubator-mxnet/pull/12696#discussion_r221443132 ## File path: docs/install/windows_setup.md ## @@ -221,9 +222,6 @@ libgfortran-3.dll (in mingw64_dll) l

[GitHub] ankkhedia commented on a change in pull request #12696: Refine mxnet python installation

2018-09-29 Thread GitBox
ankkhedia commented on a change in pull request #12696: Refine mxnet python installation URL: https://github.com/apache/incubator-mxnet/pull/12696#discussion_r221443147 ## File path: docs/install/windows_setup.md ## @@ -141,10 +141,10 @@ To build and install MXNet yourself

[GitHub] zheng-da commented on a change in pull request #12502: [NGRAPH] MXNet - nGraph initial integration

2018-09-29 Thread GitBox
zheng-da commented on a change in pull request #12502: [NGRAPH] MXNet - nGraph initial integration URL: https://github.com/apache/incubator-mxnet/pull/12502#discussion_r221442287 ## File path: src/operator/contrib/ngraph-inl.h ## @@ -0,0 +1,158 @@ +/* + * Licensed to the A

[GitHub] zheng-da commented on a change in pull request #12530: Implement mkldnn convolution fusion and quantization.

2018-09-29 Thread GitBox
zheng-da commented on a change in pull request #12530: Implement mkldnn convolution fusion and quantization. URL: https://github.com/apache/incubator-mxnet/pull/12530#discussion_r221442074 ## File path: src/operator/subgraph/mkldnn/mkldnn_conv_property.cc ## @@ -0,0 +1,239

[GitHub] zheng-da opened a new pull request #12693: Fix small bugs in the benchmark of control flow operators.

2018-09-29 Thread GitBox
zheng-da opened a new pull request #12693: Fix small bugs in the benchmark of control flow operators. URL: https://github.com/apache/incubator-mxnet/pull/12693 ## Description ## The benchmark of control flow operators is a little out-of-date. This is to fix the small issues in the bench

[GitHub] zheng-da opened a new pull request #11948: [MXNET-766] add unroll RNN for HybridBlock

2018-09-29 Thread GitBox
zheng-da opened a new pull request #11948: [MXNET-766] add unroll RNN for HybridBlock URL: https://github.com/apache/incubator-mxnet/pull/11948 ## Description ## Currently, Gluon RNNCell unroll can't be used in Gluon HybridBlock. This unroll function works with an RNN cell and can be us

[GitHub] zheng-da closed pull request #11948: [MXNET-766] add unroll RNN for HybridBlock

2018-09-29 Thread GitBox
zheng-da closed pull request #11948: [MXNET-766] add unroll RNN for HybridBlock URL: https://github.com/apache/incubator-mxnet/pull/11948 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] zheng-da closed pull request #12693: Fix small bugs in the benchmark of control flow operators.

2018-09-29 Thread GitBox
zheng-da closed pull request #12693: Fix small bugs in the benchmark of control flow operators. URL: https://github.com/apache/incubator-mxnet/pull/12693 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 provena

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

2018-09-29 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 5a551ac Bump the publish ti

[GitHub] vandanavk commented on issue #12704: [MXNet-1002] refactor and sync nav bar between desktop and mobile

2018-09-29 Thread GitBox
vandanavk commented on issue #12704: [MXNet-1002] refactor and sync nav bar between desktop and mobile URL: https://github.com/apache/incubator-mxnet/pull/12704#issuecomment-425670765 @mxnet-label-bot [pr-awaiting-review, Doc] ---

[GitHub] vandanavk commented on issue #12706: WIP Measure travis

2018-09-29 Thread GitBox
vandanavk commented on issue #12706: WIP Measure travis URL: https://github.com/apache/incubator-mxnet/pull/12706#issuecomment-425670313 @mxnet-label-bot [pr-work-in-progress] This is an automated message from the Apache Git S

[GitHub] KellenSunderland opened a new pull request #12706: WIP Measure travis

2018-09-29 Thread GitBox
KellenSunderland opened a new pull request #12706: WIP Measure travis URL: https://github.com/apache/incubator-mxnet/pull/12706 WIP: profiling travis This is an automated message from the Apache Git Service. To respond to the

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

2018-09-29 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 03e4e9a Bump the publish ti

[GitHub] vandanavk commented on issue #12356: [MXNET-860] Use modernized ranged loops where possible

2018-09-29 Thread GitBox
vandanavk commented on issue #12356: [MXNET-860] Use modernized ranged loops where possible URL: https://github.com/apache/incubator-mxnet/pull/12356#issuecomment-425667194 @sandeep-krishnamurthy Please change the label to pr-awaiting-review

[GitHub] vandanavk commented on issue #12688: Enable test_gluon.test_export

2018-09-29 Thread GitBox
vandanavk commented on issue #12688: Enable test_gluon.test_export URL: https://github.com/apache/incubator-mxnet/pull/12688#issuecomment-425666839 @sandeep-krishnamurthy This PR is ready for merge This is an automated message

[GitHub] vandanavk commented on issue #12703: Performance Regression from #12019

2018-09-29 Thread GitBox
vandanavk commented on issue #12703: Performance Regression from #12019 URL: https://github.com/apache/incubator-mxnet/issues/12703#issuecomment-425666350 @mxnet-label-bot [Performance] This is an automated message from the A

[GitHub] lebeg commented on issue #12477: [WIP] Added build with nccl

2018-09-29 Thread GitBox
lebeg commented on issue #12477: [WIP] Added build with nccl URL: https://github.com/apache/incubator-mxnet/pull/12477#issuecomment-425663732 Would need to merge https://github.com/dmlc/ps-lite/pull/140 first to get the make builds running. -

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

2018-09-29 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 a778a08 Bump the publish ti