[GitHub] javelinjs commented on issue #7878: InferShapeKeyword argument name gt_label_37 not found

2017-11-18 Thread GitBox
javelinjs commented on issue #7878: InferShapeKeyword argument name gt_label_37 not found URL: https://github.com/apache/incubator-mxnet/issues/7878#issuecomment-345426913 Actually for `NDArrayIter` you need to stack the data and labels in one `NDArray`, instead of a list of. Passing

[GitHub] anjishnu commented on issue #8625: Attempting to write a relation network in Gluon

2017-11-18 Thread GitBox
anjishnu commented on issue #8625: Attempting to write a relation network in Gluon URL: https://github.com/apache/incubator-mxnet/issues/8625#issuecomment-345434219 Relevant thread : https://discuss.mxnet.io/t/cross-product-style-architectures-with-gluon/271/3 The model doesn't

[GitHub] zhreshold commented on issue #8703: [DISCUSSION] (when) Should we deprecate support for python2?

2017-11-18 Thread GitBox
zhreshold commented on issue #8703: [DISCUSSION] (when) Should we deprecate support for python2? URL: https://github.com/apache/incubator-mxnet/issues/8703#issuecomment-345426979 Let's wait until numpy deprecate it first.

[GitHub] yajiedesign commented on issue #8702: [DISCUSSION] Should we deprecate Makefile and only use CMake?

2017-11-18 Thread GitBox
yajiedesign commented on issue #8702: [DISCUSSION] Should we deprecate Makefile and only use CMake? URL: https://github.com/apache/incubator-mxnet/issues/8702#issuecomment-345433066 shared or static opencv depends on how user provide opencv.the opencv also use cmake build system and

[GitHub] yajiedesign commented on issue #8702: [DISCUSSION] Should we deprecate Makefile and only use CMake?

2017-11-18 Thread GitBox
yajiedesign commented on issue #8702: [DISCUSSION] Should we deprecate Makefile and only use CMake? URL: https://github.com/apache/incubator-mxnet/issues/8702#issuecomment-345433066 @szha shared or static opencv depends on how user provide opencv.the opencv also use cmake build system

[GitHub] anjishnu commented on issue #8625: Attempting to write a relation network in Gluon

2017-11-18 Thread GitBox
anjishnu commented on issue #8625: Attempting to write a relation network in Gluon URL: https://github.com/apache/incubator-mxnet/issues/8625#issuecomment-345434219 Relevant thread : https://discuss.mxnet.io/t/cross-product-style-architectures-with-gluon/271/3 The model doesn't

[GitHub] szha commented on issue #7522: how to calculate an evaluation metric using intermediate layer output?

2017-11-18 Thread GitBox
szha commented on issue #7522: how to calculate an evaluation metric using intermediate layer output? URL: https://github.com/apache/incubator-mxnet/issues/7522#issuecomment-345439009 This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this

[GitHub] szha closed issue #7522: how to calculate an evaluation metric using intermediate layer output?

2017-11-18 Thread GitBox
szha closed issue #7522: how to calculate an evaluation metric using intermediate layer output? URL: https://github.com/apache/incubator-mxnet/issues/7522 This is an automated message from the Apache Git Service. To respond

[GitHub] lialie commented on issue #8675: Sparse tensors causing a deadlock

2017-11-18 Thread GitBox
lialie commented on issue #8675: Sparse tensors causing a deadlock URL: https://github.com/apache/incubator-mxnet/issues/8675#issuecomment-345445110 @marcoabreu I have the same issue with Win7/GPU 1060/CPU i5. At first, the OpenBlas may be a problem, after change it with MKL package, but

[GitHub] SumNeuron commented on issue #3823: How to enable GPU P2P

2017-11-18 Thread GitBox
SumNeuron commented on issue #3823: How to enable GPU P2P URL: https://github.com/apache/incubator-mxnet/issues/3823#issuecomment-345450363 I have 8 gpus and I get the message `only 32 out of 56 GPU pairs are enabled direct access. It may affect the performance. You can set

[GitHub] yuewu001 commented on issue #8702: [DISCUSSION] Should we deprecate Makefile and only use CMake?

2017-11-18 Thread GitBox
yuewu001 commented on issue #8702: [DISCUSSION] Should we deprecate Makefile and only use CMake? URL: https://github.com/apache/incubator-mxnet/issues/8702#issuecomment-345453029 +1 cmake, especially for cross platform

