[GitHub] larroy commented on a change in pull request #10797: Better cleaning of git repo, signal handlers

2018-05-04 Thread GitBox
larroy commented on a change in pull request #10797: Better cleaning of git repo, signal handlers URL: https://github.com/apache/incubator-mxnet/pull/10797#discussion_r186132116 ## File path: ci/docker/runtime_functions.sh ## @@ -353,7 +369,18 @@ sanity_check() { } +

[GitHub] arcadiaphy commented on issue #10794: fix topk nms in multibox_detection operator

2018-05-04 Thread GitBox
arcadiaphy commented on issue #10794: fix topk nms in multibox_detection operator URL: https://github.com/apache/incubator-mxnet/pull/10794#issuecomment-386660719 Done, I find the last commit that has passed the CI and cherry-pick on it

[GitHub] marcoabreu commented on a change in pull request #10797: Better cleaning of git repo, signal handlers

2018-05-04 Thread GitBox
marcoabreu commented on a change in pull request #10797: Better cleaning of git repo, signal handlers URL: https://github.com/apache/incubator-mxnet/pull/10797#discussion_r186121711 ## File path: ci/docker/runtime_functions.sh ## @@ -353,7 +369,18 @@ sanity_check() { }

[incubator-mxnet] branch piiswrong-patch-2 updated (0d1927c -> f6f67e8)

2018-05-04 Thread jxie
This is an automated email from the ASF dual-hosted git repository. jxie pushed a change to branch piiswrong-patch-2 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 0d1927c Update index.md add f6f67e8 Update index.md No new revisions were added by this

[GitHub] marcoabreu commented on issue #10811: Update emails for build failures in Jenkins

2018-05-04 Thread GitBox
marcoabreu commented on issue #10811: Update emails for build failures in Jenkins URL: https://github.com/apache/incubator-mxnet/pull/10811#issuecomment-386638736 This is disabled on purpose. This is an automated message

[GitHub] marcoabreu closed pull request #10811: Update emails for build failures in Jenkins

2018-05-04 Thread GitBox
marcoabreu closed pull request #10811: Update emails for build failures in Jenkins URL: https://github.com/apache/incubator-mxnet/pull/10811 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

[GitHub] sergeykolychev commented on issue #10791: Unable to install mxnet in R 3.5.0

2018-05-04 Thread GitBox
sergeykolychev commented on issue #10791: Unable to install mxnet in R 3.5.0 URL: https://github.com/apache/incubator-mxnet/issues/10791#issuecomment-386665520 @marcoabreu Sorry, I am a perl maintainer not R This is an

[GitHub] marcoabreu commented on a change in pull request #10797: Better cleaning of git repo, signal handlers

2018-05-04 Thread GitBox
marcoabreu commented on a change in pull request #10797: Better cleaning of git repo, signal handlers URL: https://github.com/apache/incubator-mxnet/pull/10797#discussion_r186121711 ## File path: ci/docker/runtime_functions.sh ## @@ -353,7 +369,18 @@ sanity_check() { }

[GitHub] IFeelBloated commented on issue #10804: Use depthwise convolution(group convolution) by cuDNNv7 if available

2018-05-04 Thread GitBox
IFeelBloated commented on issue #10804: Use depthwise convolution(group convolution) by cuDNNv7 if available URL: https://github.com/apache/incubator-mxnet/pull/10804#issuecomment-386635183 I have been working on something recently with heavy use of ResNeXt building blocks, would be nice

[GitHub] piiswrong commented on issue #10804: Use depthwise convolution(group convolution) by cuDNNv7 if available

2018-05-04 Thread GitBox
piiswrong commented on issue #10804: Use depthwise convolution(group convolution) by cuDNNv7 if available URL: https://github.com/apache/incubator-mxnet/pull/10804#issuecomment-386662839 How does cudnn implementation compare to the custom kernels from tf? Should we always use cudnn?

[GitHub] ashokei opened a new pull request #10819: [MXNET-367] update mkldnn to v0.14 and disable building test examples

2018-05-04 Thread GitBox
ashokei opened a new pull request #10819: [MXNET-367] update mkldnn to v0.14 and disable building test examples URL: https://github.com/apache/incubator-mxnet/pull/10819 ## Description ## Resubmitting PR. updated mkldnn submodule to latest release version, and disabled

[GitHub] rahul003 commented on issue #8671: Discussion and troubleshooting on PyPI (pip) installation

