[GitHub] szha commented on issue #8513: --start-group not supported for macOS

2018-02-05 Thread GitBox
szha commented on issue #8513: --start-group not supported for macOS URL: https://github.com/apache/incubator-mxnet/issues/8513#issuecomment-363070301 @apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and needs triage. For general "how-to"

[GitHub] ZiyueHuang commented on issue #9625: sparse regression operators

2018-02-05 Thread GitBox
ZiyueHuang commented on issue #9625: sparse regression operators URL: https://github.com/apache/incubator-mxnet/pull/9625#issuecomment-363035472 Thank you @cjolivier01 , I am trying. This is an automated message from the

[GitHub] yanhn commented on issue #9612: CUDNN_STATUS_SUCCESS (4 vs. 0) cuDNN: CUDNN_STATUS_INTERNAL_ERROR on jetson TX2

2018-02-05 Thread GitBox
yanhn commented on issue #9612: CUDNN_STATUS_SUCCESS (4 vs. 0) cuDNN: CUDNN_STATUS_INTERNAL_ERROR on jetson TX2 URL: https://github.com/apache/incubator-mxnet/issues/9612#issuecomment-363045667 Thanks @KellenSunderland. Some said it may because out of memory

[GitHub] dimon777 commented on issue #9692: cifar10 training on P100 seems not converging

2018-02-05 Thread GitBox
dimon777 commented on issue #9692: cifar10 training on P100 seems not converging URL: https://github.com/apache/incubator-mxnet/issues/9692#issuecomment-363107707 Thanks, that was it. After 9 epoch the accuracy reached 0.8

[GitHub] dimon777 closed issue #9692: cifar10 training on P100 seems not converging

2018-02-05 Thread GitBox
dimon777 closed issue #9692: cifar10 training on P100 seems not converging URL: https://github.com/apache/incubator-mxnet/issues/9692 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] dimon777 commented on issue #9692: cifar10 training on P100 seems not converging

2018-02-05 Thread GitBox
dimon777 commented on issue #9692: cifar10 training on P100 seems not converging URL: https://github.com/apache/incubator-mxnet/issues/9692#issuecomment-363107707 Thanks, that was it. After 9 epoches the accuracy reached 0.8

[GitHub] ZiyueHuang opened a new pull request #9697: fix NAG if multi_precision = true

2018-02-05 Thread GitBox
ZiyueHuang opened a new pull request #9697: fix NAG if multi_precision = true URL: https://github.com/apache/incubator-mxnet/pull/9697 ## Description ## NAG inherits SGD, but not override SGD's `update_multi_precision` which calls `_update_impl` in SGD. NAG should not inherit SGD

[GitHub] eric-haibin-lin commented on issue #9698: Fixed 4 broken links

2018-02-05 Thread GitBox
eric-haibin-lin commented on issue #9698: Fixed 4 broken links URL: https://github.com/apache/incubator-mxnet/pull/9698#issuecomment-363264070 Can you also update the broken link in `docs/faq/multi_devices.md` for ps-lite to be http://ps-lite.readthedocs.io/en/latest/

[GitHub] szha commented on issue #8552: Problem in mx.io.ImageRecordIter

2018-02-05 Thread GitBox
szha commented on issue #8552: Problem in mx.io.ImageRecordIter URL: https://github.com/apache/incubator-mxnet/issues/8552#issuecomment-363267754 @apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and needs triage. For general "how-to"

[GitHub] szha commented on issue #5475: Difference between ImageIter and ImageRecordIter

2018-02-05 Thread GitBox
szha commented on issue #5475: Difference between ImageIter and ImageRecordIter URL: https://github.com/apache/incubator-mxnet/issues/5475#issuecomment-363267747 @apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and needs triage. For

[GitHub] szha commented on issue #8560: Upsampling: lack of arguments crashes the kernel

2018-02-05 Thread GitBox
szha commented on issue #8560: Upsampling: lack of arguments crashes the kernel URL: https://github.com/apache/incubator-mxnet/issues/8560#issuecomment-363267742 @apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and needs triage. For

[GitHub] thinksanky commented on a change in pull request #9698: Fixed 4 broken links

