[GitHub] Davdi edited a comment on issue #13526: distributed training van.cc Check failed

Davdi edited a comment on issue #13526: distributed training van.cc Check failed URL: https://github.com/apache/incubator-mxnet/issues/13526#issuecomment-444328924 > @Davdi your make instruction should use USE_DIST_KVSTORE=1 I guess you probably did it, else

[GitHub] Davdi edited a comment on issue #13526: distributed training van.cc Check failed

Davdi edited a comment on issue #13526: distributed training van.cc Check failed URL: https://github.com/apache/incubator-mxnet/issues/13526#issuecomment-444328924 > @Davdi your make instruction should use USE_DIST_KVSTORE=1 I guess you probably did it, else

[GitHub] TaoLv commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

TaoLv commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444389421 The new parameter `layout` will also be passed to quantized_pooling:

[GitHub] TaoLv commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

TaoLv commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444389018 Have you ever tried to pass NHWC input into MKL-DNN pooling? Seems it's not well handled here.

[GitHub] apeforest commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

apeforest commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444385641 Adding @sandeep-krishnamurthy for review. This is an

[GitHub] apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r238952149 ## File path: src/operator/nn/cudnn/cudnn_pooling-inl.h ## @@ -165,55 +170,78 @@ class

[GitHub] apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r238951811 ## File path: src/operator/nn/cudnn/cudnn_pooling-inl.h ## @@ -165,55 +170,78 @@ class

[GitHub] apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r238951132 ## File path: src/operator/nn/cudnn/cudnn_pooling-inl.h ## @@ -165,55 +170,78 @@ class

[GitHub] apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r238951075 ## File path: src/operator/nn/cudnn/cudnn_pooling-inl.h ## @@ -113,9 +115,11 @@ class

[GitHub] apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r238951359 ## File path: src/operator/nn/cudnn/cudnn_pooling-inl.h ## @@ -165,55 +170,78 @@ class

[GitHub] apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only)

apeforest commented on a change in pull request #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#discussion_r238951040 ## File path: src/operator/nn/cudnn/cudnn_pooling-inl.h ## @@ -73,15 +73,17 @@ class

[GitHub] Vikas89 commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

Vikas89 commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444381509 @ptrendx can you please add more details in description explaining what is this PR doing and the motivation behind doing

[GitHub] lanking520 commented on issue #13039: [MXNET-918] Random module

lanking520 commented on issue #13039: [MXNET-918] Random module URL: https://github.com/apache/incubator-mxnet/pull/13039#issuecomment-444380975 @mdespriee could you please trigger the test again? This is an automated message

[GitHub] apeforest commented on issue #13062: [MXNET-1110] Add header files required by horovod

apeforest commented on issue #13062: [MXNET-1110] Add header files required by horovod URL: https://github.com/apache/incubator-mxnet/pull/13062#issuecomment-444380627 @anirudh2290 Can this PR be merged now? This is an

[GitHub] Vikas89 commented on issue #13515: MKLDNN Perplexity Issue

Vikas89 commented on issue #13515: MKLDNN Perplexity Issue URL: https://github.com/apache/incubator-mxnet/issues/13515#issuecomment-444379931 @mseth10 please share the test training script and steps to run and instance type(I think its c5.18x but please confirm) with Patric

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

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 313284e Bump the publish

[GitHub] Vikas89 commented on issue #13526: distributed training van.cc Check failed

Vikas89 commented on issue #13526: distributed training van.cc Check failed URL: https://github.com/apache/incubator-mxnet/issues/13526#issuecomment-444379161 Are you running in containers? You said "the env is 1 ps and 1 worker and should i follow the instruction on all the 2

[GitHub] anirudhacharya commented on issue #8079: New logo idea

anirudhacharya commented on issue #8079: New logo idea URL: https://github.com/apache/incubator-mxnet/issues/8079#issuecomment-444374904 If this is still up for a vote and if we are going to put an animal face on the logo like a mascot, then I think it should be a dragon.

