[GitHub] jasonyu1996 commented on issue #9182: Optimizer wish list

2018-08-13 Thread GitBox
jasonyu1996 commented on issue #9182: Optimizer wish list URL: https://github.com/apache/incubator-mxnet/issues/9182#issuecomment-412763145 - [ ] Limited-memory BFGS (L-BFGS) This is an automated message from the Apache Git

[GitHub] pengzhao-intel commented on issue #12069: [Flaky Test] Fix test_gluon_model_zoo.test_models when MXNET_MKLDNN_DEBUG=1

2018-08-13 Thread GitBox
pengzhao-intel commented on issue #12069: [Flaky Test] Fix test_gluon_model_zoo.test_models when MXNET_MKLDNN_DEBUG=1 URL: https://github.com/apache/incubator-mxnet/pull/12069#issuecomment-412762508 LGTM, thanks for the fix :)

[GitHub] reminisce opened a new pull request #12157: Subgraph API for integrating accelerators with MXNet

2018-08-13 Thread GitBox
reminisce opened a new pull request #12157: Subgraph API for integrating accelerators with MXNet URL: https://github.com/apache/incubator-mxnet/pull/12157 ## Description ## This is a joint work of @reminisce and @zheng-da on implementing subgraph API for integrating backend

[GitHub] zheng-da commented on issue #12151: fix a minor bug in while_loop

2018-08-13 Thread GitBox
zheng-da commented on issue #12151: fix a minor bug in while_loop URL: https://github.com/apache/incubator-mxnet/pull/12151#issuecomment-412754808 i can try. This is an automated message from the Apache Git Service. To

[GitHub] nuaawzl commented on issue #12120: compile error in "image_aug_default.cc"

2018-08-13 Thread GitBox
nuaawzl commented on issue #12120: compile error in "image_aug_default.cc" URL: https://github.com/apache/incubator-mxnet/issues/12120#issuecomment-412754189 Hi, you can try do something like this: 1 image_augmenter.h ``` #if MXNET_USE_OPENCV #include +#include

[GitHub] azai91 commented on issue #12069: [Flaky Test] Fix test_gluon_model_zoo.test_models when MXNET_MKLDNN_DEBUG=1

2018-08-13 Thread GitBox
azai91 commented on issue #12069: [Flaky Test] Fix test_gluon_model_zoo.test_models when MXNET_MKLDNN_DEBUG=1 URL: https://github.com/apache/incubator-mxnet/pull/12069#issuecomment-412753907 @pengzhao-intel review/approve please

[GitHub] azai91 commented on issue #12058: MKLDNN can be turned off with env var

2018-08-13 Thread GitBox
azai91 commented on issue #12058: MKLDNN can be turned off with env var URL: https://github.com/apache/incubator-mxnet/pull/12058#issuecomment-412753812 @pengzhao-intel review / approve please This is an automated message

[GitHub] nuaawzl removed a comment on issue #12120: compile error in "image_aug_default.cc"

2018-08-13 Thread GitBox
nuaawzl removed a comment on issue #12120: compile error in "image_aug_default.cc" URL: https://github.com/apache/incubator-mxnet/issues/12120#issuecomment-412753614 Hi, you can try do something like this: 1 image_augmenter.h #if MXNET_USE_OPENCV #include + #include

[GitHub] nuaawzl commented on issue #12120: compile error in "image_aug_default.cc"

2018-08-13 Thread GitBox
nuaawzl commented on issue #12120: compile error in "image_aug_default.cc" URL: https://github.com/apache/incubator-mxnet/issues/12120#issuecomment-412753614 Hi, you can try do something like this: 1 image_augmenter.h #if MXNET_USE_OPENCV #include + #include

[GitHub] nuaawzl removed a comment on issue #12120: compile error in "image_aug_default.cc"

2018-08-13 Thread GitBox
nuaawzl removed a comment on issue #12120: compile error in "image_aug_default.cc" URL: https://github.com/apache/incubator-mxnet/issues/12120#issuecomment-412753259 Hi, you can try do something like this: > image_augmenter.h ... #if MXNET_USE_OPENCV #include +

[GitHub] nuaawzl commented on issue #12120: compile error in "image_aug_default.cc"

2018-08-13 Thread GitBox
nuaawzl commented on issue #12120: compile error in "image_aug_default.cc" URL: https://github.com/apache/incubator-mxnet/issues/12120#issuecomment-412753259 Hi, you can try do something like this: > image_augmenter.h ... #if MXNET_USE_OPENCV #include + #include