2018-02-05 Thread GitBox
thinksanky commented on a change in pull request #9698: Fixed 4 broken links URL: https://github.com/apache/incubator-mxnet/pull/9698#discussion_r166151717 ## File path: docs/tutorials/index.md ## @@ -174,7 +174,7 @@ The Gluon and Module tutorials are in Python, but you

[GitHub] reminisce opened a new pull request #9700: Squeeze op

2018-02-05 Thread GitBox
reminisce opened a new pull request #9700: Squeeze op URL: https://github.com/apache/incubator-mxnet/pull/9700 ## Description ## This PR implemented squeeze op in MXNet as numpy.squeeze. Requested by @szha . https://docs.scipy.org/doc/numpy/reference/generated/numpy.squeeze.html

[GitHub] zhanghang1989 commented on issue #1363: Error in FCN

2018-02-05 Thread GitBox
zhanghang1989 commented on issue #1363: Error in FCN URL: https://github.com/apache/incubator-mxnet/issues/1363#issuecomment-363274148 Got the same error using Gluon, any updates? This is an automated message from the Apache

[GitHub] chaoyuaw commented on a change in pull request #9165: add embedding learning example

2018-02-05 Thread GitBox
chaoyuaw commented on a change in pull request #9165: add embedding learning example URL: https://github.com/apache/incubator-mxnet/pull/9165#discussion_r166049971 ## File path: example/gluon/embedding_learning/model.py ## @@ -0,0 +1,224 @@ +# Licensed to the Apache

[GitHub] thinksanky opened a new pull request #9698: Fixed 4 broken links

2018-02-05 Thread GitBox
thinksanky opened a new pull request #9698: Fixed 4 broken links URL: https://github.com/apache/incubator-mxnet/pull/9698 ## Description ## * Fixed the remaining broken links reported on Feb 4th 2018 This is an automated

[GitHub] thinksanky commented on issue #9698: Fixed 4 broken links

2018-02-05 Thread GitBox
thinksanky commented on issue #9698: Fixed 4 broken links URL: https://github.com/apache/incubator-mxnet/pull/9698#issuecomment-363192917 @eric-haibin-lin - please review and merge This is an automated message from the Apache

[GitHub] eric-haibin-lin commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-02-05 Thread GitBox
eric-haibin-lin commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r166098419 ## File path: include/mxnet/op_attr_types.h ## @@ -261,6

[GitHub] eric-haibin-lin commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-02-05 Thread GitBox
eric-haibin-lin commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r166098311 ## File path: include/mxnet/c_api.h ## @@ -1237,8 +1237,28

[GitHub] eric-haibin-lin commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-02-05 Thread GitBox
eric-haibin-lin commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r166099294 ## File path: python/mxnet/contrib/quantization.py ## @@

[GitHub] eric-haibin-lin commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-02-05 Thread GitBox
eric-haibin-lin commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r166100616 ## File path: src/operator/quantization/quantization_utils.h

[GitHub] eric-haibin-lin commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-02-05 Thread GitBox
eric-haibin-lin commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r166097177 ## File path: example/quantization/common ## @@ -0,0 +1 @@

[GitHub] eric-haibin-lin commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-02-05 Thread GitBox
eric-haibin-lin commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r166097853 ## File path: example/quantization/imagenet_gen_qsym.py ##

[GitHub] rahul003 commented on issue #9305: Fixing two symbol files for CNN

2018-02-05 Thread GitBox
rahul003 commented on issue #9305: Fixing two symbol files for CNN URL: https://github.com/apache/incubator-mxnet/pull/9305#issuecomment-363166976 @juliusshufan Yeah, please remove the vgg changes when you update the PR. Thanks!

[GitHub] piiswrong commented on issue #9681: Better Exception Handling for Operators

2018-02-05 Thread GitBox
piiswrong commented on issue #9681: Better Exception Handling for Operators URL: https://github.com/apache/incubator-mxnet/pull/9681#issuecomment-363190218 suppose and operator has three outputs x, y and z and it raises an exception. then x.asnumpy() would raise an error. Then

[GitHub] piiswrong commented on a change in pull request #9662: Gluon PReLU, ELU, SELU, Swish

