[GitHub] dabraude opened a new pull request #9860: Nnpack

2018-02-22 Thread GitBox
dabraude opened a new pull request #9860: Nnpack URL: https://github.com/apache/incubator-mxnet/pull/9860 ## Description ## This is the first part of #9719 ## Checklist ## ### Essentials ### - [X] Passed code style checking (`make lint`) - [X] Changes are complete

[GitHub] dabraude commented on issue #9573: Fix bug with symbol loading

2018-02-22 Thread GitBox
dabraude commented on issue #9573: Fix bug with symbol loading URL: https://github.com/apache/incubator-mxnet/pull/9573#issuecomment-367613732 failed tests relate to #9856 This is an automated message from the Apache Git

[GitHub] dabraude commented on issue #9573: Fix bug with symbol loading

2018-02-22 Thread GitBox
dabraude commented on issue #9573: Fix bug with symbol loading URL: https://github.com/apache/incubator-mxnet/pull/9573#issuecomment-367613732 failed tests relate to #9856 This is an automated message from the Apache Git

[GitHub] lupesko commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
lupesko commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367608109 @szha This will help MXNet users who can't migrate from 0.11 to 1.0 just yet, but need the operator.

[GitHub] juliusshufan commented on issue #9805: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification

2018-02-22 Thread GitBox
juliusshufan commented on issue #9805: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification URL: https://github.com/apache/incubator-mxnet/pull/9805#issuecomment-366863185 ### I rerun the failed GPU case

[GitHub] dabraude commented on issue #9860: CMake NNPack support

2018-02-22 Thread GitBox
dabraude commented on issue #9860: CMake NNPack support URL: https://github.com/apache/incubator-mxnet/pull/9860#issuecomment-367623208 The test that failed is flaky #9856 This is an automated message from the Apache Git

[GitHub] juliusshufan commented on issue #9805: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification

2018-02-22 Thread GitBox
juliusshufan commented on issue #9805: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification URL: https://github.com/apache/incubator-mxnet/pull/9805#issuecomment-367618399 @szha Hi, I noticed the preci

[GitHub] juliusshufan commented on issue #9805: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification

2018-02-22 Thread GitBox
juliusshufan commented on issue #9805: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification URL: https://github.com/apache/incubator-mxnet/pull/9805#issuecomment-366863185 # I rerun the failed GPU

[GitHub] yajiedesign commented on issue #9798: fix cmake

2018-02-22 Thread GitBox
yajiedesign commented on issue #9798: fix cmake URL: https://github.com/apache/incubator-mxnet/pull/9798#issuecomment-367683536 @marcoabreu yes.the windows night build use it. This is an automated message from the Apache Git

[GitHub] marcoabreu commented on issue #9798: fix cmake

2018-02-22 Thread GitBox
marcoabreu commented on issue #9798: fix cmake URL: https://github.com/apache/incubator-mxnet/pull/9798#issuecomment-367673339 I'm not on my computer right now. could you check whether cuda toolset is actually applied during compilation in CI?

[GitHub] larroy commented on issue #9849: License tool

2018-02-22 Thread GitBox
larroy commented on issue #9849: License tool URL: https://github.com/apache/incubator-mxnet/pull/9849#issuecomment-367686078 @marcoabreu please merge? This is an automated message from the Apache Git Service. To respond to

[GitHub] KellenSunderland opened a new pull request #9861: Add CuDNN (v6) support to Jetson builds

2018-02-22 Thread GitBox
KellenSunderland opened a new pull request #9861: Add CuDNN (v6) support to Jetson builds URL: https://github.com/apache/incubator-mxnet/pull/9861 ## Description ## Add CuDNN (v6) support to Jetson builds. This should make the builds for Jetson devices 20-35% faster for many tasks.

[GitHub] ZiyueHuang commented on issue #9848: add infer_type for regression ops, fix issues#9847

2018-02-22 Thread GitBox
ZiyueHuang commented on issue #9848: add infer_type for regression ops, fix issues#9847 URL: https://github.com/apache/incubator-mxnet/pull/9848#issuecomment-367688993 Unittest is written in symbolic mode while the issue is in imperative mode. In symbolic mode, infer_type pass has a

[GitHub] marcoabreu commented on issue #9860: [WIP] CMake NNPack support