[GitHub] anirudhacharya closed issue #11496: Selu Operator not present in MXNet

2018-08-13 Thread GitBox
anirudhacharya closed issue #11496: Selu Operator not present in MXNet URL: https://github.com/apache/incubator-mxnet/issues/11496 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] reminisce closed pull request #12104: [DO NOT REVIEW] Subgraph API

2018-08-13 Thread GitBox
reminisce closed pull request #12104: [DO NOT REVIEW] Subgraph API URL: https://github.com/apache/incubator-mxnet/pull/12104 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: set bind flag after bind completes (#12155)

2018-08-13 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new 92edca8 set bind flag after bind

[GitHub] szha closed pull request #12155: set bind flag after bind completes

2018-08-13 Thread GitBox
szha closed pull request #12155: set bind flag after bind completes URL: https://github.com/apache/incubator-mxnet/pull/12155 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] anirudhacharya edited a comment on issue #12132: [Feature Request] Implementation of L-BFGS optimizer

2018-08-13 Thread GitBox
anirudhacharya edited a comment on issue #12132: [Feature Request] Implementation of L-BFGS optimizer URL: https://github.com/apache/incubator-mxnet/issues/12132#issuecomment-412420766 MXNet does not have this optimizer( [see

[GitHub] vdantu commented on issue #12145: syncbatchnorm for 3d cnn

2018-08-13 Thread GitBox
vdantu commented on issue #12145: syncbatchnorm for 3d cnn URL: https://github.com/apache/incubator-mxnet/issues/12145#issuecomment-412715610 @mxnet-label-bot [Gluon, Question] This is an automated message from the Apache

[GitHub] nswamy commented on a change in pull request #11844: [MXNET-689] add DataDesc type for the Scala Package

2018-08-13 Thread GitBox
nswamy commented on a change in pull request #11844: [MXNET-689] add DataDesc type for the Scala Package URL: https://github.com/apache/incubator-mxnet/pull/11844#discussion_r209801084 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/io/NDArrayIter.scala

[GitHub] nswamy commented on a change in pull request #11844: [MXNET-689] add DataDesc type for the Scala Package

2018-08-13 Thread GitBox
nswamy commented on a change in pull request #11844: [MXNET-689] add DataDesc type for the Scala Package URL: https://github.com/apache/incubator-mxnet/pull/11844#discussion_r209801084 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/io/NDArrayIter.scala

[GitHub] nswamy commented on a change in pull request #11844: [MXNET-689] add DataDesc type for the Scala Package

2018-08-13 Thread GitBox
nswamy commented on a change in pull request #11844: [MXNET-689] add DataDesc type for the Scala Package URL: https://github.com/apache/incubator-mxnet/pull/11844#discussion_r209797874 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/io/NDArrayIter.scala

[GitHub] nswamy commented on a change in pull request #11844: [MXNET-689] add DataDesc type for the Scala Package

2018-08-13 Thread GitBox
nswamy commented on a change in pull request #11844: [MXNET-689] add DataDesc type for the Scala Package URL: https://github.com/apache/incubator-mxnet/pull/11844#discussion_r209798359 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/IO.scala ## @@ -352,7

[GitHub] nswamy commented on a change in pull request #11844: [MXNET-689] add DataDesc type for the Scala Package

2018-08-13 Thread GitBox
nswamy commented on a change in pull request #11844: [MXNET-689] add DataDesc type for the Scala Package URL: https://github.com/apache/incubator-mxnet/pull/11844#discussion_r209798416 ## File path: scala-package/core/src/main/scala/org/apache/mxnet/Layout.scala ## @@

[GitHub] pengzhao-intel commented on issue #11664: Fall back when sparse arrays are passed to MKLDNN-enabled operators

2018-08-13 Thread GitBox
pengzhao-intel commented on issue #11664: Fall back when sparse arrays are passed to MKLDNN-enabled operators URL: https://github.com/apache/incubator-mxnet/pull/11664#issuecomment-412712965 @marcoabreu any other comments? Could you help to merge? It's a usability improvement and I

[GitHub] vdantu commented on issue #12154: the cond operator doesn't support a function that doesn't require inputs.

2018-08-13 Thread GitBox
vdantu commented on issue #12154: the cond operator doesn't support a function that doesn't require inputs. URL: https://github.com/apache/incubator-mxnet/issues/12154#issuecomment-412711570 @mxnet-label-bot [Python, Question]

[GitHub] vdantu commented on issue #12140: Impossible to provide arguments to random_normal in scala ?