[GitHub] lupesko commented on a change in pull request #13534: add cpp example inception to nightly test

lupesko commented on a change in pull request #13534: add cpp example inception to nightly test URL: https://github.com/apache/incubator-mxnet/pull/13534#discussion_r238942500 ## File path: cpp-package/example/mlp.cpp ## @@ -144,7 +144,7 @@ void MLP() {

[GitHub] sandeep-krishnamurthy commented on issue #13094: Simplifications and some fun stuff for the MNIST Gluon tutorial

sandeep-krishnamurthy commented on issue #13094: Simplifications and some fun stuff for the MNIST Gluon tutorial URL: https://github.com/apache/incubator-mxnet/pull/13094#issuecomment-444373237 Sigmoid activation is a better fit here to output the probability of 10 digits.

[GitHub] lupesko commented on issue #13346: Aggregate SGD

lupesko commented on issue #13346: Aggregate SGD URL: https://github.com/apache/incubator-mxnet/pull/13346#issuecomment-444371495 Thanks for the contribution @ptrendx ! Adding @nswamy and @sandeep-krishnamurthy to help review/merge.

[GitHub] ChaiBapchya commented on a change in pull request #13543: Chi_square_check for discrete distribution fix

ChaiBapchya commented on a change in pull request #13543: Chi_square_check for discrete distribution fix URL: https://github.com/apache/incubator-mxnet/pull/13543#discussion_r238940613 ## File path: python/mxnet/test_utils.py ## @@ -1915,8 +1915,8 @@ def

[GitHub] lupesko commented on issue #13471: Optimization of metric evaluation

lupesko commented on issue #13471: Optimization of metric evaluation URL: https://github.com/apache/incubator-mxnet/pull/13471#issuecomment-444370311 Thanks for the contribution @ptrendx ! Adding @sandeep-krishnamurthy to help review and merge.

[GitHub] lupesko commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only)

lupesko commented on issue #13362: Add NHWC layout support to Pooling (cuDNN only) URL: https://github.com/apache/incubator-mxnet/pull/13362#issuecomment-444368885 Thanks for the contribution @DickJC123 and @ptrendx ! Adding @apeforest @yuxihu and @anirudh2290 to help review.

[GitHub] ChaiBapchya commented on a change in pull request #13543: Chi_square_check for discrete distribution fix

ChaiBapchya commented on a change in pull request #13543: Chi_square_check for discrete distribution fix URL: https://github.com/apache/incubator-mxnet/pull/13543#discussion_r238938530 ## File path: python/mxnet/test_utils.py ## @@ -1915,8 +1915,8 @@ def

[GitHub] ChaiBapchya commented on a change in pull request #13543: Chi_square_check for discrete distribution fix

ChaiBapchya commented on a change in pull request #13543: Chi_square_check for discrete distribution fix URL: https://github.com/apache/incubator-mxnet/pull/13543#discussion_r238938415 ## File path: python/mxnet/test_utils.py ## @@ -1915,8 +1915,8 @@ def

[GitHub] huangzhiyuan commented on a change in pull request #12980: Add reshape op supported by MKL-DNN

huangzhiyuan commented on a change in pull request #12980: Add reshape op supported by MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/12980#discussion_r238938087 ## File path: src/operator/tensor/matrix_op.cc ## @@ -261,7 +317,9 @@ Example::

[GitHub] ZhennanQin commented on a change in pull request #12980: Add reshape op supported by MKL-DNN

ZhennanQin commented on a change in pull request #12980: Add reshape op supported by MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/12980#discussion_r238931461 ## File path: src/operator/tensor/matrix_op.cc ## @@ -261,7 +317,9 @@ Example::

[GitHub] ZhennanQin commented on a change in pull request #12980: Add reshape op supported by MKL-DNN

ZhennanQin commented on a change in pull request #12980: Add reshape op supported by MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/12980#discussion_r238931410 ## File path: src/operator/tensor/matrix_op.cc ## @@ -103,6 +103,57 @@