[GitHub] cjolivier01 commented on issue #6773: Deadlock and crashes during shutdown

2017-11-18 Thread GitBox
cjolivier01 commented on issue #6773: Deadlock and crashes during shutdown URL: https://github.com/apache/incubator-mxnet/pull/6773#issuecomment-345449788 That?s odd because as far as I know, notify_all() should never block. Maybe it?s been destroyed at this point? Does it happen every

[GitHub] cjolivier01 opened a new pull request #8706: Invert environment check

2017-11-18 Thread GitBox
cjolivier01 opened a new pull request #8706: Invert environment check URL: https://github.com/apache/incubator-mxnet/pull/8706 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### - [ ] Passed code style checking (`make lint`)

[GitHub] SumNeuron commented on issue #3823: How to enable GPU P2P

2017-11-18 Thread GitBox
SumNeuron commented on issue #3823: How to enable GPU P2P URL: https://github.com/apache/incubator-mxnet/issues/3823#issuecomment-345450363 I have 8 gpus and I get the message `only 32 out of 56 GPU pairs are enabled direct access. It may affect the performance. You can set

[GitHub] xumengwei opened a new issue #8707: Fail to build amalgamation for Android in latest version

2017-11-18 Thread GitBox
xumengwei opened a new issue #8707: Fail to build amalgamation for Android in latest version URL: https://github.com/apache/incubator-mxnet/issues/8707 Environment: MacOS, target: Nexus 6 I built the toolchain as follows `python build/tools/make_standalone_toolchain.py --arch arm

[GitHub] cjolivier01 commented on a change in pull request #8686: Kernel operator tuning

2017-11-18 Thread GitBox
cjolivier01 commented on a change in pull request #8686: Kernel operator tuning URL: https://github.com/apache/incubator-mxnet/pull/8686#discussion_r151841255 ## File path: src/operator/tensor/elemwise_binary_broadcast_op.h ## @@ -134,24 +137,122 @@ inline int

[GitHub] Soonhwan-Kwon commented on issue #8674: ADD CapsNet example

2017-11-18 Thread GitBox
Soonhwan-Kwon commented on issue #8674: ADD CapsNet example URL: https://github.com/apache/incubator-mxnet/pull/8674#issuecomment-345476457 We're planning to make this capsule net example as gluon also. Is there anything to do please let me know.

[GitHub] piiswrong commented on issue #8708: dataloader._batchify function is slow

2017-11-18 Thread GitBox
piiswrong commented on issue #8708: dataloader._batchify function is slow URL: https://github.com/apache/incubator-mxnet/issues/8708#issuecomment-345477465 We are aware of this issue. But don't have a good solution yet. Especially in the case that requires shuffling.

[GitHub] mbaijal commented on issue #8696: Fixing the monitor callback of the bucketing module.

2017-11-18 Thread GitBox
mbaijal commented on issue #8696: Fixing the monitor callback of the bucketing module. URL: https://github.com/apache/incubator-mxnet/pull/8696#issuecomment-345482310 @tdomhan I cannot find this PR on Apache Jenkins anymore and I dont see the details button here either. This might have

[GitHub] cjolivier01 commented on issue #8686: Kernel operator tuning

2017-11-18 Thread GitBox
cjolivier01 commented on issue #8686: Kernel operator tuning URL: https://github.com/apache/incubator-mxnet/pull/8686#issuecomment-345473267 Resetting commit in order to cleanly remove broadcast This is an automated message

[GitHub] Soonhwan-Kwon commented on issue #8703: [DISCUSSION] (when) Should we deprecate support for python2?

2017-11-18 Thread GitBox
Soonhwan-Kwon commented on issue #8703: [DISCUSSION] (when) Should we deprecate support for python2? URL: https://github.com/apache/incubator-mxnet/issues/8703#issuecomment-345478174 Many linux versions has python 2 as default, so I think talking about deprecation is little early.

[GitHub] lialie commented on issue #6773: Deadlock and crashes during shutdown

2017-11-18 Thread GitBox
lialie commented on issue #6773: Deadlock and crashes during shutdown URL: https://github.com/apache/incubator-mxnet/pull/6773#issuecomment-345483454 @cjolivier01 yes, it blocks when exit, not a graceful exit. Happens every time. 1) When doing multiple test, it blocks at concurrency.h

[GitHub] eric-haibin-lin commented on issue #8317: replace `has_key` by `in`