2018-08-13 Thread GitBox
vdantu commented on issue #12140: Impossible to provide arguments to random_normal in scala ? URL: https://github.com/apache/incubator-mxnet/issues/12140#issuecomment-412710470 @mxnet-label-bot [Scala, Question] This is an

[GitHub] azai91 opened a new pull request #12156: [Incomplete] Add randn to random module

2018-08-13 Thread GitBox
azai91 opened a new pull request #12156: [Incomplete] Add randn to random module URL: https://github.com/apache/incubator-mxnet/pull/12156 ## Description ## Added a randn method to random module. numpy currently has support for a random tensor generator that accepts a shape and outputs

[GitHub] nswamy commented on issue #11844: [MXNET-689] add DataDesc type for the Scala Package

2018-08-13 Thread GitBox
nswamy commented on issue #11844: [MXNET-689] add DataDesc type for the Scala Package URL: https://github.com/apache/incubator-mxnet/pull/11844#issuecomment-412707509 This PR adds quite a lot of new APIs, please add corresponding tests.

[GitHub] zheng-da commented on issue #11664: Fall back when sparse arrays are passed to MKLDNN-enabled operators

2018-08-13 Thread GitBox
zheng-da commented on issue #11664: Fall back when sparse arrays are passed to MKLDNN-enabled operators URL: https://github.com/apache/incubator-mxnet/pull/11664#issuecomment-412706348 thanks for providing the performance results.

[GitHub] zheng-da commented on a change in pull request #11831: [MXNET-484] MKLDNN C++ test for LRN operator

2018-08-13 Thread GitBox
zheng-da commented on a change in pull request #11831: [MXNET-484] MKLDNN C++ test for LRN operator URL: https://github.com/apache/incubator-mxnet/pull/11831#discussion_r209792689 ## File path: tests/cpp/operator/mkldnn.cc ## @@ -1094,6 +1191,99 @@ void TestConcatOp(const

[GitHub] zheng-da commented on a change in pull request #11831: [MXNET-484] MKLDNN C++ test for LRN operator

2018-08-13 Thread GitBox
zheng-da commented on a change in pull request #11831: [MXNET-484] MKLDNN C++ test for LRN operator URL: https://github.com/apache/incubator-mxnet/pull/11831#discussion_r209792297 ## File path: tests/cpp/operator/mkldnn.cc ## @@ -1094,6 +1191,99 @@ void TestConcatOp(const

[GitHub] zheng-da commented on a change in pull request #11831: [MXNET-484] MKLDNN C++ test for LRN operator

2018-08-13 Thread GitBox
zheng-da commented on a change in pull request #11831: [MXNET-484] MKLDNN C++ test for LRN operator URL: https://github.com/apache/incubator-mxnet/pull/11831#discussion_r209792228 ## File path: tests/cpp/operator/mkldnn.cc ## @@ -1094,6 +1191,99 @@ void TestConcatOp(const

[GitHub] zheng-da commented on a change in pull request #11831: [MXNET-484] MKLDNN C++ test for LRN operator

2018-08-13 Thread GitBox
zheng-da commented on a change in pull request #11831: [MXNET-484] MKLDNN C++ test for LRN operator URL: https://github.com/apache/incubator-mxnet/pull/11831#discussion_r209790288 ## File path: src/operator/nn/mkldnn/mkldnn_lrn-inl.h ## @@ -229,6 +232,7 @@ void

[GitHub] zheng-da commented on a change in pull request #11831: [MXNET-484] MKLDNN C++ test for LRN operator

2018-08-13 Thread GitBox
zheng-da commented on a change in pull request #11831: [MXNET-484] MKLDNN C++ test for LRN operator URL: https://github.com/apache/incubator-mxnet/pull/11831#discussion_r209790531 ## File path: tests/cpp/operator/mkldnn.cc ## @@ -535,6 +552,40 @@ void PrintVerifyMsg(const

[GitHub] zheng-da commented on a change in pull request #11831: [MXNET-484] MKLDNN C++ test for LRN operator

2018-08-13 Thread GitBox
zheng-da commented on a change in pull request #11831: [MXNET-484] MKLDNN C++ test for LRN operator URL: https://github.com/apache/incubator-mxnet/pull/11831#discussion_r209791360 ## File path: tests/cpp/operator/mkldnn.cc ## @@ -610,8 +670,12 @@ std::vector

[GitHub] zheng-da commented on a change in pull request #11831: [MXNET-484] MKLDNN C++ test for LRN operator