2018-05-04 Thread GitBox
rahul003 commented on issue #8671: Discussion and troubleshooting on PyPI (pip) installation URL: https://github.com/apache/incubator-mxnet/issues/8671#issuecomment-386770513 @szha Can we turn on the USE_LIBJPEG_TURBO flag. I find that it helps improve the speed of IO pipeline

[GitHub] anirudh2290 commented on issue #9118: argmax causes python VM to crash

2018-05-04 Thread GitBox
anirudh2290 commented on issue #9118: argmax causes python VM to crash URL: https://github.com/apache/incubator-mxnet/issues/9118#issuecomment-386772793 @laszukdawid Please see: https://github.com/apache/incubator-mxnet/pull/9681

[GitHub] asitstands commented on issue #10768: Use numpy in RandomSampler

2018-05-04 Thread GitBox
asitstands commented on issue #10768: Use numpy in RandomSampler URL: https://github.com/apache/incubator-mxnet/pull/10768#issuecomment-386774321 I think that conda has no special optimization for numpy's shuffle. Numpy's shuffle uses `n` times of swaps of elements in serial, where `n` is

[GitHub] asitstands commented on issue #10768: Use numpy in RandomSampler

2018-05-04 Thread GitBox
asitstands commented on issue #10768: Use numpy in RandomSampler URL: https://github.com/apache/incubator-mxnet/pull/10768#issuecomment-386774321 I think that conda has no special optimization for numpy's shuffle. Numpy's shuffle uses `n` times of element swaps in serial, where `n` is the

[GitHub] leezu commented on issue #10768: Use numpy in RandomSampler

2018-05-04 Thread GitBox
leezu commented on issue #10768: Use numpy in RandomSampler URL: https://github.com/apache/incubator-mxnet/pull/10768#issuecomment-386779158 On my personal computer indeed I experience the same speed-up of mxnet compared to numpy. On the other machines the results I quoted above still

[GitHub] asitstands commented on issue #10768: Use numpy in RandomSampler

2018-05-04 Thread GitBox
asitstands commented on issue #10768: Use numpy in RandomSampler URL: https://github.com/apache/incubator-mxnet/pull/10768#issuecomment-386779966 I'll test on some other environments including AWS and make a PR if I'm sure that the performance hit is not usual.

[GitHub] ashokei commented on issue #10591: [MXNET-365] handle inplace in mkldnn FallBackCompute

2018-05-04 Thread GitBox
ashokei commented on issue #10591: [MXNET-365] handle inplace in mkldnn FallBackCompute URL: https://github.com/apache/incubator-mxnet/pull/10591#issuecomment-386768975 @marcoabreu is this something you can merge. thanks.

[GitHub] rahul003 commented on issue #8671: Discussion and troubleshooting on PyPI (pip) installation

2018-05-04 Thread GitBox
rahul003 commented on issue #8671: Discussion and troubleshooting on PyPI (pip) installation URL: https://github.com/apache/incubator-mxnet/issues/8671#issuecomment-386770513 @szha Can we turn on the USE_LIBJPEG_TURBO flag. I find that it helps improve the speed of IO pipeline

[GitHub] asitstands commented on issue #10768: Use numpy in RandomSampler

2018-05-04 Thread GitBox
asitstands commented on issue #10768: Use numpy in RandomSampler URL: https://github.com/apache/incubator-mxnet/pull/10768#issuecomment-386774321 I think that conda has no special optimization for numpy's shuffle. Numpy's shuffle uses `n` times of element swaps in serial, where `n` is the

[GitHub] TaoLv commented on issue #10104: [WIP][MXNET-107] Fused RNN implementation for CPU

2018-05-04 Thread GitBox
TaoLv commented on issue #10104: [WIP][MXNET-107] Fused RNN implementation for CPU URL: https://github.com/apache/incubator-mxnet/pull/10104#issuecomment-386776831 rebase code to master branch and retrigger ci. This is an

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

2018-05-04 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new 7453bf9 Bump the publish

[GitHub] leezu commented on issue #10768: Use numpy in RandomSampler

2018-05-04 Thread GitBox
leezu commented on issue #10768: Use numpy in RandomSampler URL: https://github.com/apache/incubator-mxnet/pull/10768#issuecomment-386780280 Sounds great, thanks! This is an automated message from the Apache Git Service.

[GitHub] ThomasDelteil commented on issue #10816: Update threaded_engine.cc

2018-05-04 Thread GitBox
ThomasDelteil commented on issue #10816: Update threaded_engine.cc URL: https://github.com/apache/incubator-mxnet/pull/10816#issuecomment-386781720 I see, why not doing this modification along the related one in your follow-up PR then? That would make it easier to follow.

