[GitHub] ThomasDelteil commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build

2018-04-24 Thread GitBox
ThomasDelteil commented on a change in pull request #10485: [MXNET-304][RFC] Jenkins docs build URL: https://github.com/apache/incubator-mxnet/pull/10485#discussion_r183890449 ## File path: ci/docker/install/ubuntu_scala.sh ## @@ -1,4 +1,4 @@ -#!/bin/bash + #!/bin/bash

[incubator-mxnet] branch master updated: move parameters to backend for hybridblock (#10632)

2018-04-24 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie 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 faffe56 move parameters to backend for

[GitHub] piiswrong closed pull request #10632: move parameters to backend for hybridblock

2018-04-24 Thread GitBox
piiswrong closed pull request #10632: move parameters to backend for hybridblock URL: https://github.com/apache/incubator-mxnet/pull/10632 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] piiswrong closed pull request #10657: Fix typo in __setitem__ doc

2018-04-24 Thread GitBox
piiswrong closed pull request #10657: Fix typo in __setitem__ doc URL: https://github.com/apache/incubator-mxnet/pull/10657 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] piiswrong closed pull request #10607: New tutorial on how to create a new custom layer in Gluon

2018-04-24 Thread GitBox
piiswrong closed pull request #10607: New tutorial on how to create a new custom layer in Gluon URL: https://github.com/apache/incubator-mxnet/pull/10607 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

[incubator-mxnet] branch master updated: New tutorial on how to create a new custom layer in Gluon (#10607)

2018-04-24 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie 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 8011f3b New tutorial on how to create a

[GitHub] piiswrong closed pull request #10021: [MXNET-33] SSD example not working with mkl-dnn

2018-04-24 Thread GitBox
piiswrong closed pull request #10021: [MXNET-33] SSD example not working with mkl-dnn URL: https://github.com/apache/incubator-mxnet/pull/10021 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[incubator-mxnet] branch master updated: [MXNET-33] SSD example not working with mkl-dnn (#10021)

2018-04-24 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie 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 fd55a2e [MXNET-33] SSD example not

[GitHub] spidyDev commented on a change in pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
spidyDev commented on a change in pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#discussion_r183850165 ## File path: python/mxnet/contrib/onnx/_import/op_translations.py ## @@ -229,9 +258,38 @@ def deconv(attrs,

[GitHub] spidyDev commented on a change in pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
spidyDev commented on a change in pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#discussion_r183849902 ## File path: python/mxnet/contrib/onnx/_import/op_translations.py ## @@ -214,9 +214,38 @@ def conv(attrs,

[GitHub] anirudhacharya closed pull request #10662: [MXNET-347] Logical AND operator

2018-04-24 Thread GitBox
anirudhacharya closed pull request #10662: [MXNET-347] Logical AND operator URL: https://github.com/apache/incubator-mxnet/pull/10662 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] zheng-da commented on a change in pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
zheng-da commented on a change in pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#discussion_r183858705 ## File path: python/mxnet/contrib/onnx/_import/op_translations.py ## @@ -214,9 +214,25 @@ def conv(attrs,

[GitHub] zheng-da commented on issue #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
zheng-da commented on issue #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#issuecomment-384060380 Is there a unit test to cover this? This is an automated message from the Apache

[GitHub] anirudh2290 commented on issue #10569: [MXNET-324] Use default nvcc when configured nvcc not present.

2018-04-24 Thread GitBox
anirudh2290 commented on issue #10569: [MXNET-324] Use default nvcc when configured nvcc not present. URL: https://github.com/apache/incubator-mxnet/pull/10569#issuecomment-384080306 @piiswrong @marcoabreu can this be merged ?

[GitHub] anirudh2290 commented on issue #10566: [MXNET-324] Use default nvcc when configured nvcc not present.

2018-04-24 Thread GitBox
anirudh2290 commented on issue #10566: [MXNET-324] Use default nvcc when configured nvcc not present. URL: https://github.com/apache/incubator-mxnet/pull/10566#issuecomment-384080239 @piiswrong @marcoabreu can this be merged ?

[GitHub] anirudh2290 commented on issue #10661: mark MKLDNN experimantal.

2018-04-24 Thread GitBox
anirudh2290 commented on issue #10661: mark MKLDNN experimantal. URL: https://github.com/apache/incubator-mxnet/pull/10661#issuecomment-384067467 ONNX in release notes is updated as part of #10677 This is an automated

[GitHub] zheng-da opened a new pull request #10678: invalidate output arrays for imperative.

2018-04-24 Thread GitBox
zheng-da opened a new pull request #10678: invalidate output arrays for imperative. URL: https://github.com/apache/incubator-mxnet/pull/10678 ## Description ## we need to invalidate the MKLDNN arrays when they are reused as output arrays. We have done it in the symbolic executor. We

[GitHub] eric-haibin-lin commented on issue #10666: [MXNET-359] fix checks on convolution parameters in MKLDNN.

2018-04-24 Thread GitBox
eric-haibin-lin commented on issue #10666: [MXNET-359] fix checks on convolution parameters in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/10666#issuecomment-384083892 Do we still want this change if onnx correctly handles padding?

[GitHub] anirudh2290 commented on issue #10666: [MXNET-359] fix checks on convolution parameters in MKLDNN.

2018-04-24 Thread GitBox
anirudh2290 commented on issue #10666: [MXNET-359] fix checks on convolution parameters in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/10666#issuecomment-384085337 @eric-haibin-lin yes we still need this change to make the behavior consistent (with or without MKLDNN)

[GitHub] anirudhacharya commented on a change in pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
anirudhacharya commented on a change in pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#discussion_r183891749 ## File path: python/mxnet/contrib/onnx/_import/op_translations.py ## @@ -214,9 +214,25 @@ def

[GitHub] anirudhacharya commented on a change in pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
anirudhacharya commented on a change in pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#discussion_r183891749 ## File path: python/mxnet/contrib/onnx/_import/op_translations.py ## @@ -214,9 +214,25 @@ def

[GitHub] zheng-da commented on a change in pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
zheng-da commented on a change in pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#discussion_r183860494 ## File path: python/mxnet/contrib/onnx/_import/op_translations.py ## @@ -229,9 +245,25 @@ def deconv(attrs,

[GitHub] anirudhacharya commented on a change in pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
anirudhacharya commented on a change in pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#discussion_r183895796 ## File path: python/mxnet/contrib/onnx/_import/op_translations.py ## @@ -229,9 +245,25 @@ def

[GitHub] anirudh2290 commented on a change in pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
anirudh2290 commented on a change in pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#discussion_r183895852 ## File path: python/mxnet/contrib/onnx/_import/op_translations.py ## @@ -214,9 +214,25 @@ def conv(attrs,

[GitHub] anirudhacharya commented on issue #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
anirudhacharya commented on issue #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#issuecomment-384096721 The CI failed in one of the flaky tests while trying to fetch test files - ProtocolError: ("Connection broken: ConnectionResetError(104,

[GitHub] anirudh2290 commented on a change in pull request #10677: Mark ONNX-MXNet experimental

2018-04-24 Thread GitBox
anirudh2290 commented on a change in pull request #10677: Mark ONNX-MXNet experimental URL: https://github.com/apache/incubator-mxnet/pull/10677#discussion_r183868344 ## File path: NEWS.md ## @@ -5,7 +5,7 @@ MXNet Change Log - Implemented new [Scala Inference

[GitHub] anirudh2290 commented on a change in pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
anirudh2290 commented on a change in pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#discussion_r183877365 ## File path: tests/python-pytest/onnx/onnx_test.py ## @@ -124,12 +124,13 @@ def

[GitHub] anirudhacharya commented on a change in pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
anirudhacharya commented on a change in pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#discussion_r183883862 ## File path: tests/python-pytest/onnx/onnx_test.py ## @@ -124,12 +124,13 @@ def

[GitHub] yajiedesign commented on issue #10629: [MXNET-343]fix Mkldnn with msvc

2018-04-24 Thread GitBox
yajiedesign commented on issue #10629: [MXNET-343]fix Mkldnn with msvc URL: https://github.com/apache/incubator-mxnet/pull/10629#issuecomment-384090424 can we merge? This is an automated message from the Apache Git Service.

[GitHub] ashokei closed pull request #10670: enable jenkins for mkldnn unittest

2018-04-24 Thread GitBox
ashokei closed pull request #10670: enable jenkins for mkldnn unittest URL: https://github.com/apache/incubator-mxnet/pull/10670 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] anirudh2290 commented on issue #10661: mark MKLDNN experimantal.

2018-04-24 Thread GitBox
anirudh2290 commented on issue #10661: mark MKLDNN experimantal. URL: https://github.com/apache/incubator-mxnet/pull/10661#issuecomment-384027516 @zheng-da can you also add the same line at the end of ONNX description. cc @anirudhacharya

[GitHub] anirudhacharya opened a new pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
anirudhacharya opened a new pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable

[GitHub] spidyDev commented on a change in pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
spidyDev commented on a change in pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#discussion_r183850413 ## File path: tests/python-pytest/onnx/onnx_test.py ## @@ -254,4 +255,4 @@ def test_bvlc_rcnn_ilsvrc13():

[GitHub] anirudhacharya commented on a change in pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
anirudhacharya commented on a change in pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#discussion_r183850592 ## File path: tests/python-pytest/onnx/onnx_test.py ## @@ -254,4 +255,4 @@ def

[GitHub] haojin2 commented on a change in pull request #10633: [MXNET-346] Hard Sigmoid Operator

2018-04-24 Thread GitBox
haojin2 commented on a change in pull request #10633: [MXNET-346] Hard Sigmoid Operator URL: https://github.com/apache/incubator-mxnet/pull/10633#discussion_r183846508 ## File path: src/operator/tensor/elemwise_unary_op.h ## @@ -388,6 +388,94 @@ void CastCompute(const

[GitHub] piiswrong commented on a change in pull request #10461: allow user to define unknown token symbol

2018-04-24 Thread GitBox
piiswrong commented on a change in pull request #10461: allow user to define unknown token symbol URL: https://github.com/apache/incubator-mxnet/pull/10461#discussion_r183823838 ## File path: python/mxnet/rnn/io.py ## @@ -58,6 +62,8 @@ def encode_sentences(sentences,

[GitHub] cells2numbers opened a new issue #10675: R Installation on MacOS fails (file not found)

2018-04-24 Thread GitBox
cells2numbers opened a new issue #10675: R Installation on MacOS fails (file not found) URL: https://github.com/apache/incubator-mxnet/issues/10675 The installation of mxnet fails on MacOS using RStudio because the file mxnet_0.10.1.tar.gz is not found. Probably the file is outdated.

[GitHub] zheng-da commented on issue #10666: [MXNET-359] fix checks on convolution parameters in MKLDNN.

2018-04-24 Thread GitBox
zheng-da commented on issue #10666: [MXNET-359] fix checks on convolution parameters in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/10666#issuecomment-384045260 @piiswrong @eric-haibin-lin @ashokei @pengzhao-intel @TaoLv Could you help review the code and merge it, so

[GitHub] anirudhacharya opened a new pull request #10677: Mark ONNX-MXNet experimental

2018-04-24 Thread GitBox
anirudhacharya opened a new pull request #10677: Mark ONNX-MXNet experimental URL: https://github.com/apache/incubator-mxnet/pull/10677 ## Description ## mark onnx-mxnet as experimental in release notes. ## Checklist ## ### Essentials ### Please feel free to remove

[incubator-mxnet] branch master updated: Fix typo in __setitem__ doc (#10657)

2018-04-24 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie 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 a8e1912 Fix typo in __setitem__ doc

[GitHub] piiswrong commented on issue #10354: Expose the number of GPUs.

2018-04-24 Thread GitBox
piiswrong commented on issue #10354: Expose the number of GPUs. URL: https://github.com/apache/incubator-mxnet/pull/10354#issuecomment-384019960 they are run with both. gpu/* tests are run with only gpu This is an automated

[GitHub] haojin2 commented on a change in pull request #10633: [MXNET-346] Hard Sigmoid Operator

2018-04-24 Thread GitBox
haojin2 commented on a change in pull request #10633: [MXNET-346] Hard Sigmoid Operator URL: https://github.com/apache/incubator-mxnet/pull/10633#discussion_r183825064 ## File path: src/operator/tensor/elemwise_unary_op.h ## @@ -388,6 +388,94 @@ void CastCompute(const

[GitHub] haojin2 commented on a change in pull request #10633: [MXNET-346] Hard Sigmoid Operator

2018-04-24 Thread GitBox
haojin2 commented on a change in pull request #10633: [MXNET-346] Hard Sigmoid Operator URL: https://github.com/apache/incubator-mxnet/pull/10633#discussion_r183846508 ## File path: src/operator/tensor/elemwise_unary_op.h ## @@ -388,6 +388,94 @@ void CastCompute(const

[GitHub] spidyDev commented on a change in pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
spidyDev commented on a change in pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#discussion_r183849902 ## File path: python/mxnet/contrib/onnx/_import/op_translations.py ## @@ -214,9 +214,38 @@ def conv(attrs,

[incubator-mxnet] branch master updated: attrs key should be 'attrs' not 'attr' (#10671)

2018-04-24 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie 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 b847c6f attrs key should be 'attrs' not

[GitHub] piiswrong closed pull request #10671: attrs key should be 'attrs' not 'attr'

2018-04-24 Thread GitBox
piiswrong closed pull request #10671: attrs key should be 'attrs' not 'attr' URL: https://github.com/apache/incubator-mxnet/pull/10671 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] anirudh2290 commented on a change in pull request #10666: [MXNET-359] fix checks on convolution parameters in MKLDNN.

2018-04-24 Thread GitBox
anirudh2290 commented on a change in pull request #10666: [MXNET-359] fix checks on convolution parameters in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/10666#discussion_r183821568 ## File path: src/operator/nn/mkldnn/mkldnn_convolution.cc ## @@ -88,26

[GitHub] piiswrong commented on a change in pull request #10633: [MXNET-346] Hard Sigmoid Operator

2018-04-24 Thread GitBox
piiswrong commented on a change in pull request #10633: [MXNET-346] Hard Sigmoid Operator URL: https://github.com/apache/incubator-mxnet/pull/10633#discussion_r183824195 ## File path: src/operator/tensor/elemwise_unary_op.h ## @@ -388,6 +388,94 @@ void CastCompute(const

[GitHub] anirudh2290 commented on a change in pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
anirudh2290 commented on a change in pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#discussion_r183832591 ## File path: python/mxnet/contrib/onnx/_import/op_translations.py ## @@ -214,10 +214,14 @@ def conv(attrs,

[GitHub] piiswrong commented on a change in pull request #10633: [MXNET-346] Hard Sigmoid Operator

2018-04-24 Thread GitBox
piiswrong commented on a change in pull request #10633: [MXNET-346] Hard Sigmoid Operator URL: https://github.com/apache/incubator-mxnet/pull/10633#discussion_r183826137 ## File path: src/operator/tensor/elemwise_unary_op.h ## @@ -388,6 +388,94 @@ void CastCompute(const

[GitHub] anirudh2290 commented on a change in pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
anirudh2290 commented on a change in pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#discussion_r183832404 ## File path: python/mxnet/contrib/onnx/_import/op_translations.py ## @@ -214,10 +214,14 @@ def conv(attrs,

[GitHub] szha commented on issue #10630: make the slice of sequential a sequential

2018-04-24 Thread GitBox
szha commented on issue #10630: make the slice of sequential a sequential URL: https://github.com/apache/incubator-mxnet/pull/10630#issuecomment-384050775 @piiswrong This is an automated message from the Apache Git Service.

[GitHub] haojin2 commented on issue #10371: [MXNET-263] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-24 Thread GitBox
haojin2 commented on issue #10371: [MXNET-263] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#issuecomment-384114344 benchmark script: `import mxnet as mx import numpy as np from mxnet.test_utils import

[GitHub] lanking520 commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design

2018-04-24 Thread GitBox
lanking520 commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design URL: https://github.com/apache/incubator-mxnet/pull/10660#discussion_r183911000 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/SymbolMacro.scala ## @@

[GitHub] haojin2 commented on issue #10371: [MXNET-263] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-24 Thread GitBox
haojin2 commented on issue #10371: [MXNET-263] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#issuecomment-384114344 benchmark script: `import mxnet as mx import numpy as np from mxnet.test_utils import

[GitHub] lanking520 commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design

2018-04-24 Thread GitBox
lanking520 commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design URL: https://github.com/apache/incubator-mxnet/pull/10660#discussion_r183911074 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/SymbolMacro.scala ## @@

[GitHub] anirudh2290 commented on issue #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
anirudh2290 commented on issue #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#issuecomment-384120028 Okay it seems to be set in test_tutorials to 7 mins:

[GitHub] anirudhacharya commented on issue #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
anirudhacharya commented on issue #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#issuecomment-384119781 @marcoabreu the last two CI runs failed on the following tests - - test_tutorials.test_onnx_inference_on_onnx_model (tutorial tests

[GitHub] haojin2 commented on a change in pull request #10679: [MXNET-347] Logical Operators AND, XOR, OR

2018-04-24 Thread GitBox
haojin2 commented on a change in pull request #10679: [MXNET-347] Logical Operators AND, XOR, OR URL: https://github.com/apache/incubator-mxnet/pull/10679#discussion_r183918262 ## File path: src/operator/tensor/elemwise_binary_broadcast_op_logic.cc ## @@ -137,5 +137,59 @@

[GitHub] haojin2 commented on a change in pull request #10679: [MXNET-347] Logical Operators AND, XOR, OR

2018-04-24 Thread GitBox
haojin2 commented on a change in pull request #10679: [MXNET-347] Logical Operators AND, XOR, OR URL: https://github.com/apache/incubator-mxnet/pull/10679#discussion_r183918304 ## File path: src/operator/tensor/elemwise_binary_broadcast_op_logic.cc ## @@ -137,5 +137,59 @@

[GitHub] lanking520 commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design

2018-04-24 Thread GitBox
lanking520 commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design URL: https://github.com/apache/incubator-mxnet/pull/10660#discussion_r183924736 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/SymbolMacro.scala ## @@

[GitHub] TaoLv commented on a change in pull request #10666: [MXNET-359] fix checks on convolution parameters in MKLDNN.

2018-04-24 Thread GitBox
TaoLv commented on a change in pull request #10666: [MXNET-359] fix checks on convolution parameters in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/10666#discussion_r183926258 ## File path: src/operator/nn/mkldnn/mkldnn_deconvolution.cc ## @@ -67,31 +73,18

[GitHub] haojin2 commented on issue #10371: [MXNET-263] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-24 Thread GitBox
haojin2 commented on issue #10371: [MXNET-263] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#issuecomment-384114091 Benchmark results: [density of rhs csr(%)] [speedup rate] - dot(dense, csr): - without

[GitHub] haojin2 commented on issue #10371: [MXNET-263] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-24 Thread GitBox
haojin2 commented on issue #10371: [MXNET-263] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#issuecomment-384114091 Benchmark results: (density(%) speedup rate) - dot(dense, csr): - without fallback time:

[GitHub] yzhliu commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design

2018-04-24 Thread GitBox
yzhliu commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design URL: https://github.com/apache/incubator-mxnet/pull/10660#discussion_r183908931 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/SymbolMacro.scala ## @@ -48,67

[GitHub] yzhliu commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design

2018-04-24 Thread GitBox
yzhliu commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design URL: https://github.com/apache/incubator-mxnet/pull/10660#discussion_r183910476 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/SymbolMacro.scala ## @@ -136,20

[GitHub] stupidZZ closed issue #10672: Bug at L304: Gluon/block.py, v1.2.0

2018-04-24 Thread GitBox
stupidZZ closed issue #10672: Bug at L304: Gluon/block.py, v1.2.0 URL: https://github.com/apache/incubator-mxnet/issues/10672 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] yzhliu commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design

2018-04-24 Thread GitBox
yzhliu commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design URL: https://github.com/apache/incubator-mxnet/pull/10660#discussion_r183910073 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/SymbolMacro.scala ## @@ -136,20

[GitHub] yzhliu commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design

2018-04-24 Thread GitBox
yzhliu commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design URL: https://github.com/apache/incubator-mxnet/pull/10660#discussion_r183910178 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/SymbolMacro.scala ## @@ -136,20

[GitHub] yzhliu commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design

2018-04-24 Thread GitBox
yzhliu commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design URL: https://github.com/apache/incubator-mxnet/pull/10660#discussion_r183909723 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/SymbolMacro.scala ## @@ -174,6

[GitHub] yzhliu commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design

2018-04-24 Thread GitBox
yzhliu commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design URL: https://github.com/apache/incubator-mxnet/pull/10660#discussion_r183909390 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/SymbolMacro.scala ## @@ -136,20

[GitHub] anirudh2290 commented on issue #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
anirudh2290 commented on issue #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#issuecomment-384119139 @marcoabreu I see that tutorials test have been added to the CI. How can I find the timeout set for the tutorials job ?

[GitHub] zheng-da commented on issue #10666: [MXNET-359] fix checks on convolution parameters in MKLDNN.

2018-04-24 Thread GitBox
zheng-da commented on issue #10666: [MXNET-359] fix checks on convolution parameters in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/10666#issuecomment-384140628 I’m not sure about other front ends. What I see is mxnet conv operator always assumes two-elements tuple in

[GitHub] nicklhy commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0

2018-04-24 Thread GitBox
nicklhy commented on issue #9396: inference speed drop after updating mxnet from 0.10.0 to 1.0.0 URL: https://github.com/apache/incubator-mxnet/issues/9396#issuecomment-384140555 @eric-haibin-lin @DickJC123 , any update about this ?

[GitHub] CodingCat commented on a change in pull request #10462: [MXNET-62] add test against spark integration

2018-04-24 Thread GitBox
CodingCat commented on a change in pull request #10462: [MXNET-62] add test against spark integration URL: https://github.com/apache/incubator-mxnet/pull/10462#discussion_r183938670 ## File path: scala-package/spark/src/test/scala/org/apache/mxnet/spark/MXNetGeneralSuite.scala

[GitHub] anirudhacharya commented on a change in pull request #10676: fix for asymmetric padding

2018-04-24 Thread GitBox
anirudhacharya commented on a change in pull request #10676: fix for asymmetric padding URL: https://github.com/apache/incubator-mxnet/pull/10676#discussion_r183897755 ## File path: python/mxnet/contrib/onnx/_import/op_translations.py ## @@ -214,9 +214,25 @@ def

[GitHub] anirudhacharya opened a new pull request #10679: [MXNET-347] Logical Operators AND, XOR, OR

2018-04-24 Thread GitBox
anirudhacharya opened a new pull request #10679: [MXNET-347] Logical Operators AND, XOR, OR URL: https://github.com/apache/incubator-mxnet/pull/10679 ## Description ## Adds Logical AND, XOR, OR operator. ## Checklist ## ### Essentials ### Please feel free to remove

[GitHub] anirudhacharya commented on issue #10662: [MXNET-347] Logical AND operator

2018-04-24 Thread GitBox
anirudhacharya commented on issue #10662: [MXNET-347] Logical AND operator URL: https://github.com/apache/incubator-mxnet/pull/10662#issuecomment-384122307 part of this PR - https://github.com/apache/incubator-mxnet/pull/10679

[GitHub] rasefon opened a new pull request #10680: Update CONTRIBUTORS.md

2018-04-24 Thread GitBox
rasefon opened a new pull request #10680: Update CONTRIBUTORS.md URL: https://github.com/apache/incubator-mxnet/pull/10680 #10671 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable

[GitHub] rasefon commented on issue #10680: Update CONTRIBUTORS.md

2018-04-24 Thread GitBox
rasefon commented on issue #10680: Update CONTRIBUTORS.md URL: https://github.com/apache/incubator-mxnet/pull/10680#issuecomment-384139177 #10671 This is an automated message from the Apache Git Service. To respond to the

[GitHub] yzhliu commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design

2018-04-24 Thread GitBox
yzhliu commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design URL: https://github.com/apache/incubator-mxnet/pull/10660#discussion_r183919376 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/SymbolMacro.scala ## @@ -48,67

[GitHub] haojin2 commented on issue #10371: [MXNET-263] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU

2018-04-24 Thread GitBox
haojin2 commented on issue #10371: [MXNET-263] Support for dot(dns, csr) = dns and dot(dns, csr.T) = dns on GPU URL: https://github.com/apache/incubator-mxnet/pull/10371#issuecomment-384114344 benchmark script: ```python import mxnet as mx import numpy as np from

[GitHub] piiswrong closed issue #10226: [DOC] ImageRecordIter doesn't render correctly

2018-04-24 Thread GitBox
piiswrong closed issue #10226: [DOC] ImageRecordIter doesn't render correctly URL: https://github.com/apache/incubator-mxnet/issues/10226 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] piiswrong closed issue #10238: API Doc not refreshed on website

2018-04-24 Thread GitBox
piiswrong closed issue #10238: API Doc not refreshed on website URL: https://github.com/apache/incubator-mxnet/issues/10238 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] piiswrong closed issue #10273: Rendering problem of documentations

2018-04-24 Thread GitBox
piiswrong closed issue #10273: Rendering problem of documentations URL: https://github.com/apache/incubator-mxnet/issues/10273 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] haojin2 commented on a change in pull request #10679: [MXNET-347] Logical Operators AND, XOR, OR

2018-04-24 Thread GitBox
haojin2 commented on a change in pull request #10679: [MXNET-347] Logical Operators AND, XOR, OR URL: https://github.com/apache/incubator-mxnet/pull/10679#discussion_r183918109 ## File path: src/operator/tensor/elemwise_binary_broadcast_op_logic.cc ## @@ -137,5 +137,59 @@

[GitHub] anirudh2290 commented on issue #10677: Mark ONNX-MXNet experimental

2018-04-24 Thread GitBox
anirudh2290 commented on issue #10677: Mark ONNX-MXNet experimental URL: https://github.com/apache/incubator-mxnet/pull/10677#issuecomment-384123436 @eric-haibin-lin can this be merged ? This is an automated message from the

[GitHub] haojin2 commented on a change in pull request #10679: [MXNET-347] Logical Operators AND, XOR, OR

2018-04-24 Thread GitBox
haojin2 commented on a change in pull request #10679: [MXNET-347] Logical Operators AND, XOR, OR URL: https://github.com/apache/incubator-mxnet/pull/10679#discussion_r183918078 ## File path: src/operator/tensor/elemwise_binary_broadcast_op_logic.cc ## @@ -137,5 +137,59 @@

[GitHub] anirudh2290 commented on issue #10661: mark MKLDNN experimantal.

2018-04-24 Thread GitBox
anirudh2290 commented on issue #10661: mark MKLDNN experimantal. URL: https://github.com/apache/incubator-mxnet/pull/10661#issuecomment-384123490 @eric-haibin-lin can this be merged ? This is an automated message from the

[GitHub] reminisce commented on issue #10673: Segmentation Fault when using large mx.nd.array

2018-04-24 Thread GitBox
reminisce commented on issue #10673: Segmentation Fault when using large mx.nd.array URL: https://github.com/apache/incubator-mxnet/issues/10673#issuecomment-384143913 The advanced indexing I meant is indexing an NDArray with indices containing lists, np.ndarray, NDArray etc., instead of

[GitHub] yzhliu commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design

2018-04-24 Thread GitBox
yzhliu commented on a change in pull request #10660: [MXNET-357][WIP] New Scala API Design URL: https://github.com/apache/incubator-mxnet/pull/10660#discussion_r183920098 ## File path: scala-package/macros/src/main/scala/org/apache/mxnet/SymbolMacro.scala ## @@ -136,20

[GitHub] anirudhacharya commented on issue #10663: mismatch of padding in conv between MXNet and ONNX

2018-04-24 Thread GitBox
anirudhacharya commented on issue #10663: mismatch of padding in conv between MXNet and ONNX URL: https://github.com/apache/incubator-mxnet/issues/10663#issuecomment-384131511 fix - https://github.com/apache/incubator-mxnet/pull/10676

[GitHub] casscw closed issue #10648: Why examlpe wide_deep runs faster on win10(cpu) than linux(gpu-Tesla M40 24GB)

2018-04-24 Thread GitBox
casscw closed issue #10648: Why examlpe wide_deep runs faster on win10(cpu) than linux(gpu-Tesla M40 24GB) URL: https://github.com/apache/incubator-mxnet/issues/10648 This is an automated message from the Apache Git

[GitHub] casscw commented on issue #10648: Why examlpe wide_deep runs faster on win10(cpu) than linux(gpu-Tesla M40 24GB)

2018-04-24 Thread GitBox
casscw commented on issue #10648: Why examlpe wide_deep runs faster on win10(cpu) than linux(gpu-Tesla M40 24GB) URL: https://github.com/apache/incubator-mxnet/issues/10648#issuecomment-384141491 thanks This is an automated

[GitHub] DuinoDu commented on issue #9547: Implementing new operators in an external module

2018-04-24 Thread GitBox
DuinoDu commented on issue #9547: Implementing new operators in an external module URL: https://github.com/apache/incubator-mxnet/issues/9547#issuecomment-384144528 https://github.com/DuinoDu/load_op.mxnet This is an

[GitHub] Edmospheric-pressure commented on issue #9989: Cannot train example gluon style transfer

2018-04-24 Thread GitBox
Edmospheric-pressure commented on issue #9989: Cannot train example gluon style transfer URL: https://github.com/apache/incubator-mxnet/issues/9989#issuecomment-384153061 May I ask if this has been solved yet? I have encountered the same error when I want to train the network.

[GitHub] ThomasDelteil opened a new issue #10669: Bug: failed to load some images with mx.image.imread()

2018-04-24 Thread GitBox
ThomasDelteil opened a new issue #10669: Bug: failed to load some images with mx.image.imread() URL: https://github.com/apache/incubator-mxnet/issues/10669 I have a strange bug when trying to load an image (a few in 1M dataset) : `mx.image.imread('test_image.jpg')` Download this

[GitHub] ThomasDelteil commented on issue #10669: Bug: failed to load some images with mx.image.imread()

2018-04-24 Thread GitBox
ThomasDelteil commented on issue #10669: Bug: failed to load some images with mx.image.imread() URL: https://github.com/apache/incubator-mxnet/issues/10669#issuecomment-383815135 hmm `cat test_image.jpg` ``` GIF89a~ooo��٢'''�\\\:::KKK��⡡�y

[GitHub] ThomasDelteil closed issue #10669: Bug: failed to load some images with mx.image.imread()

2018-04-24 Thread GitBox
ThomasDelteil closed issue #10669: Bug: failed to load some images with mx.image.imread() URL: https://github.com/apache/incubator-mxnet/issues/10669 This is an automated message from the Apache Git Service. To respond to

[GitHub] ashokei closed pull request #10670: enable jenkins for mkldnn unittest

2018-04-24 Thread GitBox
ashokei closed pull request #10670: enable jenkins for mkldnn unittest URL: https://github.com/apache/incubator-mxnet/pull/10670 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

  1   2   >