2018-08-13 Thread GitBox
zheng-da commented on a change in pull request #11831: [MXNET-484] MKLDNN C++ test for LRN operator URL: https://github.com/apache/incubator-mxnet/pull/11831#discussion_r209790490 ## File path: tests/cpp/operator/mkldnn.cc ## @@ -535,6 +552,40 @@ void PrintVerifyMsg(const

[GitHub] sandeep-krishnamurthy closed issue #2841: Has the high level API for autoencoder in R implemented?

2018-08-13 Thread GitBox
sandeep-krishnamurthy closed issue #2841: Has the high level API for autoencoder in R implemented? URL: https://github.com/apache/incubator-mxnet/issues/2841 This is an automated message from the Apache Git Service. To

[GitHub] azai91 opened a new pull request #12155: set bind flag after bind completes

2018-08-13 Thread GitBox
azai91 opened a new pull request #12155: set bind flag after bind completes URL: https://github.com/apache/incubator-mxnet/pull/12155 ## Description ## Currently the bind flag under the module API is set in the beginning of the function. However, if any of the downstream functions error

[GitHub] zheng-da commented on a change in pull request #12104: [DO NOT REVIEW] Subgraph API

2018-08-13 Thread GitBox
zheng-da commented on a change in pull request #12104: [DO NOT REVIEW] Subgraph API URL: https://github.com/apache/incubator-mxnet/pull/12104#discussion_r209788547 ## File path: src/operator/subgraph/subgraph_property.h ## @@ -0,0 +1,132 @@ +/* + * Licensed to the Apache

[GitHub] zheng-da commented on a change in pull request #12104: [DO NOT REVIEW] Subgraph API

2018-08-13 Thread GitBox
zheng-da commented on a change in pull request #12104: [DO NOT REVIEW] Subgraph API URL: https://github.com/apache/incubator-mxnet/pull/12104#discussion_r209788603 ## File path: src/operator/subgraph/default_subgraph_property.h ## @@ -0,0 +1,81 @@ +/* + * Licensed to the

[GitHub] zheng-da commented on issue #12154: the cond operator doesn't support a function that doesn't require inputs.

2018-08-13 Thread GitBox
zheng-da commented on issue #12154: the cond operator doesn't support a function that doesn't require inputs. URL: https://github.com/apache/incubator-mxnet/issues/12154#issuecomment-412695428 @junrushao1994 This is an

[GitHub] zheng-da commented on issue #12154: the cond operator doesn't support a function that doesn't require inputs.

2018-08-13 Thread GitBox
zheng-da commented on issue #12154: the cond operator doesn't support a function that doesn't require inputs. URL: https://github.com/apache/incubator-mxnet/issues/12154#issuecomment-412695405 The reason that the cond operator fails is that CachedOp, which the cond operator uses to

[GitHub] zheng-da opened a new issue #12154: the cond operator doesn't support a function that doesn't require inputs.

2018-08-13 Thread GitBox
zheng-da opened a new issue #12154: the cond operator doesn't support a function that doesn't require inputs. URL: https://github.com/apache/incubator-mxnet/issues/12154 The python functions of the cond operator has to access some variables outside the function. ```python def

[GitHub] anirudhacharya commented on issue #2841: Has the high level API for autoencoder in R implemented?

2018-08-13 Thread GitBox
anirudhacharya commented on issue #2841: Has the high level API for autoencoder in R implemented? URL: https://github.com/apache/incubator-mxnet/issues/2841#issuecomment-412688387 @sandeep-krishnamurthy please close this, it is being tracked here -

[GitHub] anirudhacharya commented on issue #11924: Compile

2018-08-13 Thread GitBox
anirudhacharya commented on issue #11924: Compile URL: https://github.com/apache/incubator-mxnet/issues/11924#issuecomment-412687729 @sandeep-krishnamurthy please close this due to lack of activity. This is an automated

[GitHub] mdespriee commented on issue #12140: Impossible to provide arguments to random_normal in scala ?

2018-08-13 Thread GitBox
mdespriee commented on issue #12140: Impossible to provide arguments to random_normal in scala ? URL: https://github.com/apache/incubator-mxnet/issues/12140#issuecomment-412679734 @lanking520 Oh wait, I think I understand. Actually, I'm interested in this:

[GitHub] yuxiangw opened a new pull request #12153: documentation changes. added full reference

2018-08-13 Thread GitBox
yuxiangw opened a new pull request #12153: documentation changes. added full reference URL: https://github.com/apache/incubator-mxnet/pull/12153 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### Please feel free to remove

[GitHub] lanking520 commented on issue #12140: Impossible to provide arguments to random_normal in scala ?

