[GitHub] [incubator-mxnet] ZhennanQin commented on issue #15926: Add disable attr to subgraph property

2019-08-15 Thread GitBox
ZhennanQin commented on issue #15926: Add disable attr to subgraph property URL: https://github.com/apache/incubator-mxnet/pull/15926#issuecomment-521907756 @pengzhao-intel Not for now. crash is reported @wuxun-zhang . This variable isn't commonly used, so external user probably don't have

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #15926: Add disable attr to subgraph property

2019-08-15 Thread GitBox
pengzhao-intel commented on issue #15926: Add disable attr to subgraph property URL: https://github.com/apache/incubator-mxnet/pull/15926#issuecomment-521906365 Is there an issue to track the crash? This is an automated messag

[GitHub] [incubator-mxnet] ZhennanQin commented on issue #15886: [WIP] Graph Partition API

2019-08-15 Thread GitBox
ZhennanQin commented on issue #15886: [WIP] Graph Partition API URL: https://github.com/apache/incubator-mxnet/pull/15886#issuecomment-521905123 @samskalicky Thanks for the explanation, it helps a lot to understand this PR. Basically, I think `sym.optimizeFor('MKLDNN')` is the same as curre

[incubator-mxnet] branch master updated: [Opperf] Filter out deprecated ops (#15541)

2019-08-15 Thread apeforest
This is an automated email from the ASF dual-hosted git repository. apeforest 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 5a4c01b [Opperf] Filter out depreca

[incubator-mxnet] branch master updated (c8f0b21 -> 09cf75b)

2019-08-15 Thread apeforest
This is an automated email from the ASF dual-hosted git repository. apeforest pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from c8f0b21 Remove Scala package test on build (#15915) add 09cf75b Add power, exponent, log ops large

[GitHub] [incubator-mxnet] apeforest merged pull request #15541: [Opperf] Filter out deprecated ops

2019-08-15 Thread GitBox
apeforest merged pull request #15541: [Opperf] Filter out deprecated ops URL: https://github.com/apache/incubator-mxnet/pull/15541 This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [incubator-mxnet] apeforest merged pull request #15794: Add power, exponent, log ops large tensor support

2019-08-15 Thread GitBox
apeforest merged pull request #15794: Add power, exponent, log ops large tensor support URL: https://github.com/apache/incubator-mxnet/pull/15794 This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #15769: Benchmark doc fix

2019-08-15 Thread GitBox
apeforest commented on a change in pull request #15769: Benchmark doc fix URL: https://github.com/apache/incubator-mxnet/pull/15769#discussion_r314600024 ## File path: benchmark/opperf/README.md ## @@ -47,7 +47,10 @@ Hence, in this utility, we will build the functionality t

[GitHub] [incubator-mxnet] samskalicky commented on a change in pull request #15886: [WIP] Graph Partition API

2019-08-15 Thread GitBox
samskalicky commented on a change in pull request #15886: [WIP] Graph Partition API URL: https://github.com/apache/incubator-mxnet/pull/15886#discussion_r314598961 ## File path: src/operator/subgraph/subgraph_property.h ## @@ -221,6 +221,10 @@ class SubgraphProperty {

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #14940: Fix warnings

2019-08-15 Thread GitBox
apeforest commented on a change in pull request #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#discussion_r314596452 ## File path: src/operator/nn/mkldnn/mkldnn_fully_connected.cc ## @@ -26,6 +26,7 @@ #if MXNET_USE_MKLDNN == 1 #include "m

[GitHub] [incubator-mxnet] larroy closed pull request #14940: Fix warnings

2019-08-15 Thread GitBox
larroy closed pull request #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

[GitHub] [incubator-mxnet] larroy commented on issue #14940: Fix warnings

2019-08-15 Thread GitBox
larroy commented on issue #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#issuecomment-521899781 I don't find this way to review code is conductive to contributing to this project. This is an au

[GitHub] [incubator-mxnet] larroy edited a comment on issue #14940: Fix warnings

2019-08-15 Thread GitBox
larroy edited a comment on issue #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#issuecomment-521899079 So the request for changes is basically removal of two additional blank lines (which by the way are allowed by Google style guide and Pep8) this has a CI c

[GitHub] [incubator-mxnet] larroy edited a comment on issue #14940: Fix warnings

2019-08-15 Thread GitBox
larroy edited a comment on issue #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#issuecomment-521899079 So the request for changes is basically removal of two additional blank lines (which by the way are allowed by Google style guide and Pep8) this has a CI c

[GitHub] [incubator-mxnet] ThomasDelteil edited a comment on issue #15884: [WIP] New Website: New Docs [1/3]

2019-08-15 Thread GitBox
ThomasDelteil edited a comment on issue #15884: [WIP] New Website: New Docs [1/3] URL: https://github.com/apache/incubator-mxnet/pull/15884#issuecomment-521898868 Hey @pengzhao-intel, thanks for reaching out! :) We are trying to get this merged as soon as possible, aiming for 31st of Au

[GitHub] [incubator-mxnet] larroy commented on issue #14940: Fix warnings

2019-08-15 Thread GitBox
larroy commented on issue #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#issuecomment-521899079 So the request for changes is basically for two additional blank lines (which by the way are allowed by Google style guide and Pep8) this has a CI cost which I d

[GitHub] [incubator-mxnet] larroy edited a comment on issue #14940: Fix warnings

2019-08-15 Thread GitBox
larroy edited a comment on issue #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#issuecomment-521899079 So the request for changes is basically removal of two additional blank lines (which by the way are allowed by Google style guide and Pep8) this has a CI c

[GitHub] [incubator-mxnet] ThomasDelteil commented on issue #15884: [WIP] New Website: New Docs [1/3]

2019-08-15 Thread GitBox
ThomasDelteil commented on issue #15884: [WIP] New Website: New Docs [1/3] URL: https://github.com/apache/incubator-mxnet/pull/15884#issuecomment-521898868 Hey @pengzhao-intel, thanks for reaching out! :) We are trying to get this merged as soon as possible, aiming for 31st of August lau

[GitHub] [incubator-mxnet] CNApocalypse commented on issue #15721: Regression w/ Keras and MxNet 1.5.0: Error in operator batchnorm... Check failed: i >= 0 && i < ndim(): index = 1 must be in range [0

2019-08-15 Thread GitBox
CNApocalypse commented on issue #15721: Regression w/ Keras and MxNet 1.5.0: Error in operator batchnorm... Check failed: i >= 0 && i < ndim(): index = 1 must be in range [0, -1) URL: https://github.com/apache/incubator-mxnet/issues/15721#issuecomment-521898003 > @mxnet-label-bot I have

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #14940: Fix warnings

2019-08-15 Thread GitBox
ChaiBapchya commented on a change in pull request #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#discussion_r314593066 ## File path: src/operator/nn/mkldnn/mkldnn_fully_connected.cc ## @@ -26,6 +26,7 @@ #if MXNET_USE_MKLDNN == 1 #include

[GitHub] [incubator-mxnet] apeforest commented on issue #14940: Fix warnings

2019-08-15 Thread GitBox
apeforest commented on issue #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#issuecomment-521896113 It seems moving the inlining functions to a .cc file is not related to fixing warnings (or please correct me if it does). Please move it to a seperate PR if th

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #14940: Fix warnings

2019-08-15 Thread GitBox
larroy commented on a change in pull request #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#discussion_r314592583 ## File path: src/operator/nn/mkldnn/mkldnn_fully_connected.cc ## @@ -26,6 +26,7 @@ #if MXNET_USE_MKLDNN == 1 #include "mkld

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #14940: Fix warnings

2019-08-15 Thread GitBox
larroy commented on a change in pull request #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#discussion_r314592384 ## File path: include/mxnet/executor.h ## @@ -28,6 +28,8 @@ #include #include +#include Review comment: In the file

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #14940: Fix warnings

2019-08-15 Thread GitBox
larroy commented on a change in pull request #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#discussion_r314592497 ## File path: include/mxnet/executor.h ## @@ -28,6 +28,8 @@ #include #include +#include Review comment: Fixes a lin

[GitHub] [incubator-mxnet] tingying2020 commented on a change in pull request #15890: [Numpy] operator arctan2

2019-08-15 Thread GitBox
tingying2020 commented on a change in pull request #15890: [Numpy] operator arctan2 URL: https://github.com/apache/incubator-mxnet/pull/15890#discussion_r314592278 ## File path: tests/python/gpu/test_operator_gpu.py ## @@ -2331,6 +2331,82 @@ def test_math(): f

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #14940: Fix warnings

2019-08-15 Thread GitBox
larroy commented on a change in pull request #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#discussion_r314592192 ## File path: src/operator/nn/mkldnn/mkldnn_concat.cc ## @@ -25,6 +25,7 @@ #if MXNET_USE_MKLDNN == 1 #include "mkldnn_concat

[GitHub] [incubator-mxnet] larroy commented on a change in pull request #14940: Fix warnings

2019-08-15 Thread GitBox
larroy commented on a change in pull request #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#discussion_r314592065 ## File path: src/common/exec_utils.h ## @@ -49,85 +49,19 @@ namespace common { indices are not recorded * \return

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #15880: [CI] unix cpu validation Timeout

2019-08-15 Thread GitBox
ChaiBapchya commented on issue #15880: [CI] unix cpu validation Timeout URL: https://github.com/apache/incubator-mxnet/issues/15880#issuecomment-521894929 Another one #15736 http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Funix-cpu/detail/PR-15736/10/

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15753: Numpy add numpy linalg norm and numpy min

2019-08-15 Thread GitBox
haojin2 commented on a change in pull request #15753: Numpy add numpy linalg norm and numpy min URL: https://github.com/apache/incubator-mxnet/pull/15753#discussion_r314591015 ## File path: src/operator/numpy/np_broadcast_reduce_op_value.cc ## @@ -77,9 +77,9 @@ NNVM_REGIST

[GitHub] [incubator-mxnet] apeforest commented on issue #15703: Storage manager / memory usage regression in v1.5

2019-08-15 Thread GitBox
apeforest commented on issue #15703: Storage manager / memory usage regression in v1.5 URL: https://github.com/apache/incubator-mxnet/issues/15703#issuecomment-521893896 Interestingly, I found that turning on the USE_INT64_TENSOR_SIZE flag (meaning using int64_t instead of int32_t as inde

[GitHub] [incubator-mxnet] tingying2020 commented on a change in pull request #15838: [numpy] nonzero

2019-08-15 Thread GitBox
tingying2020 commented on a change in pull request #15838: [numpy] nonzero URL: https://github.com/apache/incubator-mxnet/pull/15838#discussion_r314590819 ## File path: tests/python/gpu/test_operator_gpu.py ## @@ -2331,6 +2331,47 @@ def test_math(): for op in o

[GitHub] [incubator-mxnet] tingying2020 commented on a change in pull request #15838: [numpy] nonzero

2019-08-15 Thread GitBox
tingying2020 commented on a change in pull request #15838: [numpy] nonzero URL: https://github.com/apache/incubator-mxnet/pull/15838#discussion_r314590829 ## File path: tests/python/gpu/test_operator_gpu.py ## @@ -2331,6 +2331,47 @@ def test_math(): for op in o

[GitHub] [incubator-mxnet] tingying2020 commented on a change in pull request #15838: [numpy] nonzero

2019-08-15 Thread GitBox
tingying2020 commented on a change in pull request #15838: [numpy] nonzero URL: https://github.com/apache/incubator-mxnet/pull/15838#discussion_r314590614 ## File path: src/operator/numpy/np_nonzero_op.cu ## @@ -0,0 +1,130 @@ +/* + * Licensed to the Apache Software Foundati

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #14940: Fix warnings

2019-08-15 Thread GitBox
apeforest commented on a change in pull request #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#discussion_r314590377 ## File path: include/mxnet/executor.h ## @@ -28,6 +28,8 @@ #include #include +#include Review comment: where is

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #14940: Fix warnings

2019-08-15 Thread GitBox
apeforest commented on a change in pull request #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#discussion_r314589730 ## File path: src/operator/nn/mkldnn/mkldnn_fully_connected.cc ## @@ -26,6 +26,7 @@ #if MXNET_USE_MKLDNN == 1 #include "m

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #14940: Fix warnings

2019-08-15 Thread GitBox
apeforest commented on a change in pull request #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#discussion_r314589743 ## File path: src/operator/nn/mkldnn/mkldnn_concat.cc ## @@ -25,6 +25,7 @@ #if MXNET_USE_MKLDNN == 1 #include "mkldnn_con

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #14940: Fix warnings

2019-08-15 Thread GitBox
apeforest commented on a change in pull request #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#discussion_r314589762 ## File path: src/operator/nn/cudnn/cudnn_algoreg-inl.h ## @@ -30,6 +30,7 @@ #include #include #include +#include R

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #14940: Fix warnings

2019-08-15 Thread GitBox
apeforest commented on a change in pull request #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#discussion_r314589619 ## File path: src/common/exec_utils.h ## @@ -49,85 +49,19 @@ namespace common { indices are not recorded * \retu

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #14940: Fix warnings

2019-08-15 Thread GitBox
apeforest commented on a change in pull request #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#discussion_r314589474 ## File path: src/common/exec_utils.cc ## @@ -0,0 +1,519 @@ +/* Review comment: I failed to understand what warnings di

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #14940: Fix warnings

2019-08-15 Thread GitBox
apeforest commented on a change in pull request #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#discussion_r314589551 ## File path: src/common/exec_utils.cc ## @@ -0,0 +1,526 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under on

[GitHub] [incubator-mxnet] zixuanweeei edited a comment on issue #15880: [CI] unix cpu validation Timeout

2019-08-15 Thread GitBox
zixuanweeei edited a comment on issue #15880: [CI] unix cpu validation Timeout URL: https://github.com/apache/incubator-mxnet/issues/15880#issuecomment-521890365 Some fixes from PR #15882 and PR #15922 (they have the same fixes on `test_shuffle`) has reduced the cost from more than 10 mins

[GitHub] [incubator-mxnet] apeforest commented on a change in pull request #14940: Fix warnings

2019-08-15 Thread GitBox
apeforest commented on a change in pull request #14940: Fix warnings URL: https://github.com/apache/incubator-mxnet/pull/14940#discussion_r314589408 ## File path: src/common/exec_utils.cc ## @@ -0,0 +1,526 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under on

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15753: Numpy add numpy linalg norm and numpy min

2019-08-15 Thread GitBox
haojin2 commented on a change in pull request #15753: Numpy add numpy linalg norm and numpy min URL: https://github.com/apache/incubator-mxnet/pull/15753#discussion_r314588556 ## File path: python/mxnet/symbol/numpy/_symbol.py ## @@ -1653,7 +1653,7 @@ def linspace(start, s

[GitHub] [incubator-mxnet] zixuanweeei commented on issue #15880: [CI] unix cpu validation Timeout

2019-08-15 Thread GitBox
zixuanweeei commented on issue #15880: [CI] unix cpu validation Timeout URL: https://github.com/apache/incubator-mxnet/issues/15880#issuecomment-521890365 Some fixes from PR #15882 has reduced the cost from more than 10 mins to no more than 2 mins. It cost ~41s on local test. And it seems

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15753: Numpy add numpy linalg norm and numpy min

2019-08-15 Thread GitBox
haojin2 commented on a change in pull request #15753: Numpy add numpy linalg norm and numpy min URL: https://github.com/apache/incubator-mxnet/pull/15753#discussion_r314588331 ## File path: python/mxnet/ndarray/numpy/linalg.py ## @@ -19,50 +19,184 @@ from __future__ imp

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15753: Numpy add numpy linalg norm and numpy min

2019-08-15 Thread GitBox
haojin2 commented on a change in pull request #15753: Numpy add numpy linalg norm and numpy min URL: https://github.com/apache/incubator-mxnet/pull/15753#discussion_r314588267 ## File path: python/mxnet/ndarray/numpy/linalg.py ## @@ -19,50 +19,184 @@ from __future__ imp

[GitHub] [incubator-mxnet] TaoLv commented on issue #15874: [v1.5.x] Handle fix_gamma in tensorrt subgraph conversion correctly

2019-08-15 Thread GitBox
TaoLv commented on issue #15874: [v1.5.x] Handle fix_gamma in tensorrt subgraph conversion correctly URL: https://github.com/apache/incubator-mxnet/pull/15874#issuecomment-521889387 Thank you @KellenSunderland. Merging now. T

[incubator-mxnet] branch v1.5.x updated: handle fix_gamma in tensorrt subgraph conversion correctly (#15645) (#15874)

2019-08-15 Thread taolv
This is an automated email from the ASF dual-hosted git repository. taolv pushed a commit to branch v1.5.x in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/v1.5.x by this push: new 6a36152 handle fix_gamma in tensorrt su

[GitHub] [incubator-mxnet] TaoLv merged pull request #15874: [v1.5.x] Handle fix_gamma in tensorrt subgraph conversion correctly

2019-08-15 Thread GitBox
TaoLv merged pull request #15874: [v1.5.x] Handle fix_gamma in tensorrt subgraph conversion correctly URL: https://github.com/apache/incubator-mxnet/pull/15874 This is an automated message from the Apache Git Service. To res

[GitHub] [incubator-mxnet] tingying2020 commented on a change in pull request #15819: [Numpy]flip

2019-08-15 Thread GitBox
tingying2020 commented on a change in pull request #15819: [Numpy]flip URL: https://github.com/apache/incubator-mxnet/pull/15819#discussion_r314587498 ## File path: tests/python/gpu/test_operator_gpu.py ## @@ -2331,6 +2331,72 @@ def test_math(): for op in ops:

[GitHub] [incubator-mxnet] aaronmarkham commented on issue #8773: undefined reference to `gotoblas' Makefile:406: recipe for target 'bin/im2rec' failed

2019-08-15 Thread GitBox
aaronmarkham commented on issue #8773: undefined reference to `gotoblas' Makefile:406: recipe for target 'bin/im2rec' failed URL: https://github.com/apache/incubator-mxnet/issues/8773#issuecomment-521888720 I keep running into this issue as well. A clean clone with recursive init of the s

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15905: [Numpy] Basic indexing in symbolic interface

2019-08-15 Thread GitBox
haojin2 commented on a change in pull request #15905: [Numpy] Basic indexing in symbolic interface URL: https://github.com/apache/incubator-mxnet/pull/15905#discussion_r314582942 ## File path: src/operator/numpy/np_matrix_op.cc ## @@ -135,17 +136,165 @@ bool NumpyReshapeIn

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15905: [Numpy] Basic indexing in symbolic interface

2019-08-15 Thread GitBox
haojin2 commented on a change in pull request #15905: [Numpy] Basic indexing in symbolic interface URL: https://github.com/apache/incubator-mxnet/pull/15905#discussion_r314582869 ## File path: src/operator/numpy/np_matrix_op.cc ## @@ -135,17 +136,165 @@ bool NumpyReshapeIn

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15905: [Numpy] Basic indexing in symbolic interface

2019-08-15 Thread GitBox
haojin2 commented on a change in pull request #15905: [Numpy] Basic indexing in symbolic interface URL: https://github.com/apache/incubator-mxnet/pull/15905#discussion_r314582829 ## File path: src/operator/numpy/np_matrix_op.cc ## @@ -135,17 +136,165 @@ bool NumpyReshapeIn

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15905: [Numpy] Basic indexing in symbolic interface

2019-08-15 Thread GitBox
haojin2 commented on a change in pull request #15905: [Numpy] Basic indexing in symbolic interface URL: https://github.com/apache/incubator-mxnet/pull/15905#discussion_r314582898 ## File path: src/operator/numpy/np_matrix_op.cc ## @@ -135,17 +136,165 @@ bool NumpyReshapeIn

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15905: [Numpy] Basic indexing in symbolic interface

2019-08-15 Thread GitBox
haojin2 commented on a change in pull request #15905: [Numpy] Basic indexing in symbolic interface URL: https://github.com/apache/incubator-mxnet/pull/15905#discussion_r314582779 ## File path: src/operator/numpy/np_matrix_op.cc ## @@ -135,17 +136,165 @@ bool NumpyReshapeIn

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15905: [Numpy] Basic indexing in symbolic interface

2019-08-15 Thread GitBox
haojin2 commented on a change in pull request #15905: [Numpy] Basic indexing in symbolic interface URL: https://github.com/apache/incubator-mxnet/pull/15905#discussion_r314582546 ## File path: python/mxnet/symbol/numpy/_symbol.py ## @@ -39,25 +46,90 @@ def _num_outputs(sym

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #15905: [Numpy] Basic indexing in symbolic interface

2019-08-15 Thread GitBox
haojin2 commented on a change in pull request #15905: [Numpy] Basic indexing in symbolic interface URL: https://github.com/apache/incubator-mxnet/pull/15905#discussion_r314582455 ## File path: python/mxnet/symbol/numpy/_symbol.py ## @@ -39,25 +46,90 @@ def _num_outputs(sym

[GitHub] [incubator-mxnet] ptrendx commented on issue #15918: Trainer.save_state does not work if update_on_kvstore=False

2019-08-15 Thread GitBox
ptrendx commented on issue #15918: Trainer.save_state does not work if update_on_kvstore=False URL: https://github.com/apache/incubator-mxnet/issues/15918#issuecomment-521876652 Shouldn't those states be exactly the same for all contexts? --

[GitHub] [incubator-mxnet] yzhliu commented on a change in pull request #15889: [DO NOT MERGE] enable tvm_op for ci

2019-08-15 Thread GitBox
yzhliu commented on a change in pull request #15889: [DO NOT MERGE] enable tvm_op for ci URL: https://github.com/apache/incubator-mxnet/pull/15889#discussion_r31457 ## File path: tests/python/unittest/test_tvm_op.py ## @@ -24,14 +24,13 @@ @with_seed() def test_tvm_

[GitHub] [incubator-mxnet] yzhliu commented on a change in pull request #15889: [DO NOT MERGE] enable tvm_op for ci

2019-08-15 Thread GitBox
yzhliu commented on a change in pull request #15889: [DO NOT MERGE] enable tvm_op for ci URL: https://github.com/apache/incubator-mxnet/pull/15889#discussion_r31457 ## File path: tests/python/unittest/test_tvm_op.py ## @@ -24,14 +24,13 @@ @with_seed() def test_tvm_

[GitHub] [incubator-mxnet] mxnet-label-bot commented on issue #15927: sym.Variable input init need init.dumps(), but NDarray is not JSON serializable

2019-08-15 Thread GitBox
mxnet-label-bot commented on issue #15927: sym.Variable input init need init.dumps(), but NDarray is not JSON serializable URL: https://github.com/apache/incubator-mxnet/issues/15927#issuecomment-521873240 Hey, this is the MXNet Label Bot. Thank you for submitting the issue! I will tr

[GitHub] [incubator-mxnet] david-di opened a new issue #15927: sym.Variable input init need init.dumps(), but NDarray is not JSON serializable

2019-08-15 Thread GitBox
david-di opened a new issue #15927: sym.Variable input init need init.dumps(), but NDarray is not JSON serializable URL: https://github.com/apache/incubator-mxnet/issues/15927 ## Description mxnet.symbol.Variable(name, attr=None, shape=None, lr_mult=None, wd_mult=None, dtype=None, init=

[GitHub] [incubator-mxnet] ptrendx commented on issue #15922: Refactor for windows CI 'out of heap space' errors

2019-08-15 Thread GitBox
ptrendx commented on issue #15922: Refactor for windows CI 'out of heap space' errors URL: https://github.com/apache/incubator-mxnet/pull/15922#issuecomment-521870531 As mentioned to Dick offline, I do not like this PR to be seen as an alternative to the #15882 PR. I believe we should do b

[incubator-mxnet] branch ib/jl-mxnet-root-env updated (36076c4 -> c0ab006)

2019-08-15 Thread iblis
This is an automated email from the ASF dual-hosted git repository. iblis pushed a change to branch ib/jl-mxnet-root-env in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 36076c4 Merge branch 'master' into ib/jl-mxnet-root-env add 67acbe5 [DOC] Fix many argum

[GitHub] [incubator-mxnet] ZhennanQin opened a new pull request #15926: Add disable attr to subgraph property

2019-08-15 Thread GitBox
ZhennanQin opened a new pull request #15926: Add disable attr to subgraph property URL: https://github.com/apache/incubator-mxnet/pull/15926 ## Description ## Can fix the crash after `export MXNET_DISABLE_MKLDNN_CONV_OPT=1` ## Checklist ## ### Essentials ### Please feel free

[GitHub] [incubator-mxnet] zixuanweeei commented on issue #15880: [CI] unix cpu validation Timeout

2019-08-15 Thread GitBox
zixuanweeei commented on issue #15880: [CI] unix cpu validation Timeout URL: https://github.com/apache/incubator-mxnet/issues/15880#issuecomment-521865680 Sure. @pengzhao-intel BTW, I have disabled MKLDNN subgraph backend to see whether it impacts on the efficiency of shuffle opera

[GitHub] [incubator-mxnet] DickJC123 commented on issue #15922: Refactor for windows CI 'out of heap space' errors

2019-08-15 Thread GitBox
DickJC123 commented on issue #15922: Refactor for windows CI 'out of heap space' errors URL: https://github.com/apache/incubator-mxnet/pull/15922#issuecomment-521865392 I reviewed the original laop_6 failure in one of my other PR CI runs and found it was not related to tolerances. Hence r

[GitHub] [incubator-mxnet] DickJC123 commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'.

2019-08-15 Thread GitBox
DickJC123 commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'. URL: https://github.com/apache/incubator-mxnet/pull/15882#issuecomment-521863495 @larroy I looked again at the CI failure of laop_6: http://jenkins.mxnet-ci.amazon-ml.com/blue/rest/

[GitHub] [incubator-mxnet] DickJC123 commented on issue #13143: [MXNET-1206] Support NDArray indexing with None and Ellipsis

2019-08-15 Thread GitBox
DickJC123 commented on issue #13143: [MXNET-1206] Support NDArray indexing with None and Ellipsis URL: https://github.com/apache/incubator-mxnet/pull/13143#issuecomment-521859970 @kohr-h A `git show` of this PR's commit includes: ``` diff --git a/python/mxnet/test_utils.py b/pytho

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #15880: [CI] unix cpu validation Timeout

2019-08-15 Thread GitBox
pengzhao-intel commented on issue #15880: [CI] unix cpu validation Timeout URL: https://github.com/apache/incubator-mxnet/issues/15880#issuecomment-521857381 Thanks @zixuanweeei Could we statistic and sort the runtime for all cases in CPU side (CPU, CPU+MKL, CPU+MKLDNN)? After

[incubator-mxnet] branch master updated (acc7865 -> c8f0b21)

2019-08-15 Thread zachgk
This is an automated email from the ASF dual-hosted git repository. zachgk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from acc7865 added check for empty params file and unknown param (not arg/aux) (#15917) add c8f0b21 Remov

[GitHub] [incubator-mxnet] zachgk merged pull request #15915: Remove Scala package test on build

2019-08-15 Thread GitBox
zachgk merged pull request #15915: Remove Scala package test on build URL: https://github.com/apache/incubator-mxnet/pull/15915 This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [incubator-mxnet] zixuanweeei commented on issue #15880: [CI] unix cpu validation Timeout

2019-08-15 Thread GitBox
zixuanweeei commented on issue #15880: [CI] unix cpu validation Timeout URL: https://github.com/apache/incubator-mxnet/issues/15880#issuecomment-521856502 @pengzhao-intel I've seen cpu runtimes more than 10 minutes by testing `test_random.py:test_shuffle` for three times. Seems there are l

[GitHub] [incubator-mxnet] xidulu commented on a change in pull request #15909: [numpy] random.rand

2019-08-15 Thread GitBox
xidulu commented on a change in pull request #15909: [numpy] random.rand URL: https://github.com/apache/incubator-mxnet/pull/15909#discussion_r314559195 ## File path: python/mxnet/symbol/numpy/random.py ## @@ -49,6 +49,35 @@ def _random_helper(random, sampler, params, shape

[GitHub] [incubator-mxnet] xidulu commented on a change in pull request #15909: [numpy] random.rand

2019-08-15 Thread GitBox
xidulu commented on a change in pull request #15909: [numpy] random.rand URL: https://github.com/apache/incubator-mxnet/pull/15909#discussion_r314559079 ## File path: python/mxnet/numpy/random.py ## @@ -20,7 +20,36 @@ from __future__ import absolute_import from ..ndarray

[incubator-mxnet] branch master updated (f586c07 -> acc7865)

2019-08-15 Thread zachgk
This is an automated email from the ASF dual-hosted git repository. zachgk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from f586c07 [OpPerf] Profiler flag for Python, Cpp (#15881) add acc7865 added check for empty params fil

[GitHub] [incubator-mxnet] zachgk merged pull request #15917: added check for empty params file and unknown param (not arg/aux)

2019-08-15 Thread GitBox
zachgk merged pull request #15917: added check for empty params file and unknown param (not arg/aux) URL: https://github.com/apache/incubator-mxnet/pull/15917 This is an automated message from the Apache Git Service. To resp

[GitHub] [incubator-mxnet] zachgk closed issue #15916: Error when loading models with no params

2019-08-15 Thread GitBox
zachgk closed issue #15916: Error when loading models with no params URL: https://github.com/apache/incubator-mxnet/issues/15916 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #15925: [CI] illegal memory access

2019-08-15 Thread GitBox
ChaiBapchya commented on issue #15925: [CI] illegal memory access URL: https://github.com/apache/incubator-mxnet/issues/15925#issuecomment-521852538 @mxnet-label-bot add [CI, Bug] This is an automated message from the Apache

[GitHub] [incubator-mxnet] ChaiBapchya opened a new issue #15925: [CI] illegal memory access

2019-08-15 Thread GitBox
ChaiBapchya opened a new issue #15925: [CI] illegal memory access URL: https://github.com/apache/incubator-mxnet/issues/15925 Multiple gpu tests fail due to illegal memory access PR - #15736 Pipeline - http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%

[GitHub] [incubator-mxnet] mxnet-label-bot commented on issue #15925: [CI] illegal memory access

2019-08-15 Thread GitBox
mxnet-label-bot commented on issue #15925: [CI] illegal memory access URL: https://github.com/apache/incubator-mxnet/issues/15925#issuecomment-521852278 Hey, this is the MXNet Label Bot. Thank you for submitting the issue! I will try and suggest some labels so that the appropriate MXN

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

2019-08-15 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu 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 3db7aff Bump the publish

[GitHub] [incubator-mxnet] zheng-da commented on issue #15898: missing numpy operators in MXNet.

2019-08-15 Thread GitBox
zheng-da commented on issue #15898: missing numpy operators in MXNet. URL: https://github.com/apache/incubator-mxnet/issues/15898#issuecomment-521844731 `indices_or_sections` in `mx.np.split` should be int, tuple of int or a numpy array. ---

[GitHub] [incubator-mxnet] zheng-da commented on issue #15898: missing numpy operators in MXNet.

2019-08-15 Thread GitBox
zheng-da commented on issue #15898: missing numpy operators in MXNet. URL: https://github.com/apache/incubator-mxnet/issues/15898#issuecomment-521843801 numpy operators accept python list. For example, ``` np.cumsum([1, 2, 3, 4, 5]) ``` ---

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #15880: [CI] unix cpu validation Timeout

2019-08-15 Thread GitBox
pengzhao-intel commented on issue #15880: [CI] unix cpu validation Timeout URL: https://github.com/apache/incubator-mxnet/issues/15880#issuecomment-521826808 This is interesting and we need to figure out if the increased computation leads to the problem. @zixuanweeei could you help to t

[GitHub] [incubator-mxnet] marcoabreu commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'.

2019-08-15 Thread GitBox
marcoabreu commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'. URL: https://github.com/apache/incubator-mxnet/pull/15882#issuecomment-521825065 Hi, there might be a misunderstanding. The issue here is not the cause for the timeouts. -

[GitHub] [incubator-mxnet-ci] marcoabreu merged pull request #9: Add GitHub bots

2019-08-15 Thread GitBox
marcoabreu merged pull request #9: Add GitHub bots URL: https://github.com/apache/incubator-mxnet-ci/pull/9 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[incubator-mxnet-ci] branch github-bots created (now b25d4e1)

2019-08-15 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a change to branch github-bots in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-ci.git. at b25d4e1 Add GitHub bots This branch includes the following new commits: new b25d4e1 Add

[GitHub] [incubator-mxnet-ci] marcoabreu opened a new pull request #9: Add GitHub bots

2019-08-15 Thread GitBox
marcoabreu opened a new pull request #9: Add GitHub bots URL: https://github.com/apache/incubator-mxnet-ci/pull/9 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[incubator-mxnet-ci] branch master updated: Add jenkins slave creation on windows

2019-08-15 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-ci.git The following commit(s) were added to refs/heads/master by this push: new 953bb2e Add jenkins slave creat

[GitHub] [incubator-mxnet-ci] marcoabreu merged pull request #8: Add unix slave creation

2019-08-15 Thread GitBox
marcoabreu merged pull request #8: Add unix slave creation URL: https://github.com/apache/incubator-mxnet-ci/pull/8 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[incubator-mxnet-ci] branch master updated: Add unix slave creation (#8)

2019-08-15 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-ci.git The following commit(s) were added to refs/heads/master by this push: new 3990787 Add unix slave creation

[incubator-mxnet-ci] 01/01: Add unix slave creation

2019-08-15 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a commit to branch unix-slave-creation in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-ci.git commit 628483f16f24f0d07d2172b4f4849ef524c8e4a9 Author: Marco de Abreu AuthorDate: Fri Aug 16 00:1

[incubator-mxnet-ci] branch unix-slave-creation created (now 628483f)

2019-08-15 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a change to branch unix-slave-creation in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-ci.git. at 628483f Add unix slave creation This branch includes the following new commits:

[GitHub] [incubator-mxnet-ci] marcoabreu opened a new pull request #8: Add unix slave creation

2019-08-15 Thread GitBox
marcoabreu opened a new pull request #8: Add unix slave creation URL: https://github.com/apache/incubator-mxnet-ci/pull/8 This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [incubator-mxnet] haojin2 opened a new pull request #15924: Numpy-compatible cumsum upstream

2019-08-15 Thread GitBox
haojin2 opened a new pull request #15924: Numpy-compatible cumsum upstream URL: https://github.com/apache/incubator-mxnet/pull/15924 ## Description ## As title. ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. - [ ] The PR title

[incubator-mxnet-ci] branch master updated (12de57a -> 3d74ae8)

2019-08-15 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-ci.git. discard 12de57a Add jenkins master (#7) omit 8ad0056 Add Jenkins Autoscaling (#6) new e23f5ff Add

[incubator-mxnet-ci] 02/02: Add jenkins master (#7)

2019-08-15 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-ci.git commit 3d74ae8b3f13dff5f3748d662cddfd746422d9ec Author: Marco de Abreu AuthorDate: Fri Aug 16 00:03:34 2019 +02

[incubator-mxnet-ci] branch master updated: Add jenkins master (#7)

2019-08-15 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-ci.git The following commit(s) were added to refs/heads/master by this push: new 12de57a Add jenkins master (#7)

  1   2   3   >