2018-02-05 Thread GitBox
piiswrong commented on a change in pull request #9662: Gluon PReLU, ELU, SELU, Swish URL: https://github.com/apache/incubator-mxnet/pull/9662#discussion_r166084419 ## File path: python/mxnet/gluon/nn/activations.py ## @@ -0,0 +1,215 @@ +# Licensed to the Apache Software

[GitHub] szha commented on a change in pull request #9662: Gluon PReLU, ELU, SELU, Swish

2018-02-05 Thread GitBox
szha commented on a change in pull request #9662: Gluon PReLU, ELU, SELU, Swish URL: https://github.com/apache/incubator-mxnet/pull/9662#discussion_r166087641 ## File path: src/operator/leaky_relu-inl.h ## @@ -225,7 +242,11 @@ class LeakyReLUProp : public OperatorProperty

[GitHub] eric-haibin-lin commented on a change in pull request #9698: Fixed 4 broken links

2018-02-05 Thread GitBox
eric-haibin-lin commented on a change in pull request #9698: Fixed 4 broken links URL: https://github.com/apache/incubator-mxnet/pull/9698#discussion_r166102767 ## File path: docs/tutorials/index.md ## @@ -174,7 +174,7 @@ The Gluon and Module tutorials are in Python, but

[GitHub] piiswrong commented on a change in pull request #9681: Better Exception Handling for Operators

2018-02-05 Thread GitBox
piiswrong commented on a change in pull request #9681: Better Exception Handling for Operators URL: https://github.com/apache/incubator-mxnet/pull/9681#discussion_r166079802 ## File path: src/engine/threaded_engine.h ## @@ -338,33 +346,46 @@ class ThreadedEngine : public

[GitHub] szha commented on a change in pull request #9662: Gluon PReLU, ELU, SELU, Swish

2018-02-05 Thread GitBox
szha commented on a change in pull request #9662: Gluon PReLU, ELU, SELU, Swish URL: https://github.com/apache/incubator-mxnet/pull/9662#discussion_r166086915 ## File path: python/mxnet/gluon/nn/activations.py ## @@ -0,0 +1,215 @@ +# Licensed to the Apache Software

[GitHub] GoodJoey closed issue #9430: when i use k8s to distribute training, i get the error on scheduler?(my_node_.port) != (-1) bind failed

2018-02-05 Thread GitBox
GoodJoey closed issue #9430: when i use k8s to distribute training, i get the error on scheduler?(my_node_.port) != (-1) bind failed URL: https://github.com/apache/incubator-mxnet/issues/9430 This is an automated message

[GitHub] safrooze opened a new pull request #9705: Added unittest for benchmarking metric performance

2018-02-05 Thread GitBox
safrooze opened a new pull request #9705: Added unittest for benchmarking metric performance URL: https://github.com/apache/incubator-mxnet/pull/9705 Output of the benchmark is sent to stderr ## Description ## Benchmark loops through two batch-sizes (100,000 and 1,000,000) and

[GitHub] safrooze commented on issue #9705: Added unittest for benchmarking metric performance

2018-02-05 Thread GitBox
safrooze commented on issue #9705: Added unittest for benchmarking metric performance URL: https://github.com/apache/incubator-mxnet/pull/9705#issuecomment-363302529 @szha Please review. This is an automated message from the

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166179973 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/MXNetHandler.scala ## @@ -0,0 +1,82

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166182175 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/PredictBase.scala ## @@ -0,0 +1,200

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166182830 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/PredictBase.scala ## @@ -0,0 +1,200

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166181244 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/MXNetHandler.scala ## @@ -0,0 +1,82

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166182180 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/PredictBase.scala ## @@ -0,0 +1,200

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166182196 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/PredictBase.scala ## @@ -0,0 +1,200

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166182225 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/PredictBase.scala ## @@ -0,0 +1,200

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166182325 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/PredictBase.scala ## @@ -0,0 +1,200

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166182205 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/PredictBase.scala ## @@ -0,0 +1,200

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166181419 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/PredictBase.scala ## @@ -0,0 +1,200

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166181091 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/MXNetHandler.scala ## @@ -0,0 +1,82

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166182245 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/PredictBase.scala ## @@ -0,0 +1,200

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166181172 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/MXNetHandler.scala ## @@ -0,0 +1,82

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166181917 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/PredictBase.scala ## @@ -0,0 +1,200

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166181293 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/PredictBase.scala ## @@ -0,0 +1,200

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166180002 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/MXNetHandler.scala ## @@ -0,0 +1,82

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166182170 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/PredictBase.scala ## @@ -0,0 +1,200

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166181996 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/PredictBase.scala ## @@ -0,0 +1,200

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166181466 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/MXNetHandler.scala ## @@ -0,0 +1,82

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166182192 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/PredictBase.scala ## @@ -0,0 +1,200

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166179638 ## File path: scala-package/core/src/main/scala/ml/dmlc/mxnet/IO.scala ## @@ -230,6 +230,8 @@ abstract

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166182216 ## File path: scala-package/infer/src/main/scala/ml/dmlc/mxnet/infer/PredictBase.scala ## @@ -0,0 +1,200