2018-08-13 Thread GitBox
lanking520 commented on issue #12140: Impossible to provide arguments to random_normal in scala ? URL: https://github.com/apache/incubator-mxnet/issues/12140#issuecomment-412668354 @mdespriee looks like you are using it in a wrong way, are you trying to use it in Symbolic way? Here is

[GitHub] eric-haibin-lin commented on issue #12150: wrong reference to dev list on mxnet.io

2018-08-13 Thread GitBox
eric-haibin-lin commented on issue #12150: wrong reference to dev list on mxnet.io URL: https://github.com/apache/incubator-mxnet/issues/12150#issuecomment-412668183 There's a fix for this https://github.com/apache/incubator-mxnet/pull/11858 but the CI is not reporting status, blocking

[GitHub] zhreshold commented on issue #11872: "socket.error: [Errno 111] Connection refused" while training with multiple workers

2018-08-13 Thread GitBox
zhreshold commented on issue #11872: "socket.error: [Errno 111] Connection refused" while training with multiple workers URL: https://github.com/apache/incubator-mxnet/issues/11872#issuecomment-412667553 With #11908 been merged, I am closing this for now. Feel free to ping me if it still

[GitHub] zhreshold closed issue #11872: "socket.error: [Errno 111] Connection refused" while training with multiple workers

2018-08-13 Thread GitBox
zhreshold closed issue #11872: "socket.error: [Errno 111] Connection refused" while training with multiple workers URL: https://github.com/apache/incubator-mxnet/issues/11872 This is an automated message from the Apache Git

[GitHub] nswamy closed pull request #12148: Fix RAT license checker which is broken in trunk

2018-08-13 Thread GitBox
nswamy closed pull request #12148: Fix RAT license checker which is broken in trunk URL: https://github.com/apache/incubator-mxnet/pull/12148 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] vandanavk opened a new pull request #12152: [MXNET-696] Fix profiler executer when memonger is used

2018-08-13 Thread GitBox
vandanavk opened a new pull request #12152: [MXNET-696] Fix profiler executer when memonger is used URL: https://github.com/apache/incubator-mxnet/pull/12152 ## Description ## Pylint throws an error for variables that are used but not defined previously. Since these variables are found

[GitHub] larroy commented on issue #12107: Faulty CUDA detection with cmake

2018-08-13 Thread GitBox
larroy commented on issue #12107: Faulty CUDA detection with cmake URL: https://github.com/apache/incubator-mxnet/issues/12107#issuecomment-412664530 3.10.0 seems like a super new version of cmake but is definitely worth considering if it simplifies the build so much.

[GitHub] stu1130 commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter

2018-08-13 Thread GitBox
stu1130 commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter URL: https://github.com/apache/incubator-mxnet/pull/12131#discussion_r209756855 ## File path: python/mxnet/image/image.py ## @@ -1059,16 +1059,21 @@ class

[GitHub] stu1130 commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter

2018-08-13 Thread GitBox
stu1130 commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter URL: https://github.com/apache/incubator-mxnet/pull/12131#discussion_r209756855 ## File path: python/mxnet/image/image.py ## @@ -1059,16 +1059,21 @@ class

[GitHub] lupesko commented on issue #12150: wrong reference to dev list on mxnet.io

2018-08-13 Thread GitBox
lupesko commented on issue #12150: wrong reference to dev list on mxnet.io URL: https://github.com/apache/incubator-mxnet/issues/12150#issuecomment-412660485 @mxnet-label-bot please add [Doc, Bug]. This is an automated

[GitHub] stu1130 commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter

2018-08-13 Thread GitBox
stu1130 commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter URL: https://github.com/apache/incubator-mxnet/pull/12131#discussion_r209754983 ## File path: python/mxnet/image/image.py ## @@ -1059,16 +1059,21 @@ class

[GitHub] zheng-da commented on issue #12151: fix a minor bug in while_loop

2018-08-13 Thread GitBox
zheng-da commented on issue #12151: fix a minor bug in while_loop URL: https://github.com/apache/incubator-mxnet/pull/12151#issuecomment-412658001 @junrushao1994 please review this. This is an automated message from the

[GitHub] zheng-da opened a new pull request #12151: fix a minor bug in while_loop

2018-08-13 Thread GitBox
zheng-da opened a new pull request #12151: fix a minor bug in while_loop URL: https://github.com/apache/incubator-mxnet/pull/12151 ## Description ## We don't need to make sure all loop variables are used in the loop body. Sometimes, some of the loop variables used in the condition, but