[GitHub] sernaton edited a comment on issue #8438: HybridBlocks don't work with constant NDArray initialized Parameters

sernaton edited a comment on issue #8438: HybridBlocks don't work with constant NDArray initialized Parameters URL: https://github.com/apache/incubator-mxnet/issues/8438#issuecomment-444350432 correction: the workaround did work after some tweaking, An example: ``` filter_mask = ...

[GitHub] sernaton commented on issue #8438: HybridBlocks don't work with constant NDArray initialized Parameters

sernaton commented on issue #8438: HybridBlocks don't work with constant NDArray initialized Parameters URL: https://github.com/apache/incubator-mxnet/issues/8438#issuecomment-444350432 correction: the workaround did work after some tweaking, An example: ``` filter_mask = ... # an

[GitHub] yuxihu commented on a change in pull request #13543: Chi_square_check for discrete distribution fix

yuxihu commented on a change in pull request #13543: Chi_square_check for discrete distribution fix URL: https://github.com/apache/incubator-mxnet/pull/13543#discussion_r238923920 ## File path: python/mxnet/test_utils.py ## @@ -1915,8 +1915,8 @@ def

[GitHub] yuxihu commented on a change in pull request #13543: Chi_square_check for discrete distribution fix

yuxihu commented on a change in pull request #13543: Chi_square_check for discrete distribution fix URL: https://github.com/apache/incubator-mxnet/pull/13543#discussion_r238923621 ## File path: python/mxnet/test_utils.py ## @@ -1915,8 +1915,8 @@ def

[GitHub] ChaiBapchya commented on issue #13504: flaky test ARMv7 QEMU

ChaiBapchya commented on issue #13504: flaky test ARMv7 QEMU URL: https://github.com/apache/incubator-mxnet/issues/13504#issuecomment-444349087 http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Fedge/detail/PR-13541/2/pipeline Yet again.

[GitHub] aaronmarkham commented on issue #13544: turn on Sphinx warnings as errors

aaronmarkham commented on issue #13544: turn on Sphinx warnings as errors URL: https://github.com/apache/incubator-mxnet/pull/13544#issuecomment-444339659 @mli @anirudhacharya @lupesko - Not sure if this is the best time to do this, but we've shored up the current bugs in the docs, so if

[GitHub] aaronmarkham opened a new pull request #13544: turn on Sphinx warnings as errors

aaronmarkham opened a new pull request #13544: turn on Sphinx warnings as errors URL: https://github.com/apache/incubator-mxnet/pull/13544 ## Description ## This PR updates the makefile for docs to trigger sphinx-build with the `-W` switch. This turns on the "warnings as errors" feature

[GitHub] aaronmarkham closed pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings.

aaronmarkham closed pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings. URL: https://github.com/apache/incubator-mxnet/pull/13539 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[incubator-mxnet] branch master updated: [MXAPPS-1020] Clean up some Sphinx warnings. (#13539)

This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 d2102fa [MXAPPS-1020] Clean up

[GitHub] zhaoyao73 commented on issue #13538: website document bug

zhaoyao73 commented on issue #13538: website document bug URL: https://github.com/apache/incubator-mxnet/issues/13538#issuecomment-444336414 > Hi @zhaoyao73, thanks for pointing out the C API rendering issues! Do you know how we can fix them? It would be great if you are willing to

[GitHub] aaronmarkham commented on a change in pull request #13541: Updated docs for randint operator

aaronmarkham commented on a change in pull request #13541: Updated docs for randint operator URL: https://github.com/apache/incubator-mxnet/pull/13541#discussion_r238911735 ## File path: docs/api/python/symbol/random.md ## @@ -37,6 +37,7 @@ In the rest of this document,

[GitHub] aaronmarkham commented on a change in pull request #13541: Updated docs for randint operator

aaronmarkham commented on a change in pull request #13541: Updated docs for randint operator URL: https://github.com/apache/incubator-mxnet/pull/13541#discussion_r238911671 ## File path: docs/api/python/ndarray/ndarray.md ## @@ -596,6 +596,7 @@ The `ndarray` package