2018-02-22 Thread GitBox
marcoabreu commented on issue #9860: [WIP] CMake NNPack support URL: https://github.com/apache/incubator-mxnet/pull/9860#issuecomment-367669600 That's a LOT of third party dependencies and they're coming from a private repository.

[GitHub] yajiedesign commented on issue #9798: fix cmake

2018-02-22 Thread GitBox
yajiedesign commented on issue #9798: fix cmake URL: https://github.com/apache/incubator-mxnet/pull/9798#issuecomment-367669459 sorry for later. I originally set up CUDA_TOOLSET for the purpose of selecting the CUDA version in CI. I will try to add a default auto option.

[GitHub] marcoabreu commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
marcoabreu commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367670741 We are not running any CI for v0.11 and as far as I remember, was the plan to only support the last two versions (in this case v1.0 and v1.1)

[GitHub] yajiedesign commented on a change in pull request #9798: fix cmake

2018-02-22 Thread GitBox
yajiedesign commented on a change in pull request #9798: fix cmake URL: https://github.com/apache/incubator-mxnet/pull/9798#discussion_r169942811 ## File path: CMakeLists.txt ## @@ -46,6 +46,11 @@ if(USE_CUDA AND NOT USE_OLDCMAKECUDA) (${CMAKE_VERSION}

[GitHub] marcoabreu commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
marcoabreu commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367671272 In my opinion, the right approach here would be that this change is getting done on the master and people who are interested in this operator

[GitHub] cjolivier01 commented on a change in pull request #9770: eye operator, for default storage type

2018-02-22 Thread GitBox
cjolivier01 commented on a change in pull request #9770: eye operator, for default storage type URL: https://github.com/apache/incubator-mxnet/pull/9770#discussion_r170006130 ## File path: src/operator/tensor/init_op.h ## @@ -428,6 +405,39 @@ void FillComputeZerosEx(const

[GitHub] piiswrong commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
piiswrong commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367773072 We retrofit critical bugfixes to previous releases. This is a new feature. If you do it for 0.11, then you'll need to do it for all the releases

[GitHub] cjolivier01 commented on issue #9744: Performance regression when OMP_NUM_THREADS environment variable is not set

2018-02-22 Thread GitBox
cjolivier01 commented on issue #9744: Performance regression when OMP_NUM_THREADS environment variable is not set URL: https://github.com/apache/incubator-mxnet/issues/9744#issuecomment-367727199 Are you sauing I should set the environment variable in the code:

[GitHub] cjolivier01 commented on issue #9744: Performance regression when OMP_NUM_THREADS environment variable is not set

2018-02-22 Thread GitBox
cjolivier01 commented on issue #9744: Performance regression when OMP_NUM_THREADS environment variable is not set URL: https://github.com/apache/incubator-mxnet/issues/9744#issuecomment-367727199 Are you saying I should set the environment variable in the code:

[GitHub] szha commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
szha commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367746320 Yeah, that would be the right approach. Semver doesn?t have to apply for 0.x, so cherry pick should be fine.

[GitHub] Maratyszcza commented on issue #9860: [WIP] CMake NNPack support

2018-02-22 Thread GitBox
Maratyszcza commented on issue #9860: [WIP] CMake NNPack support URL: https://github.com/apache/incubator-mxnet/pull/9860#issuecomment-367771148 PeachPy has `pre-generated` branch, which is a mirror of `master` branch after running `python setup.py generate`. I suggest you use the HEAD of

[GitHub] cjolivier01 commented on issue #9744: Performance regression when OMP_NUM_THREADS environment variable is not set

2018-02-22 Thread GitBox
cjolivier01 commented on issue #9744: Performance regression when OMP_NUM_THREADS environment variable is not set URL: https://github.com/apache/incubator-mxnet/issues/9744#issuecomment-367727571 Can you also explain what these settings do? I know it's in a manual somewhere, but from

[GitHub] Maratyszcza commented on a change in pull request #9860: [WIP] CMake NNPack support

2018-02-22 Thread GitBox
Maratyszcza commented on a change in pull request #9860: [WIP] CMake NNPack support URL: https://github.com/apache/incubator-mxnet/pull/9860#discussion_r170047405 ## File path: CMakeLists.txt ## @@ -551,6 +552,66 @@ if(NOT EXISTS

[GitHub] cjolivier01 commented on issue #9744: Performance regression when OMP_NUM_THREADS environment variable is not set