[GitHub] ThomasDelteil commented on issue #10816: Update threaded_engine.cc

2018-05-04 Thread GitBox
ThomasDelteil commented on issue #10816: Update threaded_engine.cc URL: https://github.com/apache/incubator-mxnet/pull/10816#issuecomment-386781720 I see, why not doing this modification along the related ones in your follow-up PR then? That would make it easier to follow.

[GitHub] piiswrong commented on issue #10814: fix a bug for deferred init

2018-05-04 Thread GitBox
piiswrong commented on issue #10814: fix a bug for deferred init URL: https://github.com/apache/incubator-mxnet/pull/10814#issuecomment-386755521 No. It's just unclear error message. This is an automated message from the

[GitHub] sl1pkn07 commented on issue #10558: NNVM build failed in the newst mxnet version

2018-05-04 Thread GitBox
sl1pkn07 commented on issue #10558: NNVM build failed in the newst mxnet version URL: https://github.com/apache/incubator-mxnet/issues/10558#issuecomment-386758859 seems is better select what is not valid instead of what yes. in my case, all more than 6.3.1 and 5.5.0 i need try

[GitHub] wkcn commented on issue #10723: Adding custom C++ ops without modifying mxnet source

2018-05-04 Thread GitBox
wkcn commented on issue #10723: Adding custom C++ ops without modifying mxnet source URL: https://github.com/apache/incubator-mxnet/issues/10723#issuecomment-386774110 I have tried it. It's available. The key point is to get the data pointer of NDArray using _LIB.MXNDArrayGetData

[GitHub] asitstands commented on issue #10768: Use numpy in RandomSampler

2018-05-04 Thread GitBox
asitstands commented on issue #10768: Use numpy in RandomSampler URL: https://github.com/apache/incubator-mxnet/pull/10768#issuecomment-386774321 I think that conda has no special optimization for numpy's shuffle. Numpy's shuffle uses `n` times of element swaps in serial, where `n` is the

[GitHub] sl1pkn07 commented on issue #10558: NNVM build failed in the newst mxnet version

2018-05-04 Thread GitBox
sl1pkn07 commented on issue #10558: NNVM build failed in the newst mxnet version URL: https://github.com/apache/incubator-mxnet/issues/10558#issuecomment-386758859 seems is better select what is not valid instead of what yes. in my case, all more than 6.3.1 and 5.5.0 i need try

[GitHub] hetong007 opened a new issue #10818: Feature Request: Improve ndarray.pad to be an numpy.pad equivalent

2018-05-04 Thread GitBox
hetong007 opened a new issue #10818: Feature Request: Improve ndarray.pad to be an numpy.pad equivalent URL: https://github.com/apache/incubator-mxnet/issues/10818 ## Current Limitations Comparing to `numpy.pad`, the current

[GitHub] zhreshold commented on issue #10820: fix thread contention caused by openmp

2018-05-04 Thread GitBox
zhreshold commented on issue #10820: fix thread contention caused by openmp URL: https://github.com/apache/incubator-mxnet/pull/10820#issuecomment-386769536 @piiswrong Can you have a look? This is an automated message from

[GitHub] indhub closed pull request #10621: [MXNET-340] Updated tutorials page.

2018-05-04 Thread GitBox
indhub closed pull request #10621: [MXNET-340] Updated tutorials page. URL: https://github.com/apache/incubator-mxnet/pull/10621 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: [MXNET-340] Updated tutorials page. (#10621)

2018-05-04 Thread indhub
This is an automated email from the ASF dual-hosted git repository. indhub 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 97511ba [MXNET-340] Updated tutorials

[GitHub] asitstands commented on issue #10768: Use numpy in RandomSampler

2018-05-04 Thread GitBox
asitstands commented on issue #10768: Use numpy in RandomSampler URL: https://github.com/apache/incubator-mxnet/pull/10768#issuecomment-386774321 I think that conda has no special optimization for numpy's shuffle. Numpy's shuffle uses `n` times of element swaps in serial, where `n` is the

[GitHub] ashokei commented on issue #10819: [MXNET-367] update mkldnn to v0.14 and disable building test examples

2018-05-04 Thread GitBox
ashokei commented on issue #10819: [MXNET-367] update mkldnn to v0.14 and disable building test examples URL: https://github.com/apache/incubator-mxnet/pull/10819#issuecomment-386777118 @TaoLv can you please review, i updated mkldnn formats based on new release. i notice there is