[GitHub] eric-haibin-lin commented on issue #12091: [MXNET-792] Fix for issue #9816 with dropout operator and RNG

2018-08-13 Thread GitBox
eric-haibin-lin commented on issue #12091: [MXNET-792] Fix for issue #9816 with dropout operator and RNG URL: https://github.com/apache/incubator-mxnet/pull/12091#issuecomment-412632483 This will result in incorrect values when ratio=1 on cpu. let's fix that

[GitHub] zhanghang1989 commented on issue #12145: syncbatchnorm for 3d cnn

2018-08-13 Thread GitBox
zhanghang1989 commented on issue #12145: syncbatchnorm for 3d cnn URL: https://github.com/apache/incubator-mxnet/issues/12145#issuecomment-412619824 you could reshape the data to (b,c,h,w) and then reshape back. This is an

[GitHub] eric-haibin-lin commented on issue #12145: syncbatchnorm for 3d cnn

2018-08-13 Thread GitBox
eric-haibin-lin commented on issue #12145: syncbatchnorm for 3d cnn URL: https://github.com/apache/incubator-mxnet/issues/12145#issuecomment-412618778 @zhanghang1989 This is an automated message from the Apache Git Service.

[GitHub] eric-haibin-lin closed issue #11899: non-deterministic backward of scatter_nd

2018-08-13 Thread GitBox
eric-haibin-lin closed issue #11899: non-deterministic backward of scatter_nd URL: https://github.com/apache/incubator-mxnet/issues/11899 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] srochel opened a new issue #12150: wrong reference to dev list on mxnet.io

2018-08-13 Thread GitBox
srochel opened a new issue #12150: wrong reference to dev list on mxnet.io URL: https://github.com/apache/incubator-mxnet/issues/12150 Reference to dev list at http://mxnet.incubator.apache.org/community/contribute.html#mxnet-dev-communications for dev list correct reference is:

[GitHub] vandanavk commented on a change in pull request #12137: [MXNET-696] Fix undefined name errors

2018-08-13 Thread GitBox
vandanavk commented on a change in pull request #12137: [MXNET-696] Fix undefined name errors URL: https://github.com/apache/incubator-mxnet/pull/12137#discussion_r209709514 ## File path: example/profiler/profiler_executor.py ## @@ -84,9 +84,7 @@ def get_symbol():

[GitHub] Roshrini commented on issue #12057: Error In Loading Pretrained ShuffleNet ONNX Model

2018-08-13 Thread GitBox
Roshrini commented on issue #12057: Error In Loading Pretrained ShuffleNet ONNX Model URL: https://github.com/apache/incubator-mxnet/issues/12057#issuecomment-412615386 Duplicate issue: https://github.com/apache/incubator-mxnet/issues/12018

[GitHub] Roshrini closed issue #12057: Error In Loading Pretrained ShuffleNet ONNX Model

2018-08-13 Thread GitBox
Roshrini closed issue #12057: Error In Loading Pretrained ShuffleNet ONNX Model URL: https://github.com/apache/incubator-mxnet/issues/12057 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] zhreshold commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter

2018-08-13 Thread GitBox
zhreshold commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter URL: https://github.com/apache/incubator-mxnet/pull/12131#discussion_r209700313 ## File path: python/mxnet/image/image.py ## @@ -1059,16 +1059,21 @@ class

[GitHub] zhreshold commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter

2018-08-13 Thread GitBox
zhreshold commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter URL: https://github.com/apache/incubator-mxnet/pull/12131#discussion_r209706537 ## File path: python/mxnet/image/image.py ## @@ -1207,8 +1244,31 @@ def next(self):

[GitHub] zhreshold commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter

2018-08-13 Thread GitBox
zhreshold commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter URL: https://github.com/apache/incubator-mxnet/pull/12131#discussion_r209700757 ## File path: python/mxnet/image/image.py ## @@ -1059,16 +1059,21 @@ class

[GitHub] zhreshold commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter

2018-08-13 Thread GitBox
zhreshold commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter URL: https://github.com/apache/incubator-mxnet/pull/12131#discussion_r209701223 ## File path: python/mxnet/image/image.py ## @@ -1149,22 +1157,53 @@ def

[GitHub] zhreshold commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter

2018-08-13 Thread GitBox
zhreshold commented on a change in pull request #12131: [MXNET-737][WIP] Add last batch handle for imageiter URL: https://github.com/apache/incubator-mxnet/pull/12131#discussion_r209699823 ## File path: python/mxnet/image/image.py ## @@ -1059,16 +1059,21 @@ class

