[GitHub] daleydeng opened a new pull request #13559: fix for opencv4

2018-12-05 Thread GitBox
daleydeng opened a new pull request #13559: fix for opencv4 URL: https://github.com/apache/incubator-mxnet/pull/13559 ## Description ## Fix compatibility for opencv 4 with flags started with cv namespace. This is an

[GitHub] TaoLv commented on issue #13544: turn on Sphinx warnings as errors

2018-12-05 Thread GitBox
TaoLv commented on issue #13544: turn on Sphinx warnings as errors URL: https://github.com/apache/incubator-mxnet/pull/13544#issuecomment-444780008 @aaronmarkham I have submitted a PR to revert #13478 and am waiting for the CI finishing.

[GitHub] TaoLv opened a new pull request #13558: Revert "Bumped minor version from 1.4.0 to 1.5.0 on master, updated License file"

2018-12-05 Thread GitBox
TaoLv opened a new pull request #13558: Revert "Bumped minor version from 1.4.0 to 1.5.0 on master, updated License file" URL: https://github.com/apache/incubator-mxnet/pull/13558 Reverts apache/incubator-mxnet#13478 #13478 brought in some non-related code change including those

[incubator-mxnet] 04/09: Revert "Fix #13521 (#13537)"

2018-12-05 Thread taolv
This is an automated email from the ASF dual-hosted git repository. taolv pushed a commit to branch revert-13478-master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit e5271672a89f330548575507abda46cd08bbfbc4 Author: Tao Lv AuthorDate: Thu Dec 6 15:34:18 2018 +0800

[incubator-mxnet] branch revert-13478-master created (now b228241)

2018-12-05 Thread taolv
This is an automated email from the ASF dual-hosted git repository. taolv pushed a change to branch revert-13478-master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. at b228241 Revert "Bumped minor version from 1.4.0 to 1.5.0 on master, updated License file

[incubator-mxnet] 02/09: Revert "Updated docs for randint operator (#13541)"

2018-12-05 Thread taolv
This is an automated email from the ASF dual-hosted git repository. taolv pushed a commit to branch revert-13478-master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit 61a0b69607eff3b15f3ea6bb60f4115759bf547c Author: Tao Lv AuthorDate: Thu Dec 6 15:34:18 2018 +0800

[incubator-mxnet] 06/09: Revert "[MXNET-769] Use MXNET_HOME in a tempdir in windows to prevent access denied due t… (#13531)"

2018-12-05 Thread taolv
This is an automated email from the ASF dual-hosted git repository. taolv pushed a commit to branch revert-13478-master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit c014a4b90372ea72d4fbb5c71d29d4209235fe46 Author: Tao Lv AuthorDate: Thu Dec 6 15:34:18 2018 +0800

[incubator-mxnet] 08/09: Revert "Fixing a 404 in the ubuntu setup doc (#13542)"

2018-12-05 Thread taolv
This is an automated email from the ASF dual-hosted git repository. taolv pushed a commit to branch revert-13478-master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit 171786e6a5d7aa61107d901e1eef0f77067df5e7 Author: Tao Lv AuthorDate: Thu Dec 6 15:34:18 2018 +0800

[incubator-mxnet] 09/09: Revert "Bumped minor version from 1.4.0 to 1.5.0 on master, updated License file (#13478)"

2018-12-05 Thread taolv
This is an automated email from the ASF dual-hosted git repository. taolv pushed a commit to branch revert-13478-master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit b2282417b8b2bbba9abc11bf96b235d6674f9731 Author: Tao Lv AuthorDate: Thu Dec 6 15:34:18 2018 +0800

[incubator-mxnet] 01/09: Revert "Chi_square_check for discrete distribution fix (#13543)"

2018-12-05 Thread taolv
This is an automated email from the ASF dual-hosted git repository. taolv pushed a commit to branch revert-13478-master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit d145e7dbd34f2a2fd5393a7f15d403e31bcde171 Author: Tao Lv AuthorDate: Thu Dec 6 15:34:18 2018 +0800

[incubator-mxnet] 07/09: Revert "[MXNET-1249] Fix Object Detector Performance with GPU (#13522)"