[GitHub] yajiedesign opened a new issue #10821: ci error MKLDNN_UTIL_FUNC.MemFormat

2018-05-04 Thread GitBox
yajiedesign opened a new issue #10821: ci error MKLDNN_UTIL_FUNC.MemFormat URL: https://github.com/apache/incubator-mxnet/issues/10821 ## Description ci test FAILED MKLDNN_UTIL_FUNC.MemFormat

[GitHub] yajiedesign closed issue #10821: ci error MKLDNN_UTIL_FUNC.MemFormat

2018-05-04 Thread GitBox
yajiedesign closed issue #10821: ci error MKLDNN_UTIL_FUNC.MemFormat URL: https://github.com/apache/incubator-mxnet/issues/10821 This is an automated message from the Apache Git Service. To respond to the message, please log

[incubator-mxnet] branch master updated: fix (#10814)

2018-05-04 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 3bba4c8 fix (#10814) 3bba4c8 is

[GitHub] piiswrong closed pull request #10814: fix a bug for deferred init

2018-05-04 Thread GitBox
piiswrong closed pull request #10814: fix a bug for deferred init URL: https://github.com/apache/incubator-mxnet/pull/10814 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] piiswrong commented on issue #10816: Update threaded_engine.cc

2018-05-04 Thread GitBox
piiswrong commented on issue #10816: Update threaded_engine.cc URL: https://github.com/apache/incubator-mxnet/pull/10816#issuecomment-386755763 It shouldn't have any effect for any use cases currently supported. I encountered this when working on something new. I'll add tests and

[GitHub] zhreshold opened a new pull request #10820: fix thread contention caused by openmp

2018-05-04 Thread GitBox
zhreshold opened a new pull request #10820: fix thread contention caused by openmp URL: https://github.com/apache/incubator-mxnet/pull/10820 ## Description ## fix thread contention caused by openmp. This helps improve gluon.data.dataLoader perf when num_workers is large. ##

[GitHub] szha commented on issue #8671: Discussion and troubleshooting on PyPI (pip) installation

2018-05-04 Thread GitBox
szha commented on issue #8671: Discussion and troubleshooting on PyPI (pip) installation URL: https://github.com/apache/incubator-mxnet/issues/8671#issuecomment-386770801 @rahul003 thanks for the suggestion. I will certainly take a look. For these dependencies, my approach have been to

[GitHub] szha commented on issue #8671: Discussion and troubleshooting on PyPI (pip) installation

2018-05-04 Thread GitBox
szha commented on issue #8671: Discussion and troubleshooting on PyPI (pip) installation URL: https://github.com/apache/incubator-mxnet/issues/8671#issuecomment-386770801 @rahul003 thanks for the suggestion. I will certainly take a look. For these dependencies, my approach have been to

[GitHub] rahul003 commented on issue #8671: Discussion and troubleshooting on PyPI (pip) installation

2018-05-04 Thread GitBox
rahul003 commented on issue #8671: Discussion and troubleshooting on PyPI (pip) installation URL: https://github.com/apache/incubator-mxnet/issues/8671#issuecomment-386770513 @szha Can we turn on the USE_LIBJPEG_TURBO flag. I find that it helps improve the speed of IO pipeline

[GitHub] hetong007 commented on issue #10123: train_cifar10.py hangs on first epoch in debug mode (4 P100 GPUs)

2018-05-04 Thread GitBox
hetong007 commented on issue #10123: train_cifar10.py hangs on first epoch in debug mode (4 P100 GPUs) URL: https://github.com/apache/incubator-mxnet/issues/10123#issuecomment-386704607 Can you try the train_cifar10.py script at:

[GitHub] piiswrong commented on issue #10766: Bug Cannot save/load params with Gluon model

2018-05-04 Thread GitBox
piiswrong commented on issue #10766: Bug Cannot save/load params with Gluon model URL: https://github.com/apache/incubator-mxnet/issues/10766#issuecomment-386734298 You cannot set_data before you initialize the model. @ariwaranosai yes that's the issue. I'm making a fix

[GitHub] piiswrong closed pull request #10810: Fix Reorder2Default

2018-05-04 Thread GitBox
piiswrong closed pull request #10810: Fix Reorder2Default URL: https://github.com/apache/incubator-mxnet/pull/10810 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

[incubator-mxnet] branch master updated: Reorder2Default: return directly for default format (#10810)

2018-05-04 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 ab6a25e Reorder2Default: return

[GitHub] zheng-da opened a new pull request #10812: Fix a mem error.

2018-05-04 Thread GitBox
zheng-da opened a new pull request #10812: Fix a mem error. URL: https://github.com/apache/incubator-mxnet/pull/10812 ## Description ## This memory error has been discussed in the dev mailing list and this error can be reproduced with the following commands. ``` export

[GitHub] rahul003 commented on issue #10778: Make android error

2018-05-04 Thread GitBox
rahul003 commented on issue #10778: Make android error URL: https://github.com/apache/incubator-mxnet/issues/10778#issuecomment-386724201 Are you trying to cross compile for android? Does using USE_F16C=0 build flag help?

[GitHub] rahul003 commented on issue #10778: Make android error

2018-05-04 Thread GitBox
rahul003 commented on issue #10778: Make android error URL: https://github.com/apache/incubator-mxnet/issues/10778#issuecomment-386724201 Does using USE_F16C=0 help? This is an automated message from the Apache Git Service.

[GitHub] zhanghang1989 opened a new pull request #10815: [MXNET-402] add integer type for pad

2018-05-04 Thread GitBox
zhanghang1989 opened a new pull request #10815: [MXNET-402] add integer type for pad URL: https://github.com/apache/incubator-mxnet/pull/10815 ## Description ## Add integer pad @hetong007 ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for

[GitHub] leezu opened a new pull request #10813: Fix context handling when creating sparse arrays from definition

2018-05-04 Thread GitBox
leezu opened a new pull request #10813: Fix context handling when creating sparse arrays from definition URL: https://github.com/apache/incubator-mxnet/pull/10813 ## Description ## @eric-haibin-lin Currently when creating a sparse array from definition (ie. dense data array and

[GitHub] piiswrong opened a new pull request #10814: fix a bug for deferred init

2018-05-04 Thread GitBox
piiswrong opened a new pull request #10814: fix a bug for deferred init URL: https://github.com/apache/incubator-mxnet/pull/10814 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items

[GitHub] ThomasDelteil commented on issue #10816: Update threaded_engine.cc

2018-05-04 Thread GitBox
ThomasDelteil commented on issue #10816: Update threaded_engine.cc URL: https://github.com/apache/incubator-mxnet/pull/10816#issuecomment-386751129 As we are all trying to get up to speed, it would be nice to explain why this is necessary, and what bug that caused.

[GitHub] piiswrong closed pull request #10812: Fix a mem error.

2018-05-04 Thread GitBox
piiswrong closed pull request #10812: Fix a mem error. URL: https://github.com/apache/incubator-mxnet/pull/10812 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

[incubator-mxnet] branch master updated: Fix a mem error. (#10812)

2018-05-04 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 e49fdae Fix a mem error. (#10812)

[GitHub] laszukdawid commented on issue #9118: argmax causes python VM to crash

2018-05-04 Thread GitBox
laszukdawid commented on issue #9118: argmax causes python VM to crash URL: https://github.com/apache/incubator-mxnet/issues/9118#issuecomment-386751353 @nswamy @anirudh2290 what's the commit sha? I'd like to follow this ticket.

[GitHub] snflake commented on issue #10804: Use depthwise convolution(group convolution) by cuDNNv7 if available

2018-05-04 Thread GitBox
snflake commented on issue #10804: Use depthwise convolution(group convolution) by cuDNNv7 if available URL: https://github.com/apache/incubator-mxnet/pull/10804#issuecomment-386712514 I got similar runtime with MobileNet v2 on laptop (Nvidia Quadro M1000M) using custom kernel and grouped

[GitHub] piiswrong closed pull request #10800: Update index.md

2018-05-04 Thread GitBox
piiswrong closed pull request #10800: Update index.md URL: https://github.com/apache/incubator-mxnet/pull/10800 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[incubator-mxnet] branch master updated: Update index.md (#10800)

2018-05-04 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 aabaab4 Update index.md (#10800)

[GitHub] leezu commented on issue #10768: Use numpy in RandomSampler

2018-05-04 Thread GitBox
leezu commented on issue #10768: Use numpy in RandomSampler URL: https://github.com/apache/incubator-mxnet/pull/10768#issuecomment-386742567 @asitstands I guess the difference between our experiments is that I used a optimized numpy from conda and the standard [mxnet pypi

[GitHub] sl1pkn07 commented on issue #10558: NNVM build failed in the newst mxnet version

2018-05-04 Thread GitBox
sl1pkn07 commented on issue #10558: NNVM build failed in the newst mxnet version URL: https://github.com/apache/incubator-mxnet/issues/10558#issuecomment-386743439 yes, seems GCC 5.4.0 and GCC 6.3.1(snapshot used in archlinux) build OK without get rid `-O3` in all makefiles/cmakelists

[GitHub] sl1pkn07 commented on issue #10558: NNVM build failed in the newst mxnet version

2018-05-04 Thread GitBox
sl1pkn07 commented on issue #10558: NNVM build failed in the newst mxnet version URL: https://github.com/apache/incubator-mxnet/issues/10558#issuecomment-386743439 yes, seems GCC 5.4.0 and GCC 6.3.1(snapshot used in archlinux) build OK without get rid `-O3` in all makefiles/cmakelists

[GitHub] piiswrong opened a new pull request #10817: [WIP] Do Not Merge. Static memory allocation for cached_op

2018-05-04 Thread GitBox
piiswrong opened a new pull request #10817: [WIP] Do Not Merge. Static memory allocation for cached_op URL: https://github.com/apache/incubator-mxnet/pull/10817 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel

[GitHub] zhreshold closed pull request #10794: fix topk nms in multibox_detection operator

2018-05-04 Thread GitBox
zhreshold closed pull request #10794: fix topk nms in multibox_detection operator URL: https://github.com/apache/incubator-mxnet/pull/10794 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

[incubator-mxnet] branch master updated: fix topk nms in multibox_detection operator (#10794)

2018-05-04 Thread zhreshold
This is an automated email from the ASF dual-hosted git repository. zhreshold 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 38ec93c fix topk nms in

[GitHub] zhreshold commented on issue #10794: fix topk nms in multibox_detection operator

2018-05-04 Thread GitBox
zhreshold commented on issue #10794: fix topk nms in multibox_detection operator URL: https://github.com/apache/incubator-mxnet/pull/10794#issuecomment-386713496 Merged, thanks for the contribution! This is an automated

[GitHub] piiswrong opened a new pull request #10816: Update threaded_engine.cc

2018-05-04 Thread GitBox
piiswrong opened a new pull request #10816: Update threaded_engine.cc URL: https://github.com/apache/incubator-mxnet/pull/10816 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for

[GitHub] rahul003 commented on issue #10558: NNVM build failed in the newst mxnet version

2018-05-04 Thread GitBox
rahul003 commented on issue #10558: NNVM build failed in the newst mxnet version URL: https://github.com/apache/incubator-mxnet/issues/10558#issuecomment-386752885 Good to know @sl1pkn07. Thanks for verifying that this works. That's a good idea, the only concern is that we might need to

[GitHub] thomelane commented on issue #10621: [MXNET-340] Updated tutorials page.

2018-05-04 Thread GitBox
thomelane commented on issue #10621: [MXNET-340] Updated tutorials page. URL: https://github.com/apache/incubator-mxnet/pull/10621#issuecomment-386753796 @piiswrong fixed as suggested, but kept an alternative link in brackets. @indhub are you able to merge now we have a successful

[GitHub] grainw commented on issue #2754: import mxnetReason: image not found

2018-05-04 Thread GitBox
grainw commented on issue #2754: import mxnetReason: image not found URL: https://github.com/apache/incubator-mxnet/issues/2754#issuecomment-386536477 Hey did you figure out how to solve this?@pjpan This is an automated

[GitHub] threeleafzerg commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by MPI AllReduce

2018-05-04 Thread GitBox
threeleafzerg commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by MPI AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-386539894 @rahul003 For GPU, I agree with your comment. Currently we leave the place holder for GPU for future

[GitHub] snflake commented on issue #10804: Use depthwise convolution(group convolution) by cuDNNv7 if available

2018-05-04 Thread GitBox
snflake commented on issue #10804: Use depthwise convolution(group convolution) by cuDNNv7 if available URL: https://github.com/apache/incubator-mxnet/pull/10804#issuecomment-386535684 The CI failure seems to not related to this PR. unknown file: Failure C++ exception with

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

2018-05-04 Thread GitBox
xinyu-intel commented on issue #10629: [MXNET-343]fix Mkldnn with msvc URL: https://github.com/apache/incubator-mxnet/pull/10629#issuecomment-386544479 @yajiedesign update your submodule using `git submodule update --init --recursive`, and then commit the new mkldnn to your repo.

[GitHub] snflake commented on issue #10804: Use depthwise convolution(group convolution) by cuDNNv7 if available

2018-05-04 Thread GitBox
snflake commented on issue #10804: Use depthwise convolution(group convolution) by cuDNNv7 if available URL: https://github.com/apache/incubator-mxnet/pull/10804#issuecomment-386534903 Great work! This seems to explain the current low performance of Mxnet compared to Tensorflow when

[GitHub] threeleafzerg commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by MPI AllReduce

2018-05-04 Thread GitBox
threeleafzerg commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by MPI AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-386539894 @rahul003 For GPU, I agree with your comment. Currently we leave the place holder for GPU for future

[GitHub] asitstands commented on issue #10768: Use numpy in RandomSampler

2018-05-04 Thread GitBox
asitstands commented on issue #10768: Use numpy in RandomSampler URL: https://github.com/apache/incubator-mxnet/pull/10768#issuecomment-386546813 Thanks @leezu. I wish this discussion would not bother you too much. Here is my test code. ```python import time import mxnet as mx

[GitHub] asitstands commented on issue #10768: Use numpy in RandomSampler

2018-05-04 Thread GitBox
asitstands commented on issue #10768: Use numpy in RandomSampler URL: https://github.com/apache/incubator-mxnet/pull/10768#issuecomment-386546813 Thanks @leezu. I wish this discussion would not bother you too much. Here is my test code. ```python import time import mxnet as mx

[GitHub] rahul003 commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by MPI AllReduce

2018-05-04 Thread GitBox
rahul003 commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by MPI AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-386512681 - Implementation only for CPU is very restrictive. Are you also trying to implement for GPU? Are you running

[GitHub] rahul003 commented on a change in pull request #10696: [MXNET-366]Extend MXNet Distributed Training by MPI AllReduce

2018-05-04 Thread GitBox
rahul003 commented on a change in pull request #10696: [MXNET-366]Extend MXNet Distributed Training by MPI AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#discussion_r185994804 ## File path: src/mpi_collectives/src/mpi_collectives.cxx ## @@ -0,0 +1,827

[GitHub] marcoabreu commented on issue #10808: Revert "[MXNET-367] update mkldnn to v0.14 and disable building test examples"

2018-05-04 Thread GitBox
marcoabreu commented on issue #10808: Revert "[MXNET-367] update mkldnn to v0.14 and disable building test examples" URL: https://github.com/apache/incubator-mxnet/pull/10808#issuecomment-386528117 The problem here is that there were only 10 hours in between the merges of the two clashing

[GitHub] diyang commented on issue #10805: SKIP RNN is incorrect in LSTnet

2018-05-04 Thread GitBox
diyang commented on issue #10805: SKIP RNN is incorrect in LSTnet URL: https://github.com/apache/incubator-mxnet/issues/10805#issuecomment-386563096 I have used MxNet R to implement SKIP RNN You may find it in this function. ```R rnn.skip.unroll<-function(data,

[GitHub] diyang commented on issue #10805: SKIP RNN is incorrect in LSTnet

2018-05-04 Thread GitBox
diyang commented on issue #10805: SKIP RNN is incorrect in LSTnet URL: https://github.com/apache/incubator-mxnet/issues/10805#issuecomment-386563096 I have used MxNet R to implement SKIP RNN You may find it in this function.

[incubator-mxnet] branch master updated: Revert "[MXNET-367] update mkldnn to v0.14 and disable building test examples (#10736)" (#10808)

2018-05-04 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 66365ef Revert "[MXNET-367]

[GitHub] marcoabreu closed pull request #10808: Revert "[MXNET-367] update mkldnn to v0.14 and disable building test examples"

2018-05-04 Thread GitBox
marcoabreu closed pull request #10808: Revert "[MXNET-367] update mkldnn to v0.14 and disable building test examples" URL: https://github.com/apache/incubator-mxnet/pull/10808 This is an automated message from the Apache

[GitHub] larroy commented on a change in pull request #10797: Better cleaning of git repo, signal handlers

2018-05-04 Thread GitBox
larroy commented on a change in pull request #10797: Better cleaning of git repo, signal handlers URL: https://github.com/apache/incubator-mxnet/pull/10797#discussion_r186055013 ## File path: ci/docker/runtime_functions.sh ## @@ -353,7 +369,18 @@ sanity_check() { } +

[GitHub] diyang commented on issue #10805: SKIP RNN is incorrect in LSTnet

2018-05-04 Thread GitBox
diyang commented on issue #10805: SKIP RNN is incorrect in LSTnet URL: https://github.com/apache/incubator-mxnet/issues/10805#issuecomment-386563096 I have used MxNet R to implement SKIP RNN You may find it in this function.

[GitHub] diyang commented on issue #10805: SKIP RNN is incorrect in LSTnet

2018-05-04 Thread GitBox
diyang commented on issue #10805: SKIP RNN is incorrect in LSTnet URL: https://github.com/apache/incubator-mxnet/issues/10805#issuecomment-386563096 @QiXuanWang I have used MxNet R to implement SKIP RNN You may find it in this function.

[GitHub] diyang commented on issue #10805: SKIP RNN is incorrect in LSTnet

2018-05-04 Thread GitBox
diyang commented on issue #10805: SKIP RNN is incorrect in LSTnet URL: https://github.com/apache/incubator-mxnet/issues/10805#issuecomment-386581465 @QiXuanWang By the way, according to the paper, if your data is not periodic, or period is dynamic, then you shall choose the variation of

[GitHub] marcoabreu commented on a change in pull request #10797: Better cleaning of git repo, signal handlers

2018-05-04 Thread GitBox
marcoabreu commented on a change in pull request #10797: Better cleaning of git repo, signal handlers URL: https://github.com/apache/incubator-mxnet/pull/10797#discussion_r186086546 ## File path: ci/docker/runtime_functions.sh ## @@ -353,7 +369,18 @@ sanity_check() { }

[GitHub] TaoLv opened a new pull request #10810: Fix Reorder2Default

2018-05-04 Thread GitBox
TaoLv opened a new pull request #10810: Fix Reorder2Default URL: https://github.com/apache/incubator-mxnet/pull/10810 ## Description ## Fix issue #10809. @zheng-da @pengzhao-intel @ashokei Please review. @dwSun Feel free to try if you are instereted in building from source.

[GitHub] marcoabreu commented on issue #10791: Unable to install mxnet in R 3.5.0

2018-05-04 Thread GitBox
marcoabreu commented on issue #10791: Unable to install mxnet in R 3.5.0 URL: https://github.com/apache/incubator-mxnet/issues/10791#issuecomment-386607901 @sergeykolychev This is an automated message from the Apache Git

[GitHub] snflake commented on issue #10804: Use depthwise convolution(group convolution) by cuDNNv7 if available

2018-05-04 Thread GitBox
snflake commented on issue #10804: Use depthwise convolution(group convolution) by cuDNNv7 if available URL: https://github.com/apache/incubator-mxnet/pull/10804#issuecomment-386537855 About the speed, I used TensorRT with cudnn 7 for inference and depthwise conv is very fast regardless

[GitHub] dwSun commented on issue #10807: Ndarray.asnumpy() error with gluon dense under both GPU and CPU environment

2018-05-04 Thread GitBox
dwSun commented on issue #10807: Ndarray.asnumpy() error with gluon dense under both GPU and CPU environment URL: https://github.com/apache/incubator-mxnet/issues/10807#issuecomment-386549085 modified your script as this: ```py from mxnet.gluon import nn import mxnet as mx

[GitHub] dwSun opened a new issue #10809: Check failed: format != mkl_mem_->GetFormat() (5 vs. 5)

2018-05-04 Thread GitBox
dwSun opened a new issue #10809: Check failed: format != mkl_mem_->GetFormat() (5 vs. 5) URL: https://github.com/apache/incubator-mxnet/issues/10809 ## Description Crashed when training a model. With code from [this

[GitHub] threeleafzerg commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by MPI AllReduce

2018-05-04 Thread GitBox
threeleafzerg commented on issue #10696: [MXNET-366]Extend MXNet Distributed Training by MPI AllReduce URL: https://github.com/apache/incubator-mxnet/pull/10696#issuecomment-386539894 @rahul003 For GPU, I agree with your comment. But the majority code of this PR is the infrastructure

[GitHub] TaoLv commented on issue #10809: Check failed: format != mkl_mem_->GetFormat() (5 vs. 5)

2018-05-04 Thread GitBox
TaoLv commented on issue #10809: Check failed: format != mkl_mem_->GetFormat() (5 vs. 5) URL: https://github.com/apache/incubator-mxnet/issues/10809#issuecomment-386600967 @dwSun Thanks for reporting this. I will take a look and be back to you soon.

[GitHub] zheng-da commented on issue #10810: Fix Reorder2Default

2018-05-04 Thread GitBox
zheng-da commented on issue #10810: Fix Reorder2Default URL: https://github.com/apache/incubator-mxnet/pull/10810#issuecomment-386621576 looks good. This is an automated message from the Apache Git Service. To respond to the

  1   2   >