[GitHub] szha commented on issue #9767: [feature request]arg_scope in gluon

2018-02-14 Thread GitBox
szha commented on issue #9767: [feature request]arg_scope in gluon URL: https://github.com/apache/incubator-mxnet/issues/9767#issuecomment-365531227 @dongzhuoyao I'm not sure if I follow. How do you intend to change the batch-norm behavior? Would you provide an example? --

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

2018-02-14 Thread GitBox
marcoabreu commented on issue #9625: sparse regression operators URL: https://github.com/apache/incubator-mxnet/pull/9625#issuecomment-365534913 " c:\jenkins_slave\workspace\build-cpu@3\src\operator\mxnet_op.h(456): fatal error C1002: compiler is out of heap space in pass 2 [C:\jenkins_sl

[GitHub] marcoabreu commented on issue #9771: Modify NDArrayIter constructor to receive tuple (i.e. dict in Python)?

2018-02-14 Thread GitBox
marcoabreu commented on issue #9771: Modify NDArrayIter constructor to receive tuple (i.e. dict in Python)? URL: https://github.com/apache/incubator-mxnet/pull/9771#issuecomment-365535322 @nswamy do you have an idea? This is

[GitHub] chowkamlee81 opened a new issue #9792: Any versions of NASNet/ShuffleNet models available in mxnet?

2018-02-14 Thread GitBox
chowkamlee81 opened a new issue #9792: Any versions of NASNet/ShuffleNet models available in mxnet? URL: https://github.com/apache/incubator-mxnet/issues/9792 Any official versions of pretrained models NASNet/ShuffleNet available? --

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

2018-02-14 Thread GitBox
tutnixzursache commented on issue #8530: Fatal JVM Error due to Exception in CustomOpProp#inferTypeEntry URL: https://github.com/apache/incubator-mxnet/issues/8530#issuecomment-365561448 Sorry, I've switched to eaplatanios/tensorflow_scala in the mean time. Back in November my hack was

[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-365562226 Very nice feature, thanks a lot for putting so much effort into this! This is an automat

[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_r168135862 ## File path: tests/python/unittest/common.py ## @@ -30,3 +35,190 @@ def assertRaises(expected_exception, func, *args,

[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. ---

[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): mode

[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): assert

[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 check_instance_norm_with_shape(sha

[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 ne

[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] 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] 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 th

[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: file

[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 throwin

[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 (req_

[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 fai

[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 a

[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 fai

[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 a

[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] 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 test_acc():

[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] 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 request

[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 (#978

[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 Ser

[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 resp

[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. T

[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 test_ac

[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 test_acc():

[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] 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] 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 mes

[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 t

[GitHub] zhreshold commented on a change in pull request #9784: Fix for the case where there are no detections

2018-02-14 Thread GitBox
zhreshold commented on a change in pull request #9784: Fix for the case where there are no detections URL: https://github.com/apache/incubator-mxnet/pull/9784#discussion_r168291198 ## File path: example/ssd/detect/detector.py ## @@ -136,31 +132,52 @@ class names h

[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 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 #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 #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 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 i

[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 print_layer_su

[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 automa

[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 forei

[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 https://www.software.ac.uk/b

[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] 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] 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_r168315437 ## File path: python/mxnet/ndarray/ndarray.py ## @@ -968,6 +973,14 @@ def reshape(self, shape):

[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] 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, preds

[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 print_layer_summary

[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 #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, preds

[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 ## File

[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 ## File

[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 ## File

[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 auto

[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 siz

[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 siz

[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 siz

[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-365780748 Thanks, this [blogpost](http://docs.nvidia.com/deeplearning/sdk/mixed-precision-training/index.html) mentions trainin

[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 https://github.com/apache/incubator-mxnet/issues/37

[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/ Resolv

[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 r

[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 trainin

[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. ``` make

[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, pre

[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 rep

[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 So

[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 the

[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 y

[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 #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 ask

[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, p

[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] 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 to

[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] 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 #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 instal

[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] 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] 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: https://git

[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 diff

[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] 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] 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 11

[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): la

[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_r168374258 ## File path: python/mxnet/metric.py ## @@ -503,21 +582,27 @@ class F1(EvalMetric): la

[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 C

[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 auto

[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 a

[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-365818872 @zhreshold please approve once you confirm that it's implemented correctly. This is an automated messag

[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): model

[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_r168376036 ## File path: tests/python/unittest/test_autograd.py ## @@ -360,8 +373,12 @@ def backward(self, dm, dn): assert_

  1   2   >