2017-11-18 Thread GitBox
eric-haibin-lin commented on issue #8317: replace `has_key` by `in` URL: https://github.com/apache/incubator-mxnet/pull/8317#issuecomment-345479087 Do you mind sync with master and trigger CI again? This is an automated

[incubator-mxnet] branch master updated: Doc updates for sparse operators (#8641)

2017-11-18 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 257ad1d Doc updates for sparse

[GitHub] piiswrong closed pull request #8692: mac build fix

2017-11-18 Thread GitBox
piiswrong closed pull request #8692: mac build fix URL: https://github.com/apache/incubator-mxnet/pull/8692 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

[GitHub] zhreshold commented on issue #8693: mx.image.ImageIter failed on lst files failed with " src/io/image_io.cc:168: Check failed: static_cast<void*>(dst.ptr()) == out->data().dptr"

2017-11-18 Thread GitBox
zhreshold commented on issue #8693: mx.image.ImageIter failed on lst files failed with " src/io/image_io.cc:168: Check failed: static_cast(dst.ptr()) == out->data().dptr" URL: https://github.com/apache/incubator-mxnet/issues/8693#issuecomment-345467699 I don't understand what do

[GitHub] eric-haibin-lin commented on issue #8683: DenseNet with shared memory

2017-11-18 Thread GitBox
eric-haibin-lin commented on issue #8683: DenseNet with shared memory URL: https://github.com/apache/incubator-mxnet/issues/8683#issuecomment-345479577 That was a typo in the paper. Were you looking for this? https://github.com/taineleau/densenet.mxnet

[GitHub] szha commented on issue #7469: Do we need to subtract mean pixel values when extracting cnn feautre

2017-11-18 Thread GitBox
szha commented on issue #7469: Do we need to subtract mean pixel values when extracting cnn feautre URL: https://github.com/apache/incubator-mxnet/issues/7469#issuecomment-345481959 This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this

[GitHub] szha closed issue #7469: Do we need to subtract mean pixel values when extracting cnn feautre

2017-11-18 Thread GitBox
szha closed issue #7469: Do we need to subtract mean pixel values when extracting cnn feautre URL: https://github.com/apache/incubator-mxnet/issues/7469 This is an automated message from the Apache Git Service. To respond

[GitHub] piiswrong closed pull request #8641: Doc updates for sparse operators

2017-11-18 Thread GitBox
piiswrong closed pull request #8641: Doc updates for sparse operators URL: https://github.com/apache/incubator-mxnet/pull/8641 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

[incubator-mxnet] branch master updated: mac build fix (#8692)

2017-11-18 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 bdd14a8 mac build fix (#8692) bdd14a8

[GitHub] cjolivier01 opened a new issue #8709: Test prints too many errors: test_kvstore.test_invalid_pull

2017-11-18 Thread GitBox
cjolivier01 opened a new issue #8709: Test prints too many errors: test_kvstore.test_invalid_pull URL: https://github.com/apache/incubator-mxnet/issues/8709 Unit test test_kvstore.test_invalid_pull tests catching exceptions for various invalid pulls on kvstore. This results in intended

[GitHub] cjolivier01 commented on issue #6773: Deadlock and crashes during shutdown

2017-11-18 Thread GitBox
cjolivier01 commented on issue #6773: Deadlock and crashes during shutdown URL: https://github.com/apache/incubator-mxnet/pull/6773#issuecomment-345490236 It happens every time? Is there any way you can produce a script that I can reproduce this with?

[GitHub] eric-haibin-lin commented on issue #8709: Test prints too many errors: test_kvstore.test_invalid_pull

2017-11-18 Thread GitBox
eric-haibin-lin commented on issue #8709: Test prints too many errors: test_kvstore.test_invalid_pull URL: https://github.com/apache/incubator-mxnet/issues/8709#issuecomment-345492574 Will fix it. Any suggestion for suppressing the warning message?

[GitHub] cjolivier01 commented on issue #6773: Deadlock and crashes during shutdown

2017-11-18 Thread GitBox
cjolivier01 commented on issue #6773: Deadlock and crashes during shutdown URL: https://github.com/apache/incubator-mxnet/pull/6773#issuecomment-345490379 Although it shouldn;t be necessary (

[GitHub] cjolivier01 commented on issue #6773: Deadlock and crashes during shutdown

2017-11-18 Thread GitBox
cjolivier01 commented on issue #6773: Deadlock and crashes during shutdown URL: https://github.com/apache/incubator-mxnet/pull/6773#issuecomment-345490305 Ok, so it's waiting at 220. Something owns that mutex that's trying to be acquired at line 220. Do you see any threads that in their

[GitHub] cjolivier01 closed pull request #8662: 2bit gradient compression

2017-11-18 Thread GitBox
cjolivier01 closed pull request #8662: 2bit gradient compression URL: https://github.com/apache/incubator-mxnet/pull/8662 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] cjolivier01 closed pull request #8688: Restored some copyright attribution that were accidentally removed.

2017-11-18 Thread GitBox
cjolivier01 closed pull request #8688: Restored some copyright attribution that were accidentally removed. URL: https://github.com/apache/incubator-mxnet/pull/8688 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[incubator-mxnet] branch master updated: 2bit gradient compression (#8662)

2017-11-18 Thread cjolivier01
This is an automated email from the ASF dual-hosted git repository. cjolivier01 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 a499f89 2bit gradient

[incubator-mxnet] branch master updated: fix random docs (#8676)

2017-11-18 Thread cjolivier01
This is an automated email from the ASF dual-hosted git repository. cjolivier01 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 a8f79bc fix random docs (#8676)

[GitHub] cjolivier01 closed pull request #8676: Remove experimental warning on Gluon and add Gluon tutorials

2017-11-18 Thread GitBox
cjolivier01 closed pull request #8676: Remove experimental warning on Gluon and add Gluon tutorials URL: https://github.com/apache/incubator-mxnet/pull/8676 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

[GitHub] cjolivier01 commented on issue #8709: Test prints too many errors: test_kvstore.test_invalid_pull

2017-11-18 Thread GitBox
cjolivier01 commented on issue #8709: Test prints too many errors: test_kvstore.test_invalid_pull URL: https://github.com/apache/incubator-mxnet/issues/8709#issuecomment-345492879 I am not sure, can you redirect stderr to null from python?

[GitHub] szha opened a new pull request #8711: Revert "2bit gradient compression"

2017-11-18 Thread GitBox
szha opened a new pull request #8711: Revert "2bit gradient compression" URL: https://github.com/apache/incubator-mxnet/pull/8711 Reverts apache/incubator-mxnet#8662 which was merged prematurely. @rahul003 please resubmit your work in another PR and go through review after this. Thanks.

[GitHub] piiswrong opened a new pull request #8710: Remove experimental warning on Gluon and add Gluon tutorials

2017-11-18 Thread GitBox
piiswrong opened a new pull request #8710: Remove experimental warning on Gluon and add Gluon tutorials URL: https://github.com/apache/incubator-mxnet/pull/8710 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### - [ ] Passed

[GitHub] piiswrong commented on issue #8709: Test prints too many errors: test_kvstore.test_invalid_pull

2017-11-18 Thread GitBox
piiswrong commented on issue #8709: Test prints too many errors: test_kvstore.test_invalid_pull URL: https://github.com/apache/incubator-mxnet/issues/8709#issuecomment-345496582 @DickJC123 made a utility for supressing errors

[GitHub] iblis17 commented on issue #8677: multi processing and fork fix

2017-11-18 Thread GitBox
iblis17 commented on issue #8677: multi processing and fork fix URL: https://github.com/apache/incubator-mxnet/pull/8677#issuecomment-345454048 https://travis-ci.org/dmlc/MXNet.jl/jobs/303668918#L1561-L1564 Any idea about the failure on macOS build with CPU only config? ```

[GitHub] cjolivier01 commented on issue #8686: Kernel operator tuning

2017-11-18 Thread GitBox
cjolivier01 commented on issue #8686: Kernel operator tuning URL: https://github.com/apache/incubator-mxnet/pull/8686#issuecomment-345419483 OMP overhead this run, ~8500 ns. unary and binary op times: 0.5-200 ns OperatorTuneBase::duration_t OperatorTuneBase::omp_overhead_ns_

[GitHub] marfago opened a new issue #8708: NDArray stack function is slow

2017-11-18 Thread GitBox
marfago opened a new issue #8708: NDArray stack function is slow URL: https://github.com/apache/incubator-mxnet/issues/8708 Hello, I am working on a very small NN in particular the example in mlp-dropout-scratch.ipynb. I am using only CPUs since my GPU is not yet supoported. I am