[GitHub] anirudh2290 commented on issue #9772: ndarray indexing issues

2018-02-14 Thread GitBox
anirudh2290 commented on issue #9772: ndarray indexing issues URL: https://github.com/apache/incubator-mxnet/issues/9772#issuecomment-365836694 @reminisce verified that it works for np.int64 on python2, doesn't work for np.int32. For python3, it doesnt work for either np.int32 or

[GitHub] anirudh2290 commented on issue #9772: ndarray indexing issues

2018-02-14 Thread GitBox
anirudh2290 commented on issue #9772: ndarray indexing issues URL: https://github.com/apache/incubator-mxnet/issues/9772#issuecomment-365836694 @reminisce verified that it works for np.int64 on python2, doesn't work for np.int32. For python3, it doesnt work for either np.int32 or

[GitHub] anirudh2290 commented on issue #9772: ndarray indexing issues

2018-02-14 Thread GitBox
anirudh2290 commented on issue #9772: ndarray indexing issues URL: https://github.com/apache/incubator-mxnet/issues/9772#issuecomment-365816203 @reminisce I wasnt able to reproduce the issue for python 2. The int type changed in python3 and the type is not related to numpy classes

[GitHub] pengzhao-intel commented on issue #9677: Refactor operators and add MKLDNN

2018-02-14 Thread GitBox
pengzhao-intel commented on issue #9677: Refactor operators and add MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/9677#issuecomment-365822430 All comments have been resolved and pass the CI test. Thanks, @zheng-da :) @szha , @cjolivier01 , @marcoabreu please help take

[GitHub] juliusshufan commented on a change in pull request #9793: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classific

2018-02-14 Thread GitBox
juliusshufan commented on a change in pull request #9793: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification URL: https://github.com/apache/incubator-mxnet/pull/9793#discussion_r168378166 ##