[GitHub] szha commented on a change in pull request #12064: Allow stop of arange to be inferred from dims.

2018-08-13 Thread GitBox
szha commented on a change in pull request #12064: Allow stop of arange to be inferred from dims. URL: https://github.com/apache/incubator-mxnet/pull/12064#discussion_r209695522 ## File path: src/operator/tensor/init_op.h ## @@ -471,6 +473,11 @@ inline bool

[GitHub] ankkhedia commented on issue #12119: mxnet package in r (3.5.1) - It doesn't have lstm and gluon

2018-08-13 Thread GitBox
ankkhedia commented on issue #12119: mxnet package in r (3.5.1) - It doesn't have lstm and gluon URL: https://github.com/apache/incubator-mxnet/issues/12119#issuecomment-412587866 Hi @TS0713 As @anirudhacharya mentioned, R does not have Gluon support. However, you can try using LSTM

[GitHub] samskalicky commented on issue #12091: [MXNET-792] Fix for issue #9816 with dropout operator and RNG

2018-08-13 Thread GitBox
samskalicky commented on issue #12091: [MXNET-792] Fix for issue #9816 with dropout operator and RNG URL: https://github.com/apache/incubator-mxnet/pull/12091#issuecomment-412575286 Thanks @apeforest, thats a good suggestions. I updated the title to be a little more descriptive.

[GitHub] lebeg commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

2018-08-13 Thread GitBox
lebeg commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148#discussion_r209664405 ## File path: src/operator/rnn_impl.h ## @@ -174,17 +174,21 @@ void

[GitHub] lebeg commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

2018-08-13 Thread GitBox
lebeg commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148#discussion_r209664120 ## File path: cmake/ChooseBLAS.cmake ## @@ -0,0 +1,266 @@ +# Licensed to the Apache

[GitHub] lebeg commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

2018-08-13 Thread GitBox
lebeg commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148#discussion_r209664058 ## File path: CMakeLists.txt ## @@ -268,13 +356,18 @@ else() endif() if(USE_CUDA

[GitHub] lebeg commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

2018-08-13 Thread GitBox
lebeg commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148#discussion_r209654913 ## File path: CMakeLists.txt ## @@ -38,7 +48,117 @@ mxnet_option(BUILD_CPP_EXAMPLES

[GitHub] marcoabreu commented on issue #12146: Disable broken test test_io.test_CSVIter

2018-08-13 Thread GitBox
marcoabreu commented on issue #12146: Disable broken test test_io.test_CSVIter URL: https://github.com/apache/incubator-mxnet/pull/12146#issuecomment-412556466 It did not fail all the times, only 7 out of 9 cases - highly flaky:

[incubator-mxnet] branch master updated: Disable test_io.test_CSVIter (#12146)

2018-08-13 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 a6ecb59 Disable

[GitHub] marcoabreu closed pull request #12146: Disable broken test test_io.test_CSVIter

2018-08-13 Thread GitBox
marcoabreu closed pull request #12146: Disable broken test test_io.test_CSVIter URL: https://github.com/apache/incubator-mxnet/pull/12146 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

[GitHub] rahul003 commented on issue #12086: distribute training on local machines not AWS cloud

2018-08-13 Thread GitBox
rahul003 commented on issue #12086: distribute training on local machines not AWS cloud URL: https://github.com/apache/incubator-mxnet/issues/12086#issuecomment-412555843 I actually haven't tried doing this. I'm not sure why it's not working. On Mon, Aug 13, 2018, 4:51 AM whu-lyh

[GitHub] larroy edited a comment on issue #12146: Disable broken test test_io.test_CSVIter

2018-08-13 Thread GitBox
larroy edited a comment on issue #12146: Disable broken test test_io.test_CSVIter URL: https://github.com/apache/incubator-mxnet/pull/12146#issuecomment-412531963 LGTM, please merge this we are blocked validating other PRs, how did this get through CI? seems to fail consistenly:

[GitHub] larroy commented on issue #12146: Disable broken test test_io.test_CSVIter

2018-08-13 Thread GitBox
larroy commented on issue #12146: Disable broken test test_io.test_CSVIter URL: https://github.com/apache/incubator-mxnet/pull/12146#issuecomment-412531963 LGTM, please merge this we are blocked:

[GitHub] larroy commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

2018-08-13 Thread GitBox
larroy commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148#discussion_r209621953 ## File path: src/operator/rnn_impl.h ## @@ -1890,7 +1896,11 @@ void

[GitHub] larroy commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

2018-08-13 Thread GitBox
larroy commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148#discussion_r209621811 ## File path: src/operator/rnn_impl.h ## @@ -174,17 +174,21 @@ void

[GitHub] larroy commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

2018-08-13 Thread GitBox
larroy commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148#discussion_r209620682 ## File path: cmake/ChooseBLAS.cmake ## @@ -0,0 +1,266 @@ +# Licensed to the Apache

[GitHub] larroy commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

2018-08-13 Thread GitBox
larroy commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148#discussion_r209620456 ## File path: cmake/ChooseBLAS.cmake ## @@ -0,0 +1,266 @@ +# Licensed to the Apache

[GitHub] larroy commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

2018-08-13 Thread GitBox
larroy commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148#discussion_r209620535 ## File path: cmake/ChooseBLAS.cmake ## @@ -0,0 +1,266 @@ +# Licensed to the Apache

[GitHub] larroy commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

2018-08-13 Thread GitBox
larroy commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148#discussion_r209612375 ## File path: CMakeLists.txt ## @@ -268,13 +356,18 @@ else() endif()

[GitHub] larroy commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

2018-08-13 Thread GitBox
larroy commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148#discussion_r209612020 ## File path: CMakeLists.txt ## @@ -38,7 +48,117 @@ mxnet_option(BUILD_CPP_EXAMPLES

[GitHub] larroy commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake

2018-08-13 Thread GitBox
larroy commented on a change in pull request #11148: [MXNET-679] Refactor handling BLAS libraries with cmake URL: https://github.com/apache/incubator-mxnet/pull/11148#discussion_r209612020 ## File path: CMakeLists.txt ## @@ -38,7 +48,117 @@ mxnet_option(BUILD_CPP_EXAMPLES

[GitHub] taliesinb commented on a change in pull request #12064: Allow stop of arange to be inferred from dims.

2018-08-13 Thread GitBox
taliesinb commented on a change in pull request #12064: Allow stop of arange to be inferred from dims. URL: https://github.com/apache/incubator-mxnet/pull/12064#discussion_r209592905 ## File path: src/operator/tensor/init_op.h ## @@ -471,6 +473,11 @@ inline bool

[GitHub] prathik-naidu edited a comment on issue #12127: LSTM split0 Operator Error

2018-08-13 Thread GitBox
prathik-naidu edited a comment on issue #12127: LSTM split0 Operator Error URL: https://github.com/apache/incubator-mxnet/issues/12127#issuecomment-412496468 [https://github.com/apache/incubator-mxnet/issues/11430](https://github.com/apache/incubator-mxnet/issues/11430) This github

[GitHub] prathik-naidu commented on issue #12127: LSTM split0 Operator Error

2018-08-13 Thread GitBox
prathik-naidu commented on issue #12127: LSTM split0 Operator Error URL: https://github.com/apache/incubator-mxnet/issues/12127#issuecomment-412496468 [https://github.com/apache/incubator-mxnet/issues/11430](https://github.com/apache/incubator-mxnet/issues/11430) This github issues

[GitHub] prathik-naidu closed issue #12127: LSTM split0 Operator Error

2018-08-13 Thread GitBox
prathik-naidu closed issue #12127: LSTM split0 Operator Error URL: https://github.com/apache/incubator-mxnet/issues/12127 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] larroy commented on issue #12148: Fix RAT license checker which is broken in trunk

2018-08-13 Thread GitBox
larroy commented on issue #12148: Fix RAT license checker which is broken in trunk URL: https://github.com/apache/incubator-mxnet/pull/12148#issuecomment-412495985 Addressed your comment in deploy_doc, good catch. I knew but I missed in the patch.

[GitHub] larroy commented on issue #12148: Fix RAT license checker which is broken in trunk

2018-08-13 Thread GitBox
larroy commented on issue #12148: Fix RAT license checker which is broken in trunk URL: https://github.com/apache/incubator-mxnet/pull/12148#issuecomment-412495501 I'm not mixing concerns, I'm erasing unused files which fail the license check. Have you tried to run the changes proposed in

[GitHub] whu-lyh closed issue #12086: distribute training on local machines not AWS cloud

2018-08-13 Thread GitBox
whu-lyh closed issue #12086: distribute training on local machines not AWS cloud URL: https://github.com/apache/incubator-mxnet/issues/12086 This is an automated message from the Apache Git Service. To respond to the

  1   2   >