[GitHub] CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs

2018-02-05 Thread GitBox
CodingCat commented on a change in pull request #9678: [First cut] Scala Inference APIs URL: https://github.com/apache/incubator-mxnet/pull/9678#discussion_r166179771 ## File path: scala-package/infer/pom.xml ## @@ -0,0 +1,124 @@ + +http://maven.apache.org/POM/4.0.0; +

[GitHub] ptrendx commented on issue #9583: use nd for accuracy calculation

2018-02-05 Thread GitBox
ptrendx commented on issue #9583: use nd for accuracy calculation URL: https://github.com/apache/incubator-mxnet/pull/9583#issuecomment-363306616 @szha Did you do the performance experiments that @piiswrong asked you about (and what were the results)? With this commit we see 20% perf

[GitHub] pengzhao-intel commented on a change in pull request #9677: Refactor operators and add MKLDNN

2018-02-05 Thread GitBox
pengzhao-intel commented on a change in pull request #9677: Refactor operators and add MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/9677#discussion_r166163300 ## File path: tests/cpp/include/test_core_op.h ## @@ -209,6 +209,13 @@ class CoreOpExecutor :

[GitHub] mbaijal opened a new issue #9702: [Post 1.1.0] Apply PR #9701 to the master branch

2018-02-05 Thread GitBox
mbaijal opened a new issue #9702: [Post 1.1.0] Apply PR #9701 to the master branch URL: https://github.com/apache/incubator-mxnet/issues/9702 The changes to top level LICENSE file were reverted on the release branch for the 1.1.0 release in PR #9701 Once these changes are approved,

[GitHub] anirudh2290 commented on a change in pull request #9681: Better Exception Handling for Operators

2018-02-05 Thread GitBox
anirudh2290 commented on a change in pull request #9681: Better Exception Handling for Operators URL: https://github.com/apache/incubator-mxnet/pull/9681#discussion_r166166236 ## File path: src/engine/threaded_engine.h ## @@ -338,33 +346,46 @@ class ThreadedEngine :

[GitHub] TaoLv commented on issue #9545: Profiling discussion

2018-02-05 Thread GitBox
TaoLv commented on issue #9545: Profiling discussion URL: https://github.com/apache/incubator-mxnet/issues/9545#issuecomment-363281815 > If you have better code to determine the number of "real" cores, I will use it. > Right now, we use Intel OMP (at least when building with cmake on

[GitHub] mbaijal opened a new pull request #9701: Revert "[Review Required] Fixing Licenses: Cleaning up the Top Level LICENSE file (#9484)"

2018-02-05 Thread GitBox
mbaijal opened a new pull request #9701: Revert "[Review Required] Fixing Licenses: Cleaning up the Top Level LICENSE file (#9484)" URL: https://github.com/apache/incubator-mxnet/pull/9701 This reverts commit 8930d96b265560a797c5554a9617f607cea7740f. ## Description ## As

[GitHub] cjolivier01 commented on a change in pull request #9677: Refactor operators and add MKLDNN

2018-02-05 Thread GitBox
cjolivier01 commented on a change in pull request #9677: Refactor operators and add MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/9677#discussion_r166170702 ## File path: tests/cpp/include/test_core_op.h ## @@ -209,6 +209,13 @@ class CoreOpExecutor : public