[incubator-mxnet] branch master updated: fix markdown hyperlink syntax (#9756)

2018-02-14 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 7d4b4c0 fix markdown hyperlink

[GitHub] szha closed pull request #9756: Fix markdown link syntax error

2018-02-14 Thread GitBox
szha closed pull request #9756: Fix markdown link syntax error URL: https://github.com/apache/incubator-mxnet/pull/9756 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] anirudh2290 commented on issue #9772: ndarray indexing issues

2018-02-14 Thread GitBox
anirudh2290 commented on issue #9772: ndarray indexing issues URL: https://github.com/apache/incubator-mxnet/issues/9772#issuecomment-365820033 @reminisce away from laptop. Will give that a try. Tried with np.int64 This is

[GitHub] DickJC123 commented on a change in pull request #9791: CI test randomness 3

2018-02-14 Thread GitBox
DickJC123 commented on a change in pull request #9791: CI test randomness 3 URL: https://github.com/apache/incubator-mxnet/pull/9791#discussion_r168376275 ## File path: tests/python/unittest/test_operator.py ## @@ -2265,18 +2317,18 @@ def

[GitHub] DickJC123 commented on a change in pull request #9791: CI test randomness 3

2018-02-14 Thread GitBox
DickJC123 commented on a change in pull request #9791: CI test randomness 3 URL: https://github.com/apache/incubator-mxnet/pull/9791#discussion_r168376036 ## File path: tests/python/unittest/test_autograd.py ## @@ -360,8 +373,12 @@ def backward(self, dm, dn):

[GitHub] szha commented on a change in pull request #9790: make array.reshape compatible with numpy

2018-02-14 Thread GitBox
szha commented on a change in pull request #9790: make array.reshape compatible with numpy URL: https://github.com/apache/incubator-mxnet/pull/9790#discussion_r168375961 ## File path: python/mxnet/ndarray/ndarray.py ## @@ -968,6 +973,14 @@ def reshape(self, shape):

[GitHub] DickJC123 commented on a change in pull request #9791: CI test randomness 3

2018-02-14 Thread GitBox
DickJC123 commented on a change in pull request #9791: CI test randomness 3 URL: https://github.com/apache/incubator-mxnet/pull/9791#discussion_r168375948 ## File path: tests/python/unittest/test_gluon.py ## @@ -530,13 +562,19 @@ def __init__(self, **kwargs):

[GitHub] szha commented on issue #9614: MobileNetV2

2018-02-14 Thread GitBox
szha commented on issue #9614: MobileNetV2 URL: https://github.com/apache/incubator-mxnet/pull/9614#issuecomment-365818872 @zhreshold please approve once you confirm that it's implemented correctly. This is an automated

[GitHub] szha commented on issue #9614: MobileNetV2

2018-02-14 Thread GitBox
szha commented on issue #9614: MobileNetV2 URL: https://github.com/apache/incubator-mxnet/pull/9614#issuecomment-365818591 LGTM. Would you mind adding the variants to `tests/python/unittest/test_gluon_model_zoo.py`? Click the unchecked checkbox at the top of this PR once you're done.

[GitHub] szha commented on issue #9614: MobileNetV2

2018-02-14 Thread GitBox
szha commented on issue #9614: MobileNetV2 URL: https://github.com/apache/incubator-mxnet/pull/9614#issuecomment-365818591 LGTM. Would you mind adding the variants to `tests/python/unittest/test_gluon_model_zoo.py`? This is

[GitHub] szha commented on issue #9614: MobileNetV2

2018-02-14 Thread GitBox
szha commented on issue #9614: MobileNetV2 URL: https://github.com/apache/incubator-mxnet/pull/9614#issuecomment-365818591 Would you mind adding the variants to `tests/python/unittest/test_gluon_model_zoo.py`? This is an

[GitHub] reminisce commented on issue #9772: ndarray indexing issues

2018-02-14 Thread GitBox
reminisce commented on issue #9772: ndarray indexing issues URL: https://github.com/apache/incubator-mxnet/issues/9772#issuecomment-365816759 @anirudh2290 Have you tried the following? ```python Python 2.7.14 |Anaconda custom (64-bit)| (default, Oct 5 2017, 02:28:52) [GCC 4.2.1

[GitHub] szha commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric

2018-02-14 Thread GitBox
szha commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric URL: https://github.com/apache/incubator-mxnet/pull/9777#discussion_r168374258 ## File path: python/mxnet/metric.py ## @@ -503,21 +582,27 @@ class F1(EvalMetric):

[GitHub] anirudh2290 commented on issue #9772: ndarray indexing issues

2018-02-14 Thread GitBox
anirudh2290 commented on issue #9772: ndarray indexing issues URL: https://github.com/apache/incubator-mxnet/issues/9772#issuecomment-365816203 @reminisce I wasnt able to reproduce the issue for python 2. The int type changed in python3 and not related to numpy classes

[GitHub] szha commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric

2018-02-14 Thread GitBox
szha commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric URL: https://github.com/apache/incubator-mxnet/pull/9777#discussion_r168373771 ## File path: python/mxnet/metric.py ## @@ -503,21 +582,27 @@ class F1(EvalMetric):

[GitHub] dwSun commented on issue #9614: MobileNetV2

2018-02-14 Thread GitBox
dwSun commented on issue #9614: MobileNetV2 URL: https://github.com/apache/incubator-mxnet/pull/9614#issuecomment-365812147 for reference, this is the struct I am using: ``` input shape stride output shape # conv2d 224x224x3 2 112x112x32 # bottleneck

[GitHub] dwSun commented on a change in pull request #9614: MobileNetV2

2018-02-14 Thread GitBox
dwSun commented on a change in pull request #9614: MobileNetV2 URL: https://github.com/apache/incubator-mxnet/pull/9614#discussion_r168369399 ## File path: python/mxnet/gluon/model_zoo/vision/mobilenet.py ## @@ -74,13 +123,69 @@ def hybrid_forward(self, F, x): x =

[GitHub] reminisce commented on issue #9743: from tensorboard import SummaryWriter wrong

2018-02-14 Thread GitBox
reminisce commented on issue #9743: from tensorboard import SummaryWriter wrong URL: https://github.com/apache/incubator-mxnet/issues/9743#issuecomment-365809418 `tensorboard` has been changed to refer to the TensorFlow's TensorBoard in PyPI.

[GitHub] reminisce commented on issue #9772: ndarray indexing issues

2018-02-14 Thread GitBox
reminisce commented on issue #9772: ndarray indexing issues URL: https://github.com/apache/incubator-mxnet/issues/9772#issuecomment-365808285 The problem is that in MXNet, `integer_types` are defined as native python integer types: `int` and `long`, while `np.int32` and `np.int64` are

[GitHub] anirudh2290 opened a new pull request #9795: Add tests for Exception Handling in Iterators

2018-02-14 Thread GitBox
anirudh2290 opened a new pull request #9795: Add tests for Exception Handling in Iterators URL: https://github.com/apache/incubator-mxnet/pull/9795 ## Description ## This adds additional tests for exception handling in Iterators. This requires the following PR to be merged:

[GitHub] szha commented on issue #9699: MXNet pip package installs older version of numpy (1.13)

2018-02-14 Thread GitBox
szha commented on issue #9699: MXNet pip package installs older version of numpy (1.13) URL: https://github.com/apache/incubator-mxnet/issues/9699#issuecomment-365793086 Does conda allow multiple versions of the same package in the same environment?

[GitHub] sandeep-krishnamurthy commented on issue #9702: [Post 1.1.0] Apply PR #9701 to the master branch

2018-02-14 Thread GitBox
sandeep-krishnamurthy commented on issue #9702: [Post 1.1.0] Apply PR #9701 to the master branch URL: https://github.com/apache/incubator-mxnet/issues/9702#issuecomment-365789310 @mbaijal - Any update on this issue. Can be resolved?

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

2018-02-14 Thread GitBox
sandeep-krishnamurthy commented on issue #9699: MXNet pip package installs older version of numpy (1.13) URL: https://github.com/apache/incubator-mxnet/issues/9699#issuecomment-365789074 This issue occurs, when I create a conda environment with numpy and later in the environment, I

[GitHub] sandeep-krishnamurthy closed issue #9750: Any ideas how to implement Z-Buffer algorithm?

2018-02-14 Thread GitBox
sandeep-krishnamurthy closed issue #9750: Any ideas how to implement Z-Buffer algorithm? URL: https://github.com/apache/incubator-mxnet/issues/9750 This is an automated message from the Apache Git Service. To respond to the

[GitHub] sandeep-krishnamurthy commented on issue #9750: Any ideas how to implement Z-Buffer algorithm?

2018-02-14 Thread GitBox
sandeep-krishnamurthy commented on issue #9750: Any ideas how to implement Z-Buffer algorithm? URL: https://github.com/apache/incubator-mxnet/issues/9750#issuecomment-365788604 Resolving here. @ShownX as Anirudh mentioned, please ask this query at - https://discuss.mxnet.io/

[GitHub] piiswrong commented on issue #9772: ndarray indexing issues

2018-02-14 Thread GitBox
piiswrong commented on issue #9772: ndarray indexing issues URL: https://github.com/apache/incubator-mxnet/issues/9772#issuecomment-365788276 @reminisce This is an automated message from the Apache Git Service. To respond

[GitHub] sandeep-krishnamurthy commented on issue #9758: unknown output shape before inference

2018-02-14 Thread GitBox
sandeep-krishnamurthy commented on issue #9758: unknown output shape before inference URL: https://github.com/apache/incubator-mxnet/issues/9758#issuecomment-365787342 @cccorn - Asking usability questions at discuss - https://discuss.mxnet.io/ will yield faster response.

[GitHub] sandeep-krishnamurthy closed issue #9760: How can I pad images using mx.nd.pad?

2018-02-14 Thread GitBox
sandeep-krishnamurthy closed issue #9760: How can I pad images using mx.nd.pad? URL: https://github.com/apache/incubator-mxnet/issues/9760 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] sandeep-krishnamurthy commented on issue #9760: How can I pad images using mx.nd.pad?

2018-02-14 Thread GitBox
sandeep-krishnamurthy commented on issue #9760: How can I pad images using mx.nd.pad? URL: https://github.com/apache/incubator-mxnet/issues/9760#issuecomment-365786901 MXNet supports and is more efficient with channels_first format only. Resolving. For usability question, please

[GitHub] sandeep-krishnamurthy commented on issue #9762: Should MXNet also support logical operations?

2018-02-14 Thread GitBox
sandeep-krishnamurthy commented on issue #9762: Should MXNet also support logical operations? URL: https://github.com/apache/incubator-mxnet/issues/9762#issuecomment-365786530 +1 For logical operators in symbolic interface.

[GitHub] sandeep-krishnamurthy commented on issue #9763: Too many header files need to be included when using C++ api

2018-02-14 Thread GitBox
sandeep-krishnamurthy commented on issue #9763: Too many header files need to be included when using C++ api URL: https://github.com/apache/incubator-mxnet/issues/9763#issuecomment-365785982 There is discussion started by MXNet community to address this issue. Please chime in to provide

[GitHub] sandeep-krishnamurthy closed issue #9764: Large batch size does not improve predict speed

2018-02-14 Thread GitBox
sandeep-krishnamurthy closed issue #9764: Large batch size does not improve predict speed URL: https://github.com/apache/incubator-mxnet/issues/9764 This is an automated message from the Apache Git Service. To respond to

[GitHub] sandeep-krishnamurthy commented on issue #9764: Large batch size does not improve predict speed

2018-02-14 Thread GitBox
sandeep-krishnamurthy commented on issue #9764: Large batch size does not improve predict speed URL: https://github.com/apache/incubator-mxnet/issues/9764#issuecomment-365785272 Hi @gaosanyuan - I think this question can get answer faster on discuss - https://discuss.mxnet.io/latest

[GitHub] arnaghizadeh commented on issue #9765: Installation with GPU on Fedora 27?

2018-02-14 Thread GitBox
arnaghizadeh commented on issue #9765: Installation with GPU on Fedora 27? URL: https://github.com/apache/incubator-mxnet/issues/9765#issuecomment-365784660 @sandeep-krishnamurthy I tried that but it did not work in the end I had to reinstall the CUDA and NVIDIA driver with a different

[GitHub] sethah commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric

2018-02-14 Thread GitBox
sethah commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric URL: https://github.com/apache/incubator-mxnet/pull/9777#discussion_r168348423 ## File path: python/mxnet/metric.py ## @@ -475,8 +475,84 @@ def update(self, labels,

[GitHub] sandeep-krishnamurthy commented on issue #9765: Installation with GPU on Fedora 27?

2018-02-14 Thread GitBox
sandeep-krishnamurthy commented on issue #9765: Installation with GPU on Fedora 27? URL: https://github.com/apache/incubator-mxnet/issues/9765#issuecomment-365783147 Hi, if CUDA library path is the issue, did you try compiling by passing the CUDA path. Something like below. ```

[GitHub] rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument

2018-02-14 Thread GitBox
rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument URL: https://github.com/apache/incubator-mxnet/issues/9774#issuecomment-365780748 Thanks, this [blogpost](http://docs.nvidia.com/deeplearning/sdk/mixed-precision-training/index.html) mentions

[GitHub] sandeep-krishnamurthy closed issue #9792: Any versions of NASNet/ShuffleNet models available in mxnet?

2018-02-14 Thread GitBox
sandeep-krishnamurthy closed issue #9792: Any versions of NASNet/ShuffleNet models available in mxnet? URL: https://github.com/apache/incubator-mxnet/issues/9792 This is an automated message from the Apache Git Service. To

[GitHub] sandeep-krishnamurthy commented on issue #9792: Any versions of NASNet/ShuffleNet models available in mxnet?

2018-02-14 Thread GitBox
sandeep-krishnamurthy commented on issue #9792: Any versions of NASNet/ShuffleNet models available in mxnet? URL: https://github.com/apache/incubator-mxnet/issues/9792#issuecomment-365781031 Hi, For examples and usability questions, please ask at - https://discuss.mxnet.io/

[GitHub] rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument

2018-02-14 Thread GitBox
rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument URL: https://github.com/apache/incubator-mxnet/issues/9774#issuecomment-365780748 Thanks, this [blogpost](http://docs.nvidia.com/deeplearning/sdk/mixed-precision-training/index.html) mentions

[GitHub] sandeep-krishnamurthy commented on issue #9794: MKL linker error on macOS 10.13 despite using configuration without MKL

2018-02-14 Thread GitBox
sandeep-krishnamurthy commented on issue #9794: MKL linker error on macOS 10.13 despite using configuration without MKL URL: https://github.com/apache/incubator-mxnet/issues/9794#issuecomment-365780663 Hi, * Issue looks similar to

[GitHub] ptrendx commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument

2018-02-14 Thread GitBox
ptrendx commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument URL: https://github.com/apache/incubator-mxnet/issues/9774#issuecomment-365780010 There are few possible explanations. The most probable reason is workspace size for convolutions. I tried pitching

[GitHub] rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument

2018-02-14 Thread GitBox
rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument URL: https://github.com/apache/incubator-mxnet/issues/9774#issuecomment-365769322 Thanks for the explanation. Btw, is training in fp16 supposed to be ~2x faster than fp32 for a given batch

[GitHub] rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument

2018-02-14 Thread GitBox
rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument URL: https://github.com/apache/incubator-mxnet/issues/9774#issuecomment-365769322 Thanks for the explanation. Btw, is training in fp16 supposed to be ~2x faster than fp32 for a given batch

[GitHub] rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument

2018-02-14 Thread GitBox
rahul003 commented on issue #9774: mx.io.ImageRecordIter does not respect dtype argument URL: https://github.com/apache/incubator-mxnet/issues/9774#issuecomment-365769322 Thanks for the explanation. Btw, is training in fp16 supposed to be ~2x faster than fp32 for a given batch

[GitHub] zheng-da commented on issue #9677: Refactor operators and add MKLDNN

2018-02-14 Thread GitBox
zheng-da commented on issue #9677: Refactor operators and add MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/9677#issuecomment-365764910 @marcoabreu the requests have been fixed. can you approve it? This is an

[GitHub] cjolivier01 commented on a change in pull request #9793: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classifica

2018-02-14 Thread GitBox
cjolivier01 commented on a change in pull request #9793: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification URL: https://github.com/apache/incubator-mxnet/pull/9793#discussion_r168325934 ##

[GitHub] cjolivier01 commented on a change in pull request #9793: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classifica

2018-02-14 Thread GitBox
cjolivier01 commented on a change in pull request #9793: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification URL: https://github.com/apache/incubator-mxnet/pull/9793#discussion_r168324964 ##

[GitHub] cjolivier01 commented on a change in pull request #9793: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classifica

2018-02-14 Thread GitBox
cjolivier01 commented on a change in pull request #9793: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification URL: https://github.com/apache/incubator-mxnet/pull/9793#discussion_r168325934 ##

[GitHub] szha commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric

2018-02-14 Thread GitBox
szha commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric URL: https://github.com/apache/incubator-mxnet/pull/9777#discussion_r168324990 ## File path: python/mxnet/metric.py ## @@ -475,8 +475,84 @@ def update(self, labels,

[GitHub] marcoabreu commented on issue #9018: Seg Fault on MNIST example - C++

2018-02-14 Thread GitBox
marcoabreu commented on issue #9018: Seg Fault on MNIST example - C++ URL: https://github.com/apache/incubator-mxnet/issues/9018#issuecomment-365755507 I'm afraid I can't be of help here This is an automated message from the

[GitHub] marcoabreu commented on issue #9018: Seg Fault on MNIST example - C++

2018-02-14 Thread GitBox
marcoabreu commented on issue #9018: Seg Fault on MNIST example - C++ URL: https://github.com/apache/incubator-mxnet/issues/9018#issuecomment-365755507 I'm afraid I can't be of help here This is an automated message from the

[GitHub] szha commented on a change in pull request #9492: fix print_summary bug and add groups of convolution

2018-02-14 Thread GitBox
szha commented on a change in pull request #9492: fix print_summary bug and add groups of convolution URL: https://github.com/apache/incubator-mxnet/pull/9492#discussion_r168318967 ## File path: python/mxnet/visualization.py ## @@ -134,20 +134,26 @@ def

[GitHub] szha commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric

2018-02-14 Thread GitBox
szha commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric URL: https://github.com/apache/incubator-mxnet/pull/9777#discussion_r168317776 ## File path: python/mxnet/metric.py ## @@ -475,8 +475,84 @@ def update(self, labels,

[GitHub] szha commented on a change in pull request #9790: make array.reshape compatible with numpy

2018-02-14 Thread GitBox
szha commented on a change in pull request #9790: make array.reshape compatible with numpy URL: https://github.com/apache/incubator-mxnet/pull/9790#discussion_r168315361 ## File path: python/mxnet/ndarray/ndarray.py ## @@ -926,12 +926,12 @@ def _at(self, idx):

[GitHub] cjolivier01 commented on issue #9784: Fix for the case where there are no detections

2018-02-14 Thread GitBox
cjolivier01 commented on issue #9784: Fix for the case where there are no detections URL: https://github.com/apache/incubator-mxnet/pull/9784#issuecomment-365740723 Is there a class somewhere with 'CLASS" defined as something other than zero?

[GitHub] cjolivier01 commented on issue #9784: Fix for the case where there are no detections

2018-02-14 Thread GitBox
cjolivier01 commented on issue #9784: Fix for the case where there are no detections URL: https://github.com/apache/incubator-mxnet/pull/9784#issuecomment-365740723 Is there a class somewhere with 'class_idx" defined as something other than zero?

[GitHub] katrinleinweber opened a new pull request #9318: simplify import of citation metadata

2018-02-14 Thread GitBox
katrinleinweber opened a new pull request #9318: simplify import of citation metadata URL: https://github.com/apache/incubator-mxnet/pull/9318 This is a copy of the BibTeX snippet from https://arxiv.org/abs/1512.01274 incorporated into the repo as suggested in

[GitHub] piiswrong closed pull request #9318: simplify import of citation metadata

2018-02-14 Thread GitBox
piiswrong closed pull request #9318: simplify import of citation metadata URL: https://github.com/apache/incubator-mxnet/pull/9318 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] piiswrong commented on issue #9029: Update ndarray binary ops to use kernel launch instead of mshadow operations

2018-02-14 Thread GitBox
piiswrong commented on issue #9029: Update ndarray binary ops to use kernel launch instead of mshadow operations URL: https://github.com/apache/incubator-mxnet/pull/9029#issuecomment-365736266 @rahul003 ping This is an

[GitHub] piiswrong commented on a change in pull request #9492: fix print_summary bug and add groups of convolution

2018-02-14 Thread GitBox
piiswrong commented on a change in pull request #9492: fix print_summary bug and add groups of convolution URL: https://github.com/apache/incubator-mxnet/pull/9492#discussion_r168301264 ## File path: python/mxnet/visualization.py ## @@ -134,17 +134,23 @@ def

[GitHub] piiswrong commented on issue #9688: Adapt operators from PyTorch, will keep adding

2018-02-14 Thread GitBox
piiswrong commented on issue #9688: Adapt operators from PyTorch, will keep adding URL: https://github.com/apache/incubator-mxnet/pull/9688#issuecomment-365733960 I think its more appropriate to call the BilinearResize This

[GitHub] piiswrong commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric

2018-02-14 Thread GitBox
piiswrong commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric URL: https://github.com/apache/incubator-mxnet/pull/9777#discussion_r168295625 ## File path: python/mxnet/metric.py ## @@ -475,8 +475,84 @@ def update(self, labels,

[GitHub] piiswrong commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric

2018-02-14 Thread GitBox
piiswrong commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric URL: https://github.com/apache/incubator-mxnet/pull/9777#discussion_r168295539 ## File path: python/mxnet/metric.py ## @@ -475,8 +475,84 @@ def update(self, labels,

[GitHub] piiswrong commented on a change in pull request #9790: make array.reshape compatible with numpy

2018-02-14 Thread GitBox
piiswrong commented on a change in pull request #9790: make array.reshape compatible with numpy URL: https://github.com/apache/incubator-mxnet/pull/9790#discussion_r168293562 ## File path: python/mxnet/ndarray/ndarray.py ## @@ -968,6 +973,14 @@ def reshape(self, shape):

[GitHub] piiswrong commented on a change in pull request #9790: make array.reshape compatible with numpy

2018-02-14 Thread GitBox
piiswrong commented on a change in pull request #9790: make array.reshape compatible with numpy URL: https://github.com/apache/incubator-mxnet/pull/9790#discussion_r168293201 ## File path: python/mxnet/ndarray/ndarray.py ## @@ -926,12 +926,12 @@ def _at(self, idx):

[GitHub] piiswrong commented on issue #9793: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification

2018-02-14 Thread GitBox
piiswrong commented on issue #9793: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification URL: https://github.com/apache/incubator-mxnet/pull/9793#issuecomment-365717847 This seems too ad-hoc. Its better

[GitHub] safrooze closed issue #9213: Crash when MXNet API called before spawning multiprocess

2018-02-14 Thread GitBox
safrooze closed issue #9213: Crash when MXNet API called before spawning multiprocess URL: https://github.com/apache/incubator-mxnet/issues/9213 This is an automated message from the Apache Git Service. To respond to the

[GitHub] safrooze commented on issue #9213: Crash when MXNet API called before spawning multiprocess

2018-02-14 Thread GitBox
safrooze commented on issue #9213: Crash when MXNet API called before spawning multiprocess URL: https://github.com/apache/incubator-mxnet/issues/9213#issuecomment-365717182 I confirm that this issue is fixed as of 1.0.0.post1 release.

[GitHub] DavoFranco commented on issue #8921: windows7::can not stop mxnet process

2018-02-14 Thread GitBox
DavoFranco commented on issue #8921: windows7::can not stop mxnet process URL: https://github.com/apache/incubator-mxnet/issues/8921#issuecomment-365716835 same error on Windows...using anaconda...it is weird because it was working a few days ago

[GitHub] sethah commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric

2018-02-14 Thread GitBox
sethah commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric URL: https://github.com/apache/incubator-mxnet/pull/9777#discussion_r168281425 ## File path: tests/python/unittest/test_metric.py ## @@ -56,18 +55,51 @@ def

[GitHub] sxjscience commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric

2018-02-14 Thread GitBox
sxjscience commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric URL: https://github.com/apache/incubator-mxnet/pull/9777#discussion_r168266829 ## File path: tests/python/unittest/test_metric.py ## @@ -56,18 +55,51 @@ def

[GitHub] anirudh2290 commented on issue #9187: Cannot find mxnet-cpp/op.h

2018-02-14 Thread GitBox
anirudh2290 commented on issue #9187: Cannot find mxnet-cpp/op.h URL: https://github.com/apache/incubator-mxnet/issues/9187#issuecomment-365699186 @marcoabreu This is an automated message from the Apache Git Service. To

[GitHub] anirudh2290 commented on issue #9018: Seg Fault on MNIST example - C++

2018-02-14 Thread GitBox
anirudh2290 commented on issue #9018: Seg Fault on MNIST example - C++ URL: https://github.com/apache/incubator-mxnet/issues/9018#issuecomment-365699297 @marcoabreu This is an automated message from the Apache Git Service.

[GitHub] anirudh2290 commented on issue #9417: [cpp-package] inception_bn.cpp is wrong

2018-02-14 Thread GitBox
anirudh2290 commented on issue #9417: [cpp-package] inception_bn.cpp is wrong URL: https://github.com/apache/incubator-mxnet/issues/9417#issuecomment-365699089 @marcoabreu This is an automated message from the Apache Git

[incubator-mxnet] branch master updated: add activations doc (#9789)

2018-02-14 Thread cjolivier01
This is an automated email from the ASF dual-hosted git repository. cjolivier01 pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new 581cf90 add activations doc

[GitHub] cjolivier01 closed pull request #9789: add activations doc

2018-02-14 Thread GitBox
cjolivier01 closed pull request #9789: add activations doc URL: https://github.com/apache/incubator-mxnet/pull/9789 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] rahul003 commented on issue #8373: distribute training in fp16

2018-02-14 Thread GitBox
rahul003 commented on issue #8373: distribute training in fp16 URL: https://github.com/apache/incubator-mxnet/pull/8373#issuecomment-365677294 I'm seeing that fp16 with dist_sync_device is slower than fp32 on a cluster of 2 p3.8x instances. (volta gpus) imagenet-benchmark-alexnet

[GitHub] sethah commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric

2018-02-14 Thread GitBox
sethah commented on a change in pull request #9777: [MX-9588] Add micro averaging strategy for F1 metric URL: https://github.com/apache/incubator-mxnet/pull/9777#discussion_r168239411 ## File path: tests/python/unittest/test_metric.py ## @@ -56,18 +55,51 @@ def

[GitHub] sethah commented on issue #9777: [MX-9588] Add micro averaging strategy for F1 metric

2018-02-14 Thread GitBox
sethah commented on issue #9777: [MX-9588] Add micro averaging strategy for F1 metric URL: https://github.com/apache/incubator-mxnet/pull/9777#issuecomment-365664996 Obviously, no change is expected since none of the update logic was changed here. **Before** ``` Metric

[GitHub] rahul003 commented on issue #8373: distribute training in fp16

2018-02-14 Thread GitBox
rahul003 commented on issue #8373: distribute training in fp16 URL: https://github.com/apache/incubator-mxnet/pull/8373#issuecomment-365648029 Wondering if this has to do with fp16 being half of fp32, and 6/3=2... This is an

[GitHub] rahul003 commented on issue #8373: distribute training in fp16

2018-02-14 Thread GitBox
rahul003 commented on issue #8373: distribute training in fp16 URL: https://github.com/apache/incubator-mxnet/pull/8373#issuecomment-365647665 I have merged and rebased this onto master, and I see this error. ```what(): [15:42:31] src/kvstore/././kvstore_dist_server.h:484: Check

[GitHub] rahul003 commented on issue #8373: distribute training in fp16

2018-02-14 Thread GitBox
rahul003 commented on issue #8373: distribute training in fp16 URL: https://github.com/apache/incubator-mxnet/pull/8373#issuecomment-365648029 Wondering if this has to do with fp16 being half of fp32, and 6/3=2... This is an

[GitHub] rahul003 commented on issue #8373: distribute training in fp16

2018-02-14 Thread GitBox
rahul003 commented on issue #8373: distribute training in fp16 URL: https://github.com/apache/incubator-mxnet/pull/8373#issuecomment-365647665 I have merged and rebased this onto master, and I see this error. ```what(): [15:42:31] src/kvstore/././kvstore_dist_server.h:484: Check

[GitHub] rahul003 commented on a change in pull request #8373: distribute training in fp16

2018-02-14 Thread GitBox
rahul003 commented on a change in pull request #8373: distribute training in fp16 URL: https://github.com/apache/incubator-mxnet/pull/8373#discussion_r168215044 ## File path: src/kvstore/kvstore_dist_server.h ## @@ -378,25 +379,46 @@ class KVStoreDistServer { if

[GitHub] eric-haibin-lin commented on issue #9213: Crash when MXNet API called before spawning multiprocess

2018-02-14 Thread GitBox
eric-haibin-lin commented on issue #9213: Crash when MXNet API called before spawning multiprocess URL: https://github.com/apache/incubator-mxnet/issues/9213#issuecomment-365602224 Before #8995 using Gluon data loader w/ num_workers > 2 will result: ``` terminate called after

[GitHub] kthr opened a new issue #9794: MKL linker error on macOS 10.13 despite using configuration without MKL

2018-02-14 Thread GitBox
kthr opened a new issue #9794: MKL linker error on macOS 10.13 despite using configuration without MKL URL: https://github.com/apache/incubator-mxnet/issues/9794 ## Description Compiling from source on macOS 10.13.3 gives the following linker errors: ``` /opt/local/bin/ranlib:

[GitHub] juliusshufan opened a new pull request #9793: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification

2018-02-14 Thread GitBox
juliusshufan opened a new pull request #9793: Enable the reporting of cross-entropy or nll loss value when training CNN network using the models defined by example/image-classification URL: https://github.com/apache/incubator-mxnet/pull/9793 ## Description ## MxNET already implements

[GitHub] marcoabreu commented on issue #9791: CI test randomness 3

2018-02-14 Thread GitBox
marcoabreu commented on issue #9791: CI test randomness 3 URL: https://github.com/apache/incubator-mxnet/pull/9791#issuecomment-365594780 Would it be possible to write a test for this feature? Especially around the ``with random_seed():``

[GitHub] marcoabreu commented on issue #9791: CI test randomness 3

2018-02-14 Thread GitBox
marcoabreu commented on issue #9791: CI test randomness 3 URL: https://github.com/apache/incubator-mxnet/pull/9791#issuecomment-365594449 ``` == FAIL: test_operator.test_rcbrt_op

[GitHub] szha commented on issue #8627: Documentation update for gluon.nn.Dense. `flatten` invalid keyword argument.

2018-02-14 Thread GitBox
szha commented on issue #8627: Documentation update for gluon.nn.Dense. `flatten` invalid keyword argument. URL: https://github.com/apache/incubator-mxnet/issues/8627#issuecomment-365592375 @apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and

[GitHub] marcoabreu commented on a change in pull request #9791: CI test randomness 3

2018-02-14 Thread GitBox
marcoabreu commented on a change in pull request #9791: CI test randomness 3 URL: https://github.com/apache/incubator-mxnet/pull/9791#discussion_r168138105 ## File path: tests/python/unittest/test_operator.py ## @@ -2265,18 +2317,18 @@ def

[GitHub] marcoabreu commented on a change in pull request #9791: CI test randomness 3

2018-02-14 Thread GitBox
marcoabreu commented on a change in pull request #9791: CI test randomness 3 URL: https://github.com/apache/incubator-mxnet/pull/9791#discussion_r168137653 ## File path: tests/python/unittest/test_autograd.py ## @@ -360,8 +373,12 @@ def backward(self, dm, dn):

[GitHub] marcoabreu commented on a change in pull request #9791: CI test randomness 3

2018-02-14 Thread GitBox
marcoabreu commented on a change in pull request #9791: CI test randomness 3 URL: https://github.com/apache/incubator-mxnet/pull/9791#discussion_r168137438 ## File path: tests/python/unittest/test_gluon.py ## @@ -530,13 +562,19 @@ def __init__(self, **kwargs):

[GitHub] tdomhan commented on issue #9405: BucketingModule causes uncaught exception if symbol name not specified

2018-02-14 Thread GitBox
tdomhan commented on issue #9405: BucketingModule causes uncaught exception if symbol name not specified URL: https://github.com/apache/incubator-mxnet/issues/9405#issuecomment-365563721 +1 for this. This is actually a very annoying bug.

  1   2   >