2018-12-05 Thread taolv
This is an automated email from the ASF dual-hosted git repository. taolv pushed a commit to branch revert-13478-master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit 30f2bde4da0ae4973050f417efbb29fa30b2639e Author: Tao Lv AuthorDate: Thu Dec 6 15:34:18 2018 +0800

[incubator-mxnet] 05/09: Revert "Add a retry to qemu_provision (#13551)"

2018-12-05 Thread taolv
This is an automated email from the ASF dual-hosted git repository. taolv pushed a commit to branch revert-13478-master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit 66e38c37a8bf97e9b7607a080d02a487f1eba0e0 Author: Tao Lv AuthorDate: Thu Dec 6 15:34:18 2018 +0800

[incubator-mxnet] 03/09: Revert "Simplifications and some fun stuff for the MNIST Gluon tutorial (#13094)"

2018-12-05 Thread taolv
This is an automated email from the ASF dual-hosted git repository. taolv pushed a commit to branch revert-13478-master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git commit b9096f42fd0cde31baca79efadcc5533d67df644 Author: Tao Lv AuthorDate: Thu Dec 6 15:34:18 2018 +0800

[GitHub] TaoLv commented on issue #13478: Bumped minor version from 1.4.0 to 1.5.0 on master, updated License file

2018-12-05 Thread GitBox
TaoLv commented on issue #13478: Bumped minor version from 1.4.0 to 1.5.0 on master, updated License file URL: https://github.com/apache/incubator-mxnet/pull/13478#issuecomment-444775918 @srochel I will revert this PR on the master branch to mitigate issues. Feel free to submit new PR for

[GitHub] yzhliu commented on issue #13543: Chi_square_check for discrete distribution fix

2018-12-05 Thread GitBox
yzhliu commented on issue #13543: Chi_square_check for discrete distribution fix URL: https://github.com/apache/incubator-mxnet/pull/13543#issuecomment-444774416 Nice catch! This is an automated message from the Apache Git

[GitHub] Vikas89 commented on issue #13526: distributed training van.cc Check failed

2018-12-05 Thread GitBox
Vikas89 commented on issue #13526: distributed training van.cc Check failed URL: https://github.com/apache/incubator-mxnet/issues/13526#issuecomment-444773980 is this ec2 or your own host? please share the command you use to ssh-ing to master instance? 1/ use ssh -A to login to

[GitHub] srochel commented on issue #13478: Bumped minor version from 1.4.0 to 1.5.0 on master, updated License file

2018-12-05 Thread GitBox
srochel commented on issue #13478: Bumped minor version from 1.4.0 to 1.5.0 on master, updated License file URL: https://github.com/apache/incubator-mxnet/pull/13478#issuecomment-444773282 @TaoL - sorry about this. Looks like I made a mistake with the PR and see that mklddn related

[incubator-mxnet] branch master updated (e0ff3c3 -> cf6e8cb)