[GitHub] zihaolucky commented on a change in pull request #9195: [WIP]NCE loss gluon

2018-02-05 Thread GitBox
zihaolucky commented on a change in pull request #9195: [WIP]NCE loss gluon URL: https://github.com/apache/incubator-mxnet/pull/9195#discussion_r166171054 ## File path: python/mxnet/gluon/data/sampler.py ## @@ -136,3 +138,74 @@ def __len__(self): raise ValueError(

[GitHub] zhaodongsun opened a new issue #9703: Can't run mx.nd.smooth_l1

2018-02-05 Thread GitBox
zhaodongsun opened a new issue #9703: Can't run mx.nd.smooth_l1 URL: https://github.com/apache/incubator-mxnet/issues/9703 ## Description (Brief description of the problem in no more than 2 sentences.) Can't run mx.nd.smooth_l1 ## Environment info (Required) --Python

[GitHub] Godricly opened a new issue #9704: Misleading Document for conv1d and max1d in gluon

2018-02-05 Thread GitBox
Godricly opened a new issue #9704: Misleading Document for conv1d and max1d in gluon URL: https://github.com/apache/incubator-mxnet/issues/9704 The document claims to support 'NWC' layout, while it actually not. my current mxnet version is mxnet-cu80 (1.0.0.post4). @zhreshold

[GitHub] eric-haibin-lin opened a new pull request #9706: Update years in NOTICE file to 2017-2018

2018-02-05 Thread GitBox
eric-haibin-lin opened a new pull request #9706: Update years in NOTICE file to 2017-2018 URL: https://github.com/apache/incubator-mxnet/pull/9706 This is an automated message from the Apache Git Service. To respond to the

[GitHub] cjolivier01 commented on issue #9545: Profiling discussion

2018-02-05 Thread GitBox
cjolivier01 commented on issue #9545: Profiling discussion URL: https://github.com/apache/incubator-mxnet/issues/9545#issuecomment-363168613 If you have better code to determine the number of "real" cores, I will use it. Right now, we use Intel OMP (at least when building with cmake on

[GitHub] piiswrong closed pull request #9697: fix NAG if multi_precision = true

2018-02-05 Thread GitBox
piiswrong closed pull request #9697: fix NAG if multi_precision = true URL: https://github.com/apache/incubator-mxnet/pull/9697 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] szha commented on a change in pull request #9698: Fixed 4 broken links

2018-02-05 Thread GitBox
szha commented on a change in pull request #9698: Fixed 4 broken links URL: https://github.com/apache/incubator-mxnet/pull/9698#discussion_r166088717 ## File path: python/mxnet/gluon/trainer.py ## @@ -34,7 +34,7 @@ class Trainer(object): The set of parameters to

[GitHub] reminisce commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-02-05 Thread GitBox
reminisce commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r166128812 ## File path: Jenkinsfile ## @@ -442,6 +479,42 @@ try { }

[GitHub] cjolivier01 commented on a change in pull request #9625: sparse regression operators

2018-02-05 Thread GitBox
cjolivier01 commented on a change in pull request #9625: sparse regression operators URL: https://github.com/apache/incubator-mxnet/pull/9625#discussion_r166129856 ## File path: src/operator/regression_output.cc ## @@ -90,7 +90,7 @@ The parameter `grad_scale` can be used

[GitHub] anirudh2290 commented on issue #9681: Better Exception Handling for Operators

2018-02-05 Thread GitBox
anirudh2290 commented on issue #9681: Better Exception Handling for Operators URL: https://github.com/apache/incubator-mxnet/pull/9681#issuecomment-363241899 @piiswrong Trying to understand your comment. Lets say we have some code snippet like the below: ``` try:

[GitHub] KellenSunderland commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-02-05 Thread GitBox
KellenSunderland commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r166132342 ## File path: Jenkinsfile ## @@ -442,6 +479,42 @@ try {

[GitHub] reminisce commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-02-05 Thread GitBox
reminisce commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r166132741 ## File path: Jenkinsfile ## @@ -442,6 +479,42 @@ try { }

[GitHub] zheng-da commented on a change in pull request #9677: Refactor operators and add MKLDNN