[GitHub] aaronmarkham commented on a change in pull request #13541: Updated docs for randint operator

aaronmarkham commented on a change in pull request #13541: Updated docs for randint operator URL: https://github.com/apache/incubator-mxnet/pull/13541#discussion_r238911705 ## File path: docs/api/python/ndarray/random.md ## @@ -37,6 +37,7 @@ In the rest of this document,

[GitHub] aaronmarkham commented on a change in pull request #13541: Updated docs for randint operator

aaronmarkham commented on a change in pull request #13541: Updated docs for randint operator URL: https://github.com/apache/incubator-mxnet/pull/13541#discussion_r238911759 ## File path: docs/api/python/symbol/symbol.md ## @@ -595,6 +595,7 @@ Composite multiple symbols

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

This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 0f85f5d doc fix (#13465)

[GitHub] aaronmarkham closed pull request #13465: LBSGD documentation fix

aaronmarkham closed pull request #13465: LBSGD documentation fix URL: https://github.com/apache/incubator-mxnet/pull/13465 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] zhaoyao73 commented on a change in pull request #13496: Optimize C++ API

zhaoyao73 commented on a change in pull request #13496: Optimize C++ API URL: https://github.com/apache/incubator-mxnet/pull/13496#discussion_r238908143 ## File path: cpp-package/example/utils.h ## @@ -42,7 +42,7 @@ bool check_datafiles(const std::vector _files) {

[GitHub] jiajinyu commented on issue #12958: Improve dot(csr, rsp) on CPU by 10x

jiajinyu commented on issue #12958: Improve dot(csr, rsp) on CPU by 10x URL: https://github.com/apache/incubator-mxnet/pull/12958#issuecomment-444329650 pinging for review @eric-haibin-lin @apeforest Guys, I would like to get your feedbacks on this PR. Thanks in advance!

[GitHub] Davdi commented on issue #13526: distributed training van.cc Check failed

Davdi commented on issue #13526: distributed training van.cc Check failed URL: https://github.com/apache/incubator-mxnet/issues/13526#issuecomment-444329646 > @Davdi Looks like the build instruction is not right. Have you followed this one? >

[GitHub] TaoLv commented on issue #13530: Integrate MKLDNN Conv1d and support 3d layout

TaoLv commented on issue #13530: Integrate MKLDNN Conv1d and support 3d layout URL: https://github.com/apache/incubator-mxnet/pull/13530#issuecomment-444325569 @mxnet-label-bot add [MKLDNN, Operator, pr-awaiting-review] This

[GitHub] andrewfayres commented on a change in pull request #13522: [MXNET-1249] Fix Object Detector Performance with GPU

andrewfayres commented on a change in pull request #13522: [MXNET-1249] Fix Object Detector Performance with GPU URL: https://github.com/apache/incubator-mxnet/pull/13522#discussion_r238903215 ## File path: scala-package/infer/src/main/scala/org/apache/mxnet/infer/Classifier.scala

[GitHub] andrewfayres commented on a change in pull request #13522: [MXNET-1249] Fix Object Detector Performance with GPU

andrewfayres commented on a change in pull request #13522: [MXNET-1249] Fix Object Detector Performance with GPU URL: https://github.com/apache/incubator-mxnet/pull/13522#discussion_r238903384 ## File path:

[GitHub] Davdi commented on issue #13526: distributed training van.cc Check failed

Davdi commented on issue #13526: distributed training van.cc Check failed URL: https://github.com/apache/incubator-mxnet/issues/13526#issuecomment-444328924 > @Davdi your make instruction should use USE_DIST_KVSTORE=1 I guess you probably did it, else you would

[GitHub] TccccD commented on a change in pull request #12440: Add stable nrm2 for L2 normalization

TD commented on a change in pull request #12440: Add stable nrm2 for L2 normalization URL: https://github.com/apache/incubator-mxnet/pull/12440#discussion_r238906809 ## File path: src/operator/l2_normalization-inl.h ## @@ -87,22 +88,24 @@ class L2NormalizationOp :

[GitHub] TccccD commented on a change in pull request #12440: Add stable nrm2 for L2 normalization

TD commented on a change in pull request #12440: Add stable nrm2 for L2 normalization URL: https://github.com/apache/incubator-mxnet/pull/12440#discussion_r238906809 ## File path: src/operator/l2_normalization-inl.h ## @@ -87,22 +88,24 @@ class L2NormalizationOp :

[GitHub] Vikas89 commented on issue #13526: distributed training van.cc Check failed

Vikas89 commented on issue #13526: distributed training van.cc Check failed URL: https://github.com/apache/incubator-mxnet/issues/13526#issuecomment-444328466 @Davdi your make instruction should use USE_DIST_KVSTORE=1 I guess you probably did it, else you

[GitHub] ChaiBapchya opened a new pull request #13543: Chi_square_check for discrete distribution fix

ChaiBapchya opened a new pull request #13543: Chi_square_check for discrete distribution fix URL: https://github.com/apache/incubator-mxnet/pull/13543 ## Description ## check for bucket instead of the bucket index It was incorrectly checking for index ``` for i in

[GitHub] andrewfayres commented on issue #13542: Fixing a 404 in the ubuntu setup doc

andrewfayres commented on issue #13542: Fixing a 404 in the ubuntu setup doc URL: https://github.com/apache/incubator-mxnet/pull/13542#issuecomment-444320843 Good to know. Next time I touch the Scala/Java docs I'll try and remember to go through and add .md to the links.

[GitHub] hellonico commented on issue #13523: #13441 [Clojure] Add Spec Validations for the Random namespace

hellonico commented on issue #13523: #13441 [Clojure] Add Spec Validations for the Random namespace URL: https://github.com/apache/incubator-mxnet/pull/13523#issuecomment-444319916 having a look ... This is an automated

[GitHub] gigasquid commented on issue #13507: [Clojure] Correct the versions in the README so they correspond to the latest maven.org release

gigasquid commented on issue #13507: [Clojure] Correct the versions in the README so they correspond to the latest maven.org release URL: https://github.com/apache/incubator-mxnet/pull/13507#issuecomment-444319698 Also going to wait until

[GitHub] Vikas89 commented on issue #12255: Pretty high cpu load when import mxnet

Vikas89 commented on issue #12255: Pretty high cpu load when import mxnet URL: https://github.com/apache/incubator-mxnet/issues/12255#issuecomment-444319311 This is the code in https://github.com/apache/incubator-mxnet/blob/master/src/operator/operator_tune-inl.h#L359 ``` const auto

[GitHub] gigasquid commented on issue #13523: #13441 [Clojure] Add Spec Validations for the Random namespace

gigasquid commented on issue #13523: #13441 [Clojure] Add Spec Validations for the Random namespace URL: https://github.com/apache/incubator-mxnet/pull/13523#issuecomment-444318804 Thanks for making this better! I ran some of the examples and found some bugs with the improved

[GitHub] andrewfayres opened a new pull request #13542: Fixing a 404 in the ubuntu setup doc

andrewfayres opened a new pull request #13542: Fixing a 404 in the ubuntu setup doc URL: https://github.com/apache/incubator-mxnet/pull/13542 ## Description ## Fixes a 404 linking error in the ubuntu setup doc. The url to the java setup page was incorrect. @lanking520

[GitHub] pengzhao-intel edited a comment on issue #13515: MKLDNN Perplexity Issue

pengzhao-intel edited a comment on issue #13515: MKLDNN Perplexity Issue URL: https://github.com/apache/incubator-mxnet/issues/13515#issuecomment-444318569 @Vikas89 please share how to reproduce the issue? I can't reproduce in local. Do you change the example recently? What kind of

[GitHub] pengzhao-intel commented on issue #13515: MKLDNN Perplexity Issue

pengzhao-intel commented on issue #13515: MKLDNN Perplexity Issue URL: https://github.com/apache/incubator-mxnet/issues/13515#issuecomment-444318569 @Vikas89 please share how to reproduce the issue? I can't reproduce in local. Do you change the example recently? What kind of machine

[GitHub] Vikas89 commented on issue #12255: Pretty high cpu load when import mxnet

Vikas89 commented on issue #12255: Pretty high cpu load when import mxnet URL: https://github.com/apache/incubator-mxnet/issues/12255#issuecomment-444317987 There is huge number of threads getting created in operator-tune-inl.h that causes huge cpu load, threads take long time to

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

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 1d47cae Bump the publish

[GitHub] YutingZhang edited a comment on issue #13521: Gluon DataLoader cannot release the processes in the pool

YutingZhang edited a comment on issue #13521: Gluon DataLoader cannot release the processes in the pool URL: https://github.com/apache/incubator-mxnet/issues/13521#issuecomment-444314392 @zhreshold Great. Thanks! FYI, I tested it on my Mac using anaconda Python3.6. It also caused

[GitHub] stu1130 commented on a change in pull request #13496: Optimize C++ API

stu1130 commented on a change in pull request #13496: Optimize C++ API URL: https://github.com/apache/incubator-mxnet/pull/13496#discussion_r238894857 ## File path: cpp-package/example/utils.h ## @@ -42,7 +42,7 @@ bool check_datafiles(const std::vector _files) { return

[GitHub] YutingZhang commented on issue #13521: Gluon DataLoader cannot release the processes in the pool

YutingZhang commented on issue #13521: Gluon DataLoader cannot release the processes in the pool URL: https://github.com/apache/incubator-mxnet/issues/13521#issuecomment-444314392 @zhreshold Great. Thanks! FYI, I test it on Mac. It also cause problems. Maybe caused by the anaconda

[GitHub] lanking520 commented on issue #13528: Scala jar contains the scala standard library

lanking520 commented on issue #13528: Scala jar contains the scala standard library URL: https://github.com/apache/incubator-mxnet/issues/13528#issuecomment-444312670 In fact, all Scala code are generated to `.class` file when we build the jar and I think it is JVM code. In that case,

[GitHub] lanking520 commented on issue #13538: website document bug

lanking520 commented on issue #13538: website document bug URL: https://github.com/apache/incubator-mxnet/issues/13538#issuecomment-444312399 Hi @zhaoyao73, thanks for pointing out the C API rendering issues! Do you know how we can fix them? It would be great if you are willing to

[GitHub] apeforest commented on issue #13062: [MXNET-1110] Add header files required by horovod

apeforest commented on issue #13062: [MXNET-1110] Add header files required by horovod URL: https://github.com/apache/incubator-mxnet/pull/13062#issuecomment-444312075 @mxnet-label-bot update [pr-awaiting-merge] This is an

[incubator-mxnet] branch master updated: Revert "Feature/mkldnn static 2 (#13503)" (#13540)

This is an automated email from the ASF dual-hosted git repository. anirudh2290 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 a3eca5f Revert "Feature/mkldnn

[GitHub] anirudh2290 closed pull request #13540: Revert "Feature/mkldnn static 2 (#13503)"

anirudh2290 closed pull request #13540: Revert "Feature/mkldnn static 2 (#13503)" URL: https://github.com/apache/incubator-mxnet/pull/13540 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] cjolivier01 commented on issue #12255: Pretty high cpu load when import mxnet

cjolivier01 commented on issue #12255: Pretty high cpu load when import mxnet URL: https://github.com/apache/incubator-mxnet/issues/12255#issuecomment-444310455 OMP has a very high overhead in ec2 instances with 36+ cores. this is a known problem with ec2 instances.

[GitHub] lanking520 commented on issue #13526: distributed training van.cc Check failed

lanking520 commented on issue #13526: distributed training van.cc Check failed URL: https://github.com/apache/incubator-mxnet/issues/13526#issuecomment-444310391 @Davdi Looks like the build instruction is not right. Have you followed this one?

[GitHub] marcoabreu commented on a change in pull request #13450: [WIP][MXNET-862] Basic maven jenkins pipeline

marcoabreu commented on a change in pull request #13450: [WIP][MXNET-862] Basic maven jenkins pipeline URL: https://github.com/apache/incubator-mxnet/pull/13450#discussion_r238889424 ## File path: ci/Jenkinsfile_utils.groovy ## @@ -18,18 +18,23 @@ // under the License.

[GitHub] marcoabreu commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation

marcoabreu commented on a change in pull request #13532: [MXNET-43] Fix Jetson compilation URL: https://github.com/apache/incubator-mxnet/pull/13532#discussion_r238889199 ## File path: ci/docker/Dockerfile.build.jetson ## @@ -77,6 +77,7 @@ RUN

[incubator-mxnet] branch master updated: Update java setup docs for 1.4.0 (#13536)

This is an automated email from the ASF dual-hosted git repository. lanking 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 f7192b2 Update java setup docs for

[GitHub] lanking520 closed pull request #13536: Update java setup docs for 1.4.0

lanking520 closed pull request #13536: Update java setup docs for 1.4.0 URL: https://github.com/apache/incubator-mxnet/pull/13536 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] cjolivier01 commented on issue #12255: Pretty high cpu load when import mxnet

cjolivier01 commented on issue #12255: Pretty high cpu load when import mxnet URL: https://github.com/apache/incubator-mxnet/issues/12255#issuecomment-444306107 if you fork and then try to use OMP in thenforked process and are using libgomp, the process will hang because libgomp does not

[GitHub] lupesko commented on issue #13515: MKLDNN Perplexity Issue

lupesko commented on issue #13515: MKLDNN Perplexity Issue URL: https://github.com/apache/incubator-mxnet/issues/13515#issuecomment-444302263 @Vikas89 there are different ways to implement and train an LSTM-based language model on PTB dataset. Can you please share the training script?

[GitHub] sandeep-krishnamurthy closed issue #12401: cpp_package instructions need clarification and examples need repair

sandeep-krishnamurthy closed issue #12401: cpp_package instructions need clarification and examples need repair URL: https://github.com/apache/incubator-mxnet/issues/12401 This is an automated message from the Apache Git

[GitHub] vishaalkapoor commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings.

vishaalkapoor commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings. URL: https://github.com/apache/incubator-mxnet/pull/13539#discussion_r238877491 ## File path: docs/gluon/index.md ## @@ -82,7 +82,6 @@ def forward(self, F, inputs,

[GitHub] zachgk commented on a change in pull request #13450: [WIP][MXNET-862] Basic maven jenkins pipeline

zachgk commented on a change in pull request #13450: [WIP][MXNET-862] Basic maven jenkins pipeline URL: https://github.com/apache/incubator-mxnet/pull/13450#discussion_r238878873 ## File path: ci/Jenkinsfile_utils.groovy ## @@ -18,18 +18,23 @@ // under the License. //

[GitHub] vishaalkapoor edited a comment on issue #13539: [MXAPPS-1020] Clean up some Sphinx warnings.

vishaalkapoor edited a comment on issue #13539: [MXAPPS-1020] Clean up some Sphinx warnings. URL: https://github.com/apache/incubator-mxnet/pull/13539#issuecomment-444297098 @aaronmarkham Updated as per comments including the relative links. thanks!

[GitHub] vishaalkapoor edited a comment on issue #13539: [MXAPPS-1020] Clean up some Sphinx warnings.

vishaalkapoor edited a comment on issue #13539: [MXAPPS-1020] Clean up some Sphinx warnings. URL: https://github.com/apache/incubator-mxnet/pull/13539#issuecomment-444297098 @aaronmarkham Updated as per comments including the links. thanks!

[GitHub] vishaalkapoor commented on issue #13539: [MXAPPS-1020] Clean up some Sphinx warnings.

vishaalkapoor commented on issue #13539: [MXAPPS-1020] Clean up some Sphinx warnings. URL: https://github.com/apache/incubator-mxnet/pull/13539#issuecomment-444297098 @aaronmarkham Updated as per comments. thanks! This is an

[GitHub] vishaalkapoor commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings.

vishaalkapoor commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings. URL: https://github.com/apache/incubator-mxnet/pull/13539#discussion_r238877518 ## File path: src/operator/contrib/dgl_graph.cc ## @@ -795,7 +795,8 @@ uniform

[GitHub] vishaalkapoor commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings.

vishaalkapoor commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings. URL: https://github.com/apache/incubator-mxnet/pull/13539#discussion_r238877491 ## File path: docs/gluon/index.md ## @@ -82,7 +82,6 @@ def forward(self, F, inputs,

[GitHub] vishaalkapoor commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings.

vishaalkapoor commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings. URL: https://github.com/apache/incubator-mxnet/pull/13539#discussion_r238876881 ## File path: docs/faq/develop_and_hack.md ## @@ -5,4 +5,4 @@ # Other Resources -

[GitHub] vishaalkapoor commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings.

vishaalkapoor commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings. URL: https://github.com/apache/incubator-mxnet/pull/13539#discussion_r238876881 ## File path: docs/faq/develop_and_hack.md ## @@ -5,4 +5,4 @@ # Other Resources -

[GitHub] aaronmarkham commented on issue #13539: [MXAPPS-1020] Clean up some Sphinx warnings.

aaronmarkham commented on issue #13539: [MXAPPS-1020] Clean up some Sphinx warnings. URL: https://github.com/apache/incubator-mxnet/pull/13539#issuecomment-444294993 Ok, we're good to remove those entries on the symbol page.

[GitHub] eric-haibin-lin commented on issue #13436: add graph_compact.

eric-haibin-lin commented on issue #13436: add graph_compact. URL: https://github.com/apache/incubator-mxnet/pull/13436#issuecomment-444294507 @aaronmarkham good catch. No concern from my side. Thanks for pointing that out!

[GitHub] aaronmarkham commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings.

aaronmarkham commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings. URL: https://github.com/apache/incubator-mxnet/pull/13539#discussion_r238872884 ## File path: docs/api/python/symbol/contrib.md ## @@ -55,9 +55,6 @@ In the rest of this

[GitHub] andrewfayres commented on issue #13536: Update java setup docs for 1.4.0

andrewfayres commented on issue #13536: Update java setup docs for 1.4.0 URL: https://github.com/apache/incubator-mxnet/pull/13536#issuecomment-444293119 It covers all of them I found. I also tested the java-demo after the change to verify that it still works.

[GitHub] aaronmarkham commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings.

aaronmarkham commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings. URL: https://github.com/apache/incubator-mxnet/pull/13539#discussion_r238873275 ## File path: docs/gluon/index.md ## @@ -82,7 +82,6 @@ def forward(self, F, inputs,

[GitHub] aaronmarkham commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings.

aaronmarkham commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings. URL: https://github.com/apache/incubator-mxnet/pull/13539#discussion_r238873429 ## File path: src/operator/contrib/dgl_graph.cc ## @@ -795,7 +795,8 @@ uniform

[GitHub] aaronmarkham commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings.

aaronmarkham commented on a change in pull request #13539: [MXAPPS-1020] Clean up some Sphinx warnings. URL: https://github.com/apache/incubator-mxnet/pull/13539#discussion_r238873121 ## File path: docs/faq/develop_and_hack.md ## @@ -5,4 +5,4 @@ # Other Resources -

[GitHub] ChaiBapchya opened a new pull request #13541: Updated docs for randint operator

ChaiBapchya opened a new pull request #13541: Updated docs for randint operator URL: https://github.com/apache/incubator-mxnet/pull/13541 ## Description ## Title This is an automated message from the Apache Git

  1   2   3   >