2018-02-22 Thread GitBox
cjolivier01 commented on issue #9744: Performance regression when OMP_NUM_THREADS environment variable is not set URL: https://github.com/apache/incubator-mxnet/issues/9744#issuecomment-367726497 I am only vaguely familiar with the KMP stuff. Can you please elaborate? Are you saying

[GitHub] piiswrong commented on issue #9860: [WIP] CMake NNPack support

2018-02-22 Thread GitBox
piiswrong commented on issue #9860: [WIP] CMake NNPack support URL: https://github.com/apache/incubator-mxnet/pull/9860#issuecomment-367772409 This is too many submodules. @Maratyszcza Do you have a bundled release or something like that? A single submodule that has a snap shot to all

[GitHub] ZiyueHuang commented on issue #9770: eye operator, for default storage type

2018-02-22 Thread GitBox
ZiyueHuang commented on issue #9770: eye operator, for default storage type URL: https://github.com/apache/incubator-mxnet/pull/9770#issuecomment-367719581 Thanks for your comments. Sorry for the late response. @piiswrong I increased the size and the number of trials and added M=0

[GitHub] piiswrong commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
piiswrong commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367773072 We retrofit critical bugfixes to previous releases. This is a new feature. If you do it for 0.11, then you'll need to do it for all the releases

[GitHub] szha closed pull request #9805: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification

2018-02-22 Thread GitBox
szha closed pull request #9805: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification URL: https://github.com/apache/incubator-mxnet/pull/9805 This is a PR merged from a forked repository. As GitHub