2018-02-05 Thread GitBox
zheng-da commented on a change in pull request #9677: Refactor operators and add MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/9677#discussion_r166135426 ## File path: tests/cpp/include/test_core_op.h ## @@ -209,6 +209,13 @@ class CoreOpExecutor : public

[GitHub] yzhliu closed pull request #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2018-02-05 Thread GitBox
yzhliu closed pull request #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245 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

[incubator-mxnet] branch master updated: Use argmax instead of argmax_channel in Accuracy to keep dimention (#8245)

2018-02-05 Thread liuyizhi
This is an automated email from the ASF dual-hosted git repository. liuyizhi 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 3e79aef Use argmax instead of

[GitHub] sandeep-krishnamurthy opened a new issue #9699: MXNet pip package installs older version of numpy (1.13)

2018-02-05 Thread GitBox
sandeep-krishnamurthy opened a new issue #9699: MXNet pip package installs older version of numpy (1.13) URL: https://github.com/apache/incubator-mxnet/issues/9699 MXNet pip package installs numpy (<=1.13). We need to test and upgrade to the latest (1.14). Some of the issues users can

[GitHub] yzhliu commented on issue #8530: Fatal JVM Error due to Exception in CustomOpProp#inferTypeEntry

2018-02-05 Thread GitBox
yzhliu commented on issue #8530: Fatal JVM Error due to Exception in CustomOpProp#inferTypeEntry URL: https://github.com/apache/incubator-mxnet/issues/8530#issuecomment-363230784 @tutnixzursache Could you shoot a fix for this?

[GitHub] KellenSunderland commented on issue #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-02-05 Thread GitBox
KellenSunderland commented on issue #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#issuecomment-363242452 @reminisce Do you know what version of cudnn you need for this? Is cudnn6 alright, or do you

[GitHub] reminisce commented on issue #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-02-05 Thread GitBox
reminisce commented on issue #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#issuecomment-363243755 @KellenSunderland cudnn6 should be alright.

svn commit: r24716 - in /release/incubator/mxnet: 0.11.0/ 0.12.0/ 0.12.1/

2018-02-05 Thread haibin
Author: haibin Date: Mon Feb 5 22:45:12 2018 New Revision: 24716 Log: remove older releases Removed: release/incubator/mxnet/0.11.0/ release/incubator/mxnet/0.12.0/ release/incubator/mxnet/0.12.1/

svn commit: r24717 - in /dev/incubator/mxnet: 0.11.0.rc0/ 0.11.0.rc1/ 0.11.0.rc2/ 0.11.0.rc3/ 0.12.0.rc0/ 0.12.1.rc0/ 0.12.1/ 1.0.0.rc0/ 1.0.0.rc1/

2018-02-05 Thread haibin
Author: haibin Date: Mon Feb 5 22:47:06 2018 New Revision: 24717 Log: remove older releases Removed: dev/incubator/mxnet/0.11.0.rc0/ dev/incubator/mxnet/0.11.0.rc1/ dev/incubator/mxnet/0.11.0.rc2/ dev/incubator/mxnet/0.11.0.rc3/ dev/incubator/mxnet/0.12.0.rc0/

[GitHub] Davidrjx commented on issue #212: crash when running python ./predict-with-pretrained-model.py

2018-02-05 Thread GitBox
Davidrjx commented on issue #212: crash when running python ./predict-with-pretrained-model.py URL: https://github.com/apache/incubator-mxnet/issues/212#issuecomment-363319926 i came with similar problem when running `mx.nd.ones((2,3),mx.gpu())` , but error as follows: ```

[GitHub] Davidrjx commented on issue #212: crash when running python ./predict-with-pretrained-model.py

2018-02-05 Thread GitBox
Davidrjx commented on issue #212: crash when running python ./predict-with-pretrained-model.py URL: https://github.com/apache/incubator-mxnet/issues/212#issuecomment-363319926 i came with similar problem when running `mxnet.nd.ones((2,3),mx.gpu())` , but error as follows: ```

[GitHub] eric-haibin-lin commented on issue #9705: Added unittest for benchmarking metric performance

2018-02-05 Thread GitBox
eric-haibin-lin commented on issue #9705: Added unittest for benchmarking metric performance URL: https://github.com/apache/incubator-mxnet/pull/9705#issuecomment-363326703 Why not include small batch size like 64? 100k is huge.

[GitHub] eric-haibin-lin opened a new pull request #9708: Add code signing key

2018-02-05 Thread GitBox
eric-haibin-lin opened a new pull request #9708: Add code signing key URL: https://github.com/apache/incubator-mxnet/pull/9708 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] eric-haibin-lin opened a new pull request #9707: Bump version to 1.1.0

2018-02-05 Thread GitBox
eric-haibin-lin opened a new pull request #9707: Bump version to 1.1.0 URL: https://github.com/apache/incubator-mxnet/pull/9707 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] GoodJoey opened a new issue #9709: what will happen if one of the node reboot when doing the distribute training?

2018-02-05 Thread GitBox
GoodJoey opened a new issue #9709: what will happen if one of the node reboot when doing the distribute training? URL: https://github.com/apache/incubator-mxnet/issues/9709 if i 'm doing the distribute training, supposed one of my node is rebooted(or 'dead' for some other reasons), what

[GitHub] Davidrjx commented on issue #9489: terminate called after throwing an instance of 'dmlc::Error'

2018-02-05 Thread GitBox
Davidrjx commented on issue #9489: terminate called after throwing an instance of 'dmlc::Error' URL: https://github.com/apache/incubator-mxnet/issues/9489#issuecomment-363327956 i think should run container with nvidia-docker, at lease so am i ,and i run mxnet with gpu

[GitHub] safrooze commented on issue #9705: Added unittest for benchmarking metric performance

2018-02-05 Thread GitBox
safrooze commented on issue #9705: Added unittest for benchmarking metric performance URL: https://github.com/apache/incubator-mxnet/pull/9705#issuecomment-363328924 The intention is to observe a measurable elapsed time (hence large data size) and amplify the difference between CPU and

[GitHub] Hebali commented on issue #9647: Gluon in Core API

2018-02-05 Thread GitBox
Hebali commented on issue #9647: Gluon in Core API URL: https://github.com/apache/incubator-mxnet/issues/9647#issuecomment-363209250 Core functionality for supported languages (namely Python, Scala, R, Julia, C++ and Perl) is delivered through bindings to MXNet's C API (see

[GitHub] KellenSunderland commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-02-05 Thread GitBox
KellenSunderland commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r166103566 ## File path: Jenkinsfile ## @@ -442,6 +479,42 @@ try {

[GitHub] KellenSunderland commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-02-05 Thread GitBox
KellenSunderland commented on a change in pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#discussion_r166103566 ## File path: Jenkinsfile ## @@ -442,6 +479,42 @@ try {

[GitHub] piiswrong commented on issue #9573: Added two functions to the C API

2018-02-05 Thread GitBox
piiswrong commented on issue #9573: Added two functions to the C API URL: https://github.com/apache/incubator-mxnet/pull/9573#issuecomment-363213889 The CAPI is meant to follow the hour glass model. Its like small kernel. If something can be done by composing existing API we shouldn't add

[GitHub] cjolivier01 commented on issue #9625: sparse regression operators

2018-02-05 Thread GitBox
cjolivier01 commented on issue #9625: sparse regression operators URL: https://github.com/apache/incubator-mxnet/pull/9625#issuecomment-363174023 Looks like it compiles now :) This is an automated message from the Apache Git

[GitHub] cjolivier01 commented on issue #9545: Profiling discussion

2018-02-05 Thread GitBox
cjolivier01 commented on issue #9545: Profiling discussion URL: https://github.com/apache/incubator-mxnet/issues/9545#issuecomment-363178424 I added a place for associated performance-related JIRA tasks:

[incubator-mxnet] branch master updated: Remove '+=' in inception-resnet-v2 (#9694)

2018-02-05 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 921fbe9 Remove '+=' in

[GitHub] piiswrong closed pull request #9694: Remove '+=' in inception-resnet-v2

2018-02-05 Thread GitBox
piiswrong closed pull request #9694: Remove '+=' in inception-resnet-v2 URL: https://github.com/apache/incubator-mxnet/pull/9694 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

  1   2   >