2018-12-05 Thread liuyizhi
This is an automated email from the ASF dual-hosted git repository. liuyizhi pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from e0ff3c3 Updated docs for randint operator (#13541) add cf6e8cb Chi_square_check for discrete

[GitHub] yzhliu closed pull request #13543: Chi_square_check for discrete distribution fix

2018-12-05 Thread GitBox
yzhliu closed pull request #13543: Chi_square_check for discrete distribution fix URL: https://github.com/apache/incubator-mxnet/pull/13543 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] CyberZHG opened a new pull request #13557: Fix BatchNorm converter for CoreML when fix_gamma=True

2018-12-05 Thread GitBox
CyberZHG opened a new pull request #13557: Fix BatchNorm converter for CoreML when fix_gamma=True URL: https://github.com/apache/incubator-mxnet/pull/13557 ## Description ## Set gamma values to ones while converting batch normalization in case of accidental changes of gamma.

[GitHub] TaoLv edited a comment on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
TaoLv edited a comment on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444771242 Tested this PR on CPU w/ and w/o MKL-DNN and got error message as below. w/o MKL-DNN: ``` Traceback (most

[GitHub] TaoLv commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
TaoLv commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444771242 Tested this PR on CPU w/ and w/o MKL-DNN and got error message as below. w/o MKL-DNN: ``` Traceback (most recent call

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

2018-12-05 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 4f98049 Bump the publish

[GitHub] aaronmarkham edited a comment on issue #13544: turn on Sphinx warnings as errors

2018-12-05 Thread GitBox
aaronmarkham edited a comment on issue #13544: turn on Sphinx warnings as errors URL: https://github.com/apache/incubator-mxnet/pull/13544#issuecomment-444761968 Well that was a bit baffling... I rebased and the bug fixes were gone. Looks like this [commit for

[GitHub] aaronmarkham commented on issue #13544: turn on Sphinx warnings as errors

2018-12-05 Thread GitBox
aaronmarkham commented on issue #13544: turn on Sphinx warnings as errors URL: https://github.com/apache/incubator-mxnet/pull/13544#issuecomment-444761968 Well that was a bit baffling... I rebased and the bug fixes were gone. Looks like this [commit for

[GitHub] TaoLv edited a comment on issue #13478: Bumped minor version from 1.4.0 to 1.5.0 on master, updated License file

2018-12-05 Thread GitBox
TaoLv edited a comment on issue #13478: Bumped minor version from 1.4.0 to 1.5.0 on master, updated License file URL: https://github.com/apache/incubator-mxnet/pull/13478#issuecomment-444759570 @srochel @sergeykolychev Seems this PR was messed up by code rebasing and many non-related code

[GitHub] aaronmarkham commented on issue #13544: turn on Sphinx warnings as errors

2018-12-05 Thread GitBox
aaronmarkham commented on issue #13544: turn on Sphinx warnings as errors URL: https://github.com/apache/incubator-mxnet/pull/13544#issuecomment-444759810 It failed on the website pipeline. So, yay? The failure is on a docs bug we already fixed here:

[GitHub] TaoLv commented on issue #13478: Bumped minor version from 1.4.0 to 1.5.0 on master, updated License file

2018-12-05 Thread GitBox
TaoLv commented on issue #13478: Bumped minor version from 1.4.0 to 1.5.0 on master, updated License file URL: https://github.com/apache/incubator-mxnet/pull/13478#issuecomment-444759570 @srochel @sergeykolychev Seems this PR was messed up by code rebasing and many non-related code was

[incubator-mxnet] branch master updated: Updated docs for randint operator (#13541)

2018-12-05 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 e0ff3c3 Updated docs for

[GitHub] aaronmarkham closed pull request #13541: Updated docs for randint operator

2018-12-05 Thread GitBox
aaronmarkham closed pull request #13541: Updated docs for randint operator URL: https://github.com/apache/incubator-mxnet/pull/13541 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] szha opened a new pull request #13556: build config for maven and pip

2018-12-05 Thread GitBox
szha opened a new pull request #13556: build config for maven and pip URL: https://github.com/apache/incubator-mxnet/pull/13556 ## Description ## build config for maven and pip ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. -

[GitHub] apeforest removed a comment on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
apeforest removed a comment on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444753750 @DickJC123 Thanks for your detailed explanation about the operator selection logic. I like the way you used a

[GitHub] aaronmarkham commented on issue #13544: turn on Sphinx warnings as errors

2018-12-05 Thread GitBox
aaronmarkham commented on issue #13544: turn on Sphinx warnings as errors URL: https://github.com/apache/incubator-mxnet/pull/13544#issuecomment-444753836 > At what stage in the CI is this check performed? Can you please point to the logs where we can check and confirm? I just added

[GitHub] apeforest commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
apeforest commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444753750 @DickJC123 Thanks for your detailed explanation about the operator selection logic. I like the way you used a boolean

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

2018-12-05 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-444753144 unix-gpu error is weird: ``` Sending interrupt signal to process ``` any clue why did it fail?

[GitHub] aaronmarkham commented on a change in pull request #13544: turn on Sphinx warnings as errors

2018-12-05 Thread GitBox
aaronmarkham commented on a change in pull request #13544: turn on Sphinx warnings as errors URL: https://github.com/apache/incubator-mxnet/pull/13544#discussion_r239331752 ## File path: docs/build_version_doc/build_all_version.sh ## @@ -117,6 +120,8 @@ function checkout

[GitHub] apeforest commented on issue #13555: [MXNET-1253] fix control_flow_op

2018-12-05 Thread GitBox
apeforest commented on issue #13555: [MXNET-1253] fix control_flow_op URL: https://github.com/apache/incubator-mxnet/pull/13555#issuecomment-444748084 @zheng-da @anirudh2290 @sandeep-krishnamurthy @yuxihu @frankfliu Please review

[GitHub] apeforest opened a new pull request #13555: [MXNET-1253] fix control_flow_op

2018-12-05 Thread GitBox
apeforest opened a new pull request #13555: [MXNET-1253] fix control_flow_op URL: https://github.com/apache/incubator-mxnet/pull/13555 ## Description ## Support large array in control_flow_ops ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items

[GitHub] huangzhiyuan commented on a change in pull request #12980: Add reshape op supported by MKL-DNN

2018-12-05 Thread GitBox
huangzhiyuan commented on a change in pull request #12980: Add reshape op supported by MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/12980#discussion_r239327409 ## File path: src/operator/tensor/matrix_op.cc ## @@ -914,7 +972,7 @@

[GitHub] anirudh2290 commented on issue #13438: libc getenv is not threadsafe

2018-12-05 Thread GitBox
anirudh2290 commented on issue #13438: libc getenv is not threadsafe URL: https://github.com/apache/incubator-mxnet/issues/13438#issuecomment-444741884 The problem as provided in the article linked in this issue and related article here: https://rachelbythebay.com/w/2017/01/30/env/ is

[GitHub] TaoLv commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
TaoLv commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444741474 Sure. It's a problem of MXNet, not this PR. Do you mind documenting some where what does the output layout look like? Then user

[GitHub] anirudhacharya commented on issue #13544: turn on Sphinx warnings as errors

2018-12-05 Thread GitBox
anirudhacharya commented on issue #13544: turn on Sphinx warnings as errors URL: https://github.com/apache/incubator-mxnet/pull/13544#issuecomment-444736486 At what stage in the CI is this check performed? Can you please point to the logs where we can check and confirm?

[GitHub] DickJC123 commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
DickJC123 commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444734362 The forced transposition I referred to is done currently by the user in the python model. With this new feature, the

[GitHub] pengzhao-intel commented on issue #13515: MKLDNN Perplexity Issue

2018-12-05 Thread GitBox
pengzhao-intel commented on issue #13515: MKLDNN Perplexity Issue URL: https://github.com/apache/incubator-mxnet/issues/13515#issuecomment-444730942 Good to know the problem is fixed :) Feel free to ping me if anything needs our help.

[GitHub] marcoabreu commented on a change in pull request #13544: turn on Sphinx warnings as errors

2018-12-05 Thread GitBox
marcoabreu commented on a change in pull request #13544: turn on Sphinx warnings as errors URL: https://github.com/apache/incubator-mxnet/pull/13544#discussion_r239313917 ## File path: docs/build_version_doc/build_all_version.sh ## @@ -117,6 +120,8 @@ function checkout ()

[GitHub] marcoabreu commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
marcoabreu commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444730273 I love that approach about the operator selection, dick!

[GitHub] TaoLv commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
TaoLv commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444728394 > Prior to this PR, in MXNet, the Pooling operator did not have a parameter to specify the layout, forcing a transposition

[GitHub] Davdi edited a comment on issue #13526: distributed training van.cc Check failed

2018-12-05 Thread GitBox
Davdi edited a comment on issue #13526: distributed training van.cc Check failed URL: https://github.com/apache/incubator-mxnet/issues/13526#issuecomment-444727742 > As I see there are 3 different issues here: > > ``` > File

[GitHub] Davdi commented on issue #13526: distributed training van.cc Check failed

2018-12-05 Thread GitBox
Davdi commented on issue #13526: distributed training van.cc Check failed URL: https://github.com/apache/incubator-mxnet/issues/13526#issuecomment-444727742 > As I see there are 3 different issues here: > > ``` > File "/usr/local/lib/python3.6/dist-packages/mxnet/base.py", line

[GitHub] tranvanhoa533 commented on issue #12001: SyncBatchNorm problems

2018-12-05 Thread GitBox
tranvanhoa533 commented on issue #12001: SyncBatchNorm problems URL: https://github.com/apache/incubator-mxnet/issues/12001#issuecomment-444726902 @kaleidoscopical Which version of mxnet did you use ? I use mxnet version 1.3.1 but It still runs into a fail of asnumpy().

[GitHub] hellonico opened a new pull request #13554: #13385 [Clojure] - Turn examples into integration tests

2018-12-05 Thread GitBox
hellonico opened a new pull request #13554: #13385 [Clojure] - Turn examples into integration tests URL: https://github.com/apache/incubator-mxnet/pull/13554 !!! This is work in progress. Would: - the committed script - adding a bare minimum deftest inside each example

[GitHub] Joke09 commented on issue #13545: For inference, I have the same problem. The client send jpg to server, then the server use cv2 to do resize. When put the image data into the mx.nd.array, it

2018-12-05 Thread GitBox
Joke09 commented on issue #13545: For inference, I have the same problem. The client send jpg to server, then the server use cv2 to do resize. When put the image data into the mx.nd.array, it's very slow. And the Utilization of GPU is low too. How to solve it? Thank you! URL:

[GitHub] ChaiBapchya commented on a change in pull request #13541: Updated docs for randint operator

2018-12-05 Thread GitBox
ChaiBapchya commented on a change in pull request #13541: Updated docs for randint operator URL: https://github.com/apache/incubator-mxnet/pull/13541#discussion_r239306194 ## File path: docs/api/python/ndarray/random.md ## @@ -37,6 +37,7 @@ In the rest of this document,

[GitHub] ChaiBapchya commented on a change in pull request #13541: Updated docs for randint operator

2018-12-05 Thread GitBox
ChaiBapchya commented on a change in pull request #13541: Updated docs for randint operator URL: https://github.com/apache/incubator-mxnet/pull/13541#discussion_r239306068 ## File path: docs/api/python/symbol/random.md ## @@ -37,6 +37,7 @@ In the rest of this document, we

[GitHub] ChaiBapchya commented on a change in pull request #13541: Updated docs for randint operator

2018-12-05 Thread GitBox
ChaiBapchya commented on a change in pull request #13541: Updated docs for randint operator URL: https://github.com/apache/incubator-mxnet/pull/13541#discussion_r239306053 ## File path: docs/api/python/symbol/symbol.md ## @@ -595,6 +595,7 @@ Composite multiple symbols

[GitHub] ZhennanQin commented on a change in pull request #12980: Add reshape op supported by MKL-DNN

2018-12-05 Thread GitBox
ZhennanQin commented on a change in pull request #12980: Add reshape op supported by MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/12980#discussion_r239306007 ## File path: src/operator/tensor/matrix_op.cc ## @@ -210,24 +272,18 @@ static void FlattenEx(const

[GitHub] ChaiBapchya commented on a change in pull request #13541: Updated docs for randint operator

2018-12-05 Thread GitBox
ChaiBapchya commented on a change in pull request #13541: Updated docs for randint operator URL: https://github.com/apache/incubator-mxnet/pull/13541#discussion_r239306040 ## File path: docs/api/python/ndarray/random.md ## @@ -37,6 +37,7 @@ In the rest of this document,

[GitHub] ChaiBapchya commented on a change in pull request #13541: Updated docs for randint operator

2018-12-05 Thread GitBox
ChaiBapchya commented on a change in pull request #13541: Updated docs for randint operator URL: https://github.com/apache/incubator-mxnet/pull/13541#discussion_r239306027 ## File path: docs/api/python/ndarray/ndarray.md ## @@ -596,6 +596,7 @@ The `ndarray` package

[GitHub] Joke09 edited a comment on issue #13545: For inference, I have the same problem. The client send jpg to server, then the server use cv2 to do resize. When put the image data into the mx.nd.ar

2018-12-05 Thread GitBox
Joke09 edited a comment on issue #13545: For inference, I have the same problem. The client send jpg to server, then the server use cv2 to do resize. When put the image data into the mx.nd.array, it's very slow. And the Utilization of GPU is low too. How to solve it? Thank you! URL:

[GitHub] Joke09 commented on issue #13545: For inference, I have the same problem. The client send jpg to server, then the server use cv2 to do resize. When put the image data into the mx.nd.array, it

2018-12-05 Thread GitBox
Joke09 commented on issue #13545: For inference, I have the same problem. The client send jpg to server, then the server use cv2 to do resize. When put the image data into the mx.nd.array, it's very slow. And the Utilization of GPU is low too. How to solve it? Thank you! URL:

[GitHub] aaronmarkham commented on issue #13529: Improve dev_menu usability, local build and virtualenv

2018-12-05 Thread GitBox
aaronmarkham commented on issue #13529: Improve dev_menu usability, local build and virtualenv URL: https://github.com/apache/incubator-mxnet/pull/13529#issuecomment-444715306 The menu is nice. Is defaulting to CUDA=ON the preferred way? I thought OFF was the default with make right

[GitHub] gigasquid commented on issue #13523: #13441 [Clojure] Add Spec Validations for the Random namespace

2018-12-05 Thread GitBox
gigasquid commented on issue #13523: #13441 [Clojure] Add Spec Validations for the Random namespace URL: https://github.com/apache/incubator-mxnet/pull/13523#issuecomment-444709860 Thanks @hellonico for your work in improving this. I ran through the examples and everything looks good. It

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

2018-12-05 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 cd215ef Bump the publish

[GitHub] aaronmarkham commented on issue #13544: turn on Sphinx warnings as errors

2018-12-05 Thread GitBox
aaronmarkham commented on issue #13544: turn on Sphinx warnings as errors URL: https://github.com/apache/incubator-mxnet/pull/13544#issuecomment-444708562 This is time sensitive. As it ages, we'll have another round of cleaning up bugs that are introduced to get a clean start.

[GitHub] hellonico edited a comment on issue #13523: #13441 [Clojure] Add Spec Validations for the Random namespace

2018-12-05 Thread GitBox
hellonico edited a comment on issue #13523: #13441 [Clojure] Add Spec Validations for the Random namespace URL: https://github.com/apache/incubator-mxnet/pull/13523#issuecomment-444707535 > What do you think would be more user friendly? number? was a better choice, and is nicer to

[GitHub] hellonico commented on issue #13523: #13441 [Clojure] Add Spec Validations for the Random namespace

2018-12-05 Thread GitBox
hellonico commented on issue #13523: #13441 [Clojure] Add Spec Validations for the Random namespace URL: https://github.com/apache/incubator-mxnet/pull/13523#issuecomment-444707535 > What do you think would be more user friendly? number? was a better choice, and is nicer to the user.

[incubator-mxnet] branch master updated: Simplifications and some fun stuff for the MNIST Gluon tutorial (#13094)

2018-12-05 Thread thomasdelteil
This is an automated email from the ASF dual-hosted git repository. thomasdelteil 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 8bbac82 Simplifications and

[GitHub] ThomasDelteil closed pull request #13094: Simplifications and some fun stuff for the MNIST Gluon tutorial

2018-12-05 Thread GitBox
ThomasDelteil closed pull request #13094: Simplifications and some fun stuff for the MNIST Gluon tutorial URL: https://github.com/apache/incubator-mxnet/pull/13094 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] vandanavk removed a comment on issue #13553: [WIP] ONNX test code cleanup

2018-12-05 Thread GitBox
vandanavk removed a comment on issue #13553: [WIP] ONNX test code cleanup URL: https://github.com/apache/incubator-mxnet/pull/13553#issuecomment-444704079 @mxnet-label-bot add [ONNX, pr-work-in-progress] This is an automated

[GitHub] vandanavk commented on issue #13553: [WIP] ONNX test code cleanup

2018-12-05 Thread GitBox
vandanavk commented on issue #13553: [WIP] ONNX test code cleanup URL: https://github.com/apache/incubator-mxnet/pull/13553#issuecomment-444704079 @mxnet-label-bot add [ONNX, pr-work-in-progress] This is an automated message

[GitHub] vandanavk opened a new pull request #13553: [WIP] ONNX test code cleanup

2018-12-05 Thread GitBox
vandanavk opened a new pull request #13553: [WIP] ONNX test code cleanup URL: https://github.com/apache/incubator-mxnet/pull/13553 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items

[GitHub] roywei commented on a change in pull request #13534: add cpp example inception to nightly test

2018-12-05 Thread GitBox
roywei commented on a change in pull request #13534: add cpp example inception to nightly test URL: https://github.com/apache/incubator-mxnet/pull/13534#discussion_r239289338 ## File path: cpp-package/example/mlp.cpp ## @@ -144,7 +144,7 @@ void MLP() {

[GitHub] zhreshold commented on issue #13550: Fixes infinite loop using imagedetiter

2018-12-05 Thread GitBox
zhreshold commented on issue #13550: Fixes infinite loop using imagedetiter URL: https://github.com/apache/incubator-mxnet/pull/13550#issuecomment-444700527 LGTM, suggest to cherry-pick into v1.4.x as well. This is an

[GitHub] zachgk commented on a change in pull request #13493: [MXNET-1224]: improve scala maven jni build.

2018-12-05 Thread GitBox
zachgk commented on a change in pull request #13493: [MXNET-1224]: improve scala maven jni build. URL: https://github.com/apache/incubator-mxnet/pull/13493#discussion_r239286858 ## File path: scala-package/pom.xml ## @@ -39,6 +39,8 @@ 2.11.8 2.11 +g++

[GitHub] DickJC123 commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
DickJC123 commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444698761 @apeforest Re: bool return of Forward(), Backward(), etc. As you know, MXNet has been moving to a more imperative

[GitHub] ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239280157 ## File path: tests/python/gpu/test_operator_gpu.py ## @@ -608,6 +608,52 @@ def

[GitHub] ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239279756 ## File path: src/operator/nn/pooling.cu ## @@ -116,10 +118,12 @@ void

[GitHub] ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239277593 ## File path: python/mxnet/gluon/nn/conv_layers.py ## @@ -738,12 +741,13 @@ class

[GitHub] DickJC123 commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
DickJC123 commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444690325 @TaoLv , providing details and motivation of the PR (partial duplicate of info just added by @ptrendx ): In MXNet,

[GitHub] ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
ptrendx commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239277149 ## File path: src/operator/nn/cudnn/cudnn_pooling-inl.h ## @@ -165,55 +170,78 @@ class

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239276470 ## File path: cpp-package/example/README.md ## @@ -2,7

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239276259 ## File path: cpp-package/example/inference/README.md ##

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239275736 ## File path:

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239276098 ## File path:

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239275908 ## File path:

[GitHub] ptrendx commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
ptrendx commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444689260 @Vikas89 I took a stab at modifying PR description to include motivation and testing that we did.

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239276048 ## File path:

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239275884 ## File path:

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239275057 ## File path:

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239274844 ## File path:

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239274873 ## File path:

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239274909 ## File path:

[GitHub] leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API

2018-12-05 Thread GitBox
leleamol commented on a change in pull request #13294: [MXNET-1083] Add the example to demonstrate the inference workflow using C++ API URL: https://github.com/apache/incubator-mxnet/pull/13294#discussion_r239274813 ## File path: cpp-package/example/inference/README.md ##

[GitHub] sandeep-krishnamurthy commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
sandeep-krishnamurthy commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239271822 ## File path: src/operator/nn/pooling.cu ## @@ -116,10 +118,12 @@ void

[GitHub] sandeep-krishnamurthy commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
sandeep-krishnamurthy commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239272144 ## File path: tests/python/gpu/test_operator_gpu.py ## @@ -608,6 +608,52 @@

[GitHub] sandeep-krishnamurthy commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
sandeep-krishnamurthy commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239259258 ## File path: python/mxnet/gluon/nn/conv_layers.py ## @@ -738,12 +741,13 @@

[GitHub] apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r239265544 ## File path: src/operator/nn/cudnn/cudnn_pooling-inl.h ## @@ -165,55 +170,78 @@ class

[GitHub] ChaiBapchya commented on a change in pull request #13543: Chi_square_check for discrete distribution fix

2018-12-05 Thread GitBox
ChaiBapchya commented on a change in pull request #13543: Chi_square_check for discrete distribution fix URL: https://github.com/apache/incubator-mxnet/pull/13543#discussion_r239260946 ## File path: python/mxnet/test_utils.py ## @@ -1911,12 +1911,15 @@ def

[GitHub] ptrendx commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
ptrendx commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444670966 @apeforest @Vikas89 Not sure I understand the question. Are you asking why do we need NHWC support for pooling?

[GitHub] ptrendx commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

2018-12-05 Thread GitBox
ptrendx commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444669921 @TaoLv Re MKLDNN: Per title of this PR this is enabling NHWC support only for cuDNN path. I don't know whether MKLDNN

  1   2   3   >