[incubator-mxnet] branch master updated: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification (#9805)

2018-02-22 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new e1d8c66 Enable the reporting of

[GitHub] chsin commented on a change in pull request #9809: fix optimizer bug in CPP-Package

2018-02-22 Thread GitBox
chsin commented on a change in pull request #9809: fix optimizer bug in CPP-Package URL: https://github.com/apache/incubator-mxnet/pull/9809#discussion_r169996703 ## File path: cpp-package/include/mxnet-cpp/optimizer.hpp ## @@ -125,13 +125,16 @@ inline float

[GitHub] zheng-da commented on issue #9820: Flaky test_gluon_model_zoo_gpu.test_training @ Python3: MKLDNN-GPU

2018-02-22 Thread GitBox
zheng-da commented on issue #9820: Flaky test_gluon_model_zoo_gpu.test_training @ Python3: MKLDNN-GPU URL: https://github.com/apache/incubator-mxnet/issues/9820#issuecomment-367791248 After one day of testing, I think I know where the problem is now. Basically, the current implementation

[GitHub] nswamy commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
nswamy commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367788415 @piiswrong @szha Some background: One of the customer is using MXNet-Scala Interface of 0.11 version. They are soon to go live with their

[GitHub] Lorna-Liu commented on issue #9517: Check failed: err == CUDNN_STATUS_SUCCESS (6 vs. 0) CUDNN_STATUS_ARCH_MISMATCH

2018-02-22 Thread GitBox
Lorna-Liu commented on issue #9517: Check failed: err == CUDNN_STATUS_SUCCESS (6 vs. 0) CUDNN_STATUS_ARCH_MISMATCH URL: https://github.com/apache/incubator-mxnet/issues/9517#issuecomment-367805855 @liangyong928 I was wondering if you have solve the problem. I just encountered exactly

[GitHub] marcoabreu commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
marcoabreu commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367815373 Of course, but we can't guarantee correctness due to lacking CI and we'd have to make a formal release according to Apache in order to be able

[GitHub] sifmelcara commented on issue #9809: fix optimizer bug in CPP-Package

2018-02-22 Thread GitBox
sifmelcara commented on issue #9809: fix optimizer bug in CPP-Package URL: https://github.com/apache/incubator-mxnet/pull/9809#issuecomment-367822502 Hello! Just got a ping from @chsin for this PR. I'm the author of the ODR fix. After some git blame, I found this bug was accidentally

[GitHub] sifmelcara commented on issue #9809: fix optimizer bug in CPP-Package

2018-02-22 Thread GitBox
sifmelcara commented on issue #9809: fix optimizer bug in CPP-Package URL: https://github.com/apache/incubator-mxnet/pull/9809#issuecomment-367822502 Hello! Just got a ping from @chsin for this PR. I'm the author of the ODR fix. After some git blame, I found this bug was accidentally

[GitHub] Lorna-Liu commented on issue #9517: Check failed: err == CUDNN_STATUS_SUCCESS (6 vs. 0) CUDNN_STATUS_ARCH_MISMATCH

2018-02-22 Thread GitBox
Lorna-Liu commented on issue #9517: Check failed: err == CUDNN_STATUS_SUCCESS (6 vs. 0) CUDNN_STATUS_ARCH_MISMATCH URL: https://github.com/apache/incubator-mxnet/issues/9517#issuecomment-367805855 @liangyong928 I was wondering if you have solve the problem. I just encountered exactly

[GitHub] marcoabreu commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
marcoabreu commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367810206 Considering the fact that we are not making any releases for v0.11, it doesn't really make sense to work on that branch. Or does the customer

[GitHub] cjolivier01 commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
cjolivier01 commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367813362 Rules are nice, but IMHO, customer adoption right now trumps everything This

[GitHub] cjolivier01 commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
cjolivier01 commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367813362 Rules are nice, but IMHO, customer adoption right now trumps everything. Well, not EVERYHING, but it's important.

[GitHub] szha commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
szha commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367820170 The rules are standards we set for ourselves, and we probably don't want people to think this is a community that's OK to bend the rules whenever it's

[GitHub] sifmelcara commented on issue #9809: fix optimizer bug in CPP-Package

2018-02-22 Thread GitBox
sifmelcara commented on issue #9809: fix optimizer bug in CPP-Package URL: https://github.com/apache/incubator-mxnet/pull/9809#issuecomment-367824559 Hmm, in fact I think adding back `static int __make_ ## OptimizerType ## _ ## Name ## __ =` is a little bit better because local static

[GitHub] marcoabreu commented on issue #9809: fix optimizer bug in CPP-Package

2018-02-22 Thread GitBox
marcoabreu commented on issue #9809: fix optimizer bug in CPP-Package URL: https://github.com/apache/incubator-mxnet/pull/9809#issuecomment-367830881 @larroy This is an automated message from the Apache Git Service. To

[GitHub] dabraude commented on issue #9860: [WIP] CMake NNPack support

2018-02-22 Thread GitBox
dabraude commented on issue #9860: [WIP] CMake NNPack support URL: https://github.com/apache/incubator-mxnet/pull/9860#issuecomment-367794784 @piiswrong NNPack's CmakeLists.txt downloads the dependencies, I can change it so that they are not submodules.

[GitHub] wellner commented on issue #9656: Sparse Tensor support in scala api

2018-02-22 Thread GitBox
wellner commented on issue #9656: Sparse Tensor support in scala api URL: https://github.com/apache/incubator-mxnet/issues/9656#issuecomment-367802177 I'd second this request for support of sparse tensors/ndarrays in Scala. Thanks!

[GitHub] sifmelcara commented on issue #9809: fix optimizer bug in CPP-Package

2018-02-22 Thread GitBox
sifmelcara commented on issue #9809: fix optimizer bug in CPP-Package URL: https://github.com/apache/incubator-mxnet/pull/9809#issuecomment-367822502 Hello! Just got a ping from @chsin for this PR. I'm the author of the ODR fix. After some git blame, I found this bug was accidentally

[GitHub] marcoabreu commented on issue #9820: Flaky test_gluon_model_zoo_gpu.test_training @ Python3: MKLDNN-GPU

2018-02-22 Thread GitBox
marcoabreu commented on issue #9820: Flaky test_gluon_model_zoo_gpu.test_training @ Python3: MKLDNN-GPU URL: https://github.com/apache/incubator-mxnet/issues/9820#issuecomment-367792349 Great news, thanks for diving deep on this problem!

[GitHub] cjolivier01 commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
cjolivier01 commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367818944 probably this discussion should occur on dev and see what the mentors think On Thu, Feb 22, 2018 at 12:42 PM, Marco de Abreu

[GitHub] cjolivier01 commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
cjolivier01 commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367833441 "whenever it's convenient"? none of this seems terribly "convenient". Not sure where that framing came from. Customer obsessions is rarely

[GitHub] szha commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
szha commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367820170 The rules are standards we set for ourselves, and we probably don't want people to think this is a community that's OK with bending the rules whenever

[GitHub] zheng-da opened a new pull request #9862: Fix a race condition in converting data layouts.

2018-02-22 Thread GitBox
zheng-da opened a new pull request #9862: Fix a race condition in converting data layouts. URL: https://github.com/apache/incubator-mxnet/pull/9862 ## Description ## There is a race condition in data layout conversion in the MKLDNN implementation. Currently, when NDArray::data() is

[GitHub] zheng-da commented on issue #9856: Flaky test_random.test_gamma_generator

2018-02-22 Thread GitBox
zheng-da commented on issue #9856: Flaky test_random.test_gamma_generator URL: https://github.com/apache/incubator-mxnet/issues/9856#issuecomment-367839764 Is this problem related to MKLDNN? This is an automated message from

[GitHub] marcoabreu closed pull request #9861: Add CuDNN (v6) support to Jetson builds

2018-02-22 Thread GitBox
marcoabreu closed pull request #9861: Add CuDNN (v6) support to Jetson builds URL: https://github.com/apache/incubator-mxnet/pull/9861 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

[incubator-mxnet] branch master updated: Add CuDNN (v6) support to Jetson builds (#9861)

2018-02-22 Thread marcoabreu
This is an automated email from the ASF dual-hosted git repository. marcoabreu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new 18816a5 Add CuDNN (v6) support to

[GitHub] marcoabreu commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
marcoabreu commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#discussion_r170113380 ## File path: tests/python/gpu/test_gluon_model_zoo_gpu.py ## @@ -105,6 +107,7

[GitHub] marcoabreu commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
marcoabreu commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#discussion_r170113340 ## File path: tests/python/gpu/test_gluon_model_zoo_gpu.py ## @@ -37,11 +37,11

[GitHub] cjolivier01 commented on issue #9853: Flaky test_operator.test_binary_op

2018-02-22 Thread GitBox
cjolivier01 commented on issue #9853: Flaky test_operator.test_binary_op URL: https://github.com/apache/incubator-mxnet/issues/9853#issuecomment-367850176 Wasn't elemwise_add changed to use mkl? btw, was it verified that mkl is faster for all shapes and types? I saw it allocates

[GitHub] marcoabreu commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
marcoabreu commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#discussion_r170119050 ## File path: src/ndarray/ndarray.cc ## @@ -1017,6 +1017,7 @@ inline void

[GitHub] cjolivier01 commented on issue #9848: add infer_type for regression ops, fix issues#9847

2018-02-22 Thread GitBox
cjolivier01 commented on issue #9848: add infer_type for regression ops, fix issues#9847 URL: https://github.com/apache/incubator-mxnet/pull/9848#issuecomment-367853082 Please feel free to add infer type for the other ones if you like (not required for this PR, however).

[GitHub] marcoabreu commented on issue #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
marcoabreu commented on issue #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#issuecomment-367865250 We don't need fixed seeds since #9791 has been merged. Just make sure that the with_seed decorator is properly

[GitHub] sampathchanda commented on issue #9100: Python - ImportError: cannot import name 'c_array_buf'

2018-02-22 Thread GitBox
sampathchanda commented on issue #9100: Python - ImportError: cannot import name 'c_array_buf' URL: https://github.com/apache/incubator-mxnet/issues/9100#issuecomment-367866625 Am getting the same issue, even when having mxnet installed from 'pip install mxnet-cu80==0.11.0'. Any ideas

[GitHub] marcoabreu commented on issue #9848: add infer_type for regression ops, fix issues#9847

2018-02-22 Thread GitBox
marcoabreu commented on issue #9848: add infer_type for regression ops, fix issues#9847 URL: https://github.com/apache/incubator-mxnet/pull/9848#issuecomment-367852882 Very nice catch, @ZiyueHuang! Sounds like a good idea.

[GitHub] cjolivier01 commented on issue #9848: add infer_type for regression ops, fix issues#9847

2018-02-22 Thread GitBox
cjolivier01 commented on issue #9848: add infer_type for regression ops, fix issues#9847 URL: https://github.com/apache/incubator-mxnet/pull/9848#issuecomment-367852782 Makes sense. This is an automated message from the

[GitHub] nswamy commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
nswamy commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367855882 Ok, I registered the symbol under contrib. This is an automated message from the

[GitHub] marcoabreu commented on issue #9856: Flaky test_random.test_gamma_generator

2018-02-22 Thread GitBox
marcoabreu commented on issue #9856: Flaky test_random.test_gamma_generator URL: https://github.com/apache/incubator-mxnet/issues/9856#issuecomment-367857691 No, it's using OpenBLAS. This is an automated message from the

[GitHub] piiswrong commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
piiswrong commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367857655 We can't just put stuff into arbitrary previous versions and then have it disappear later then reappear. I would just make a fork for this

[GitHub] marcoabreu commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
marcoabreu commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367857486 Contrib also sounds good to me. Are we going to make a formal release or will that customer just checkout the branch head and compile

[GitHub] cjolivier01 commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
cjolivier01 commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367858332 +1 would have to go into all releases after 0.11 This is an automated message

[GitHub] marcoabreu commented on issue #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
marcoabreu commented on issue #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#issuecomment-367866215 I'm afraid I still don't really understand. I see that you have re-enabled this test, but unfortunately I'm

[GitHub] zheng-da commented on issue #9853: Flaky test_operator.test_binary_op

2018-02-22 Thread GitBox
zheng-da commented on issue #9853: Flaky test_operator.test_binary_op URL: https://github.com/apache/incubator-mxnet/issues/9853#issuecomment-367868599 I don't have a clue right now. so far we see failures in random generators and binary operators. it's weird why it fails in these simple

[GitHub] larroy commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
larroy commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#discussion_r170139781 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -270,9 +270,24 @@ void

[GitHub] marcoabreu commented on issue #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
marcoabreu commented on issue #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#issuecomment-367872237 Would it be possible to create a test that introduces a delay or uses a for-loop in order to force this race

[GitHub] marcoabreu commented on issue #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
marcoabreu commented on issue #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#issuecomment-367850610 But would this re-enabled test have caught this error? Please explain how we're preventing this problem from

[GitHub] marcoabreu commented on issue #9853: Flaky test_operator.test_binary_op

2018-02-22 Thread GitBox
marcoabreu commented on issue #9853: Flaky test_operator.test_binary_op URL: https://github.com/apache/incubator-mxnet/issues/9853#issuecomment-367860051 Well even if it was changed to use MKL, this would not apply here since we're running on OpenBLAS, right?

[GitHub] zheng-da commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
zheng-da commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#discussion_r170129335 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -270,9 +270,24 @@ void

[GitHub] zheng-da commented on issue #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
zheng-da commented on issue #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#issuecomment-367861997 @cjolivier01 The seed is set so we know what is the expected result. It's easier to tell whether CPU or GPU compute

[GitHub] pengzhao-intel commented on issue #9744: Performance regression when OMP_NUM_THREADS environment variable is not set

2018-02-22 Thread GitBox
pengzhao-intel commented on issue #9744: Performance regression when OMP_NUM_THREADS environment variable is not set URL: https://github.com/apache/incubator-mxnet/issues/9744#issuecomment-367873227 Thanks, @cjolivier01, I mean we set this environment in the code as OMP_NUM_THREADS.

[GitHub] zheng-da commented on issue #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
zheng-da commented on issue #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#issuecomment-367845835 @marcoabreu previously, I disabled the inference test. Now I enabled all tests. I also added some prints to clearly

[GitHub] marcoabreu opened a new pull request #9863: Fix broken website build and update version number

2018-02-22 Thread GitBox
marcoabreu opened a new pull request #9863: Fix broken website build and update version number URL: https://github.com/apache/incubator-mxnet/pull/9863 Update website build script to point to the right file Add 1.1.0 to build_all_versions.sh Made build scripts fail fast with set -xe

[GitHub] cjolivier01 commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
cjolivier01 commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#discussion_r170116831 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -270,9 +270,24 @@

[GitHub] cjolivier01 commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
cjolivier01 commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#discussion_r170117620 ## File path: tests/python/gpu/test_gluon_model_zoo_gpu.py ## @@ -37,11

[GitHub] marcoabreu commented on issue #9843: Flaky test_random.test_exponential_generator @ Python2: MKLDNN-CPU

2018-02-22 Thread GitBox
marcoabreu commented on issue #9843: Flaky test_random.test_exponential_generator @ Python2: MKLDNN-CPU URL: https://github.com/apache/incubator-mxnet/issues/9843#issuecomment-367858507 That might fix the test, but could we risk masking a problem that way? I'm afraid I'm unfamiliar with

[GitHub] marcoabreu commented on issue #9853: Flaky test_operator.test_binary_op

2018-02-22 Thread GitBox
marcoabreu commented on issue #9853: Flaky test_operator.test_binary_op URL: https://github.com/apache/incubator-mxnet/issues/9853#issuecomment-367859620 I don't think we have the tools to measure performance on that scale yet. As far as I know, this is in the works. Since there's still

[GitHub] nswamy commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
nswamy commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367859595 I am ok to put it to all the releases after 0.11.0 upto master, but I don't think it has to be done right away.

[GitHub] zheng-da commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
zheng-da commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#discussion_r170130612 ## File path: src/ndarray/ndarray.cc ## @@ -1017,6 +1017,7 @@ inline void

[GitHub] marcoabreu commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
marcoabreu commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#discussion_r170132782 ## File path: src/ndarray/ndarray.cc ## @@ -1017,6 +1017,7 @@ inline void

[GitHub] nswamy closed pull request #9858: softsign activation function

2018-02-22 Thread GitBox
nswamy closed pull request #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] nswamy commented on issue #9858: softsign activation function

2018-02-22 Thread GitBox
nswamy commented on issue #9858: softsign activation function URL: https://github.com/apache/incubator-mxnet/pull/9858#issuecomment-367872369 I am closing this PR. Customer is ok to apply the patch on v.0.11. This is an

[GitHub] marcoabreu commented on issue #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
marcoabreu commented on issue #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#issuecomment-367872237 Would it be possible to create a test that introduces a delay in order to force this race condition?

[GitHub] nswamy closed pull request #9851: Softsign op

2018-02-22 Thread GitBox
nswamy closed pull request #9851: Softsign op URL: https://github.com/apache/incubator-mxnet/pull/9851 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a

[incubator-mxnet] branch master updated: Softsign Activation Function (#9851)

2018-02-22 Thread nswamy
This is an automated email from the ASF dual-hosted git repository. nswamy 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 6ebec87 Softsign Activation Function

[GitHub] zheng-da commented on issue #9843: Flaky test_random.test_exponential_generator @ Python2: MKLDNN-CPU

2018-02-22 Thread GitBox
zheng-da commented on issue #9843: Flaky test_random.test_exponential_generator @ Python2: MKLDNN-CPU URL: https://github.com/apache/incubator-mxnet/issues/9843#issuecomment-367840947 It seems this problem is related to https://github.com/apache/incubator-mxnet/issues/9856

[GitHub] marcoabreu commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
marcoabreu commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#discussion_r170113089 ## File path: src/ndarray/ndarray.cc ## @@ -1017,6 +1017,7 @@ inline void

[GitHub] marcoabreu commented on issue #9864: Flaky hanging test_operator.test_laop_3 @ Python3: MKLDNN-CPU

2018-02-22 Thread GitBox
marcoabreu commented on issue #9864: Flaky hanging test_operator.test_laop_3 @ Python3: MKLDNN-CPU URL: https://github.com/apache/incubator-mxnet/issues/9864#issuecomment-367853511 @asmushetzel would you like to investigate?

[GitHub] marcoabreu opened a new issue #9864: Flaky hanging @ Python3: MKLDNN-CPU

2018-02-22 Thread GitBox
marcoabreu opened a new issue #9864: Flaky hanging @ Python3: MKLDNN-CPU URL: https://github.com/apache/incubator-mxnet/issues/9864 Test times out due to hang. http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-9848/1/pipeline ```

[GitHub] marcoabreu commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN.

2018-02-22 Thread GitBox
marcoabreu commented on a change in pull request #9862: Fix a race condition in converting data layouts in MKLDNN. URL: https://github.com/apache/incubator-mxnet/pull/9862#discussion_r170132857 ## File path: tests/python/gpu/test_gluon_model_zoo_gpu.py ## @@ -37,8 +37,7

[GitHub] cjolivier01 commented on issue #9853: Flaky test_operator.test_binary_op

2018-02-22 Thread GitBox
cjolivier01 commented on issue #9853: Flaky test_operator.test_binary_op URL: https://github.com/apache/incubator-mxnet/issues/9853#issuecomment-367866040 i don?t know what is invoked in the process of calling test_bmod(). could be that elemwise_add() isn?t called, or is called before

  1   2   >