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

2018-02-13 Thread GitBox
solin319 commented on a change in pull request #8373: distribute training in fp16 URL: https://github.com/apache/incubator-mxnet/pull/8373#discussion_r168089512 ## File path: src/kvstore/kvstore_dist.h ## @@ -272,16 +282,35 @@ class KVStoreDist : public KVStoreLocal {

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

2018-02-13 Thread GitBox
gaosanyuan commented on issue #9764: Large batch size does not improve predict speed URL: https://github.com/apache/incubator-mxnet/issues/9764#issuecomment-365511819 @anirudh2290 This is an automated message from the

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

2018-02-13 Thread GitBox
gaosanyuan commented on issue #9764: Large batch size does not improve predict speed URL: https://github.com/apache/incubator-mxnet/issues/9764#issuecomment-365484738 I use mxnet 0.12.0 to do image semantic segmentation, and the input data size is (Batch, C, H, W). When I set Batch is 1,

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

2018-02-13 Thread GitBox
gaosanyuan commented on issue #9764: Large batch size does not improve predict speed URL: https://github.com/apache/incubator-mxnet/issues/9764#issuecomment-365484667 I use mxnet 0.12.0 to do image semantic segmentation, and the input data size is (Batch, C, H, W). When I set Batch is 1,

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

2018-02-13 Thread GitBox
gaosanyuan commented on issue #9764: Large batch size does not improve predict speed URL: https://github.com/apache/incubator-mxnet/issues/9764#issuecomment-365484667 I use mxnet 0.12.0 to do image semantic segmentation, and the input data size is (Batch, C, H, W). When I set Batch is 1,

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

2018-02-13 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-365484238 Engine does not seem to differentiate between first layer and subsequent layers on that it considers data going into

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

2018-02-13 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-365483069 I don't think it will perform better than producing fp32 and then casting to fp16 at the beginning of the training.

[GitHub] cccorn commented on issue #9758: unknown output shape before inference

2018-02-13 Thread GitBox
cccorn commented on issue #9758: unknown output shape before inference URL: https://github.com/apache/incubator-mxnet/issues/9758#issuecomment-365481468 @ZiyueHuang Thank you for your response. There is a maximum shape of my operator's output. Did you mean the output and actual size

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

2018-02-13 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-365481034 This issue appears to be fixed by the change in [#8995](https://github.com/apache/incubator-mxnet/pull/8995) which

[GitHub] sxjscience commented on a change in pull request #9747: Add contrib.rand_log_uniform

2018-02-13 Thread GitBox
sxjscience commented on a change in pull request #9747: Add contrib.rand_log_uniform URL: https://github.com/apache/incubator-mxnet/pull/9747#discussion_r168065613 ## File path: python/mxnet/ndarray/contrib.py ## @@ -18,9 +18,76 @@ # coding: utf-8 # pylint:

[GitHub] eric-haibin-lin commented on issue #9675: Add contrib.compute_accidental_hits operator for candidate sampling

2018-02-13 Thread GitBox
eric-haibin-lin commented on issue #9675: Add contrib.compute_accidental_hits operator for candidate sampling URL: https://github.com/apache/incubator-mxnet/pull/9675#issuecomment-365476686 Closing it for now until the hash issue is fixed.

[GitHub] eric-haibin-lin closed pull request #9675: Add contrib.compute_accidental_hits operator for candidate sampling

2018-02-13 Thread GitBox
eric-haibin-lin closed pull request #9675: Add contrib.compute_accidental_hits operator for candidate sampling URL: https://github.com/apache/incubator-mxnet/pull/9675 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] eric-haibin-lin commented on a change in pull request #9747: Add contrib.rand_log_uniform

2018-02-13 Thread GitBox
eric-haibin-lin commented on a change in pull request #9747: Add contrib.rand_log_uniform URL: https://github.com/apache/incubator-mxnet/pull/9747#discussion_r168063477 ## File path: python/mxnet/ndarray/contrib.py ## @@ -18,9 +18,76 @@ # coding: utf-8 # pylint:

[GitHub] feevos commented on issue #9288: Get HybridBlock layer shape on runtime

2018-02-13 Thread GitBox
feevos commented on issue #9288: Get HybridBlock layer shape on runtime URL: https://github.com/apache/incubator-mxnet/issues/9288#issuecomment-365475030 Dear all, do we have any progress on this? Thank you very much for your time.

[GitHub] eric-haibin-lin commented on issue #9625: sparse regression operators

2018-02-13 Thread GitBox
eric-haibin-lin commented on issue #9625: sparse regression operators URL: https://github.com/apache/incubator-mxnet/pull/9625#issuecomment-365471224 @marcoabreu any idea on the error msg ``` c:\jenkins_slave\workspace\build-cpu@3\src\operator\mxnet_op.h(456): fatal error C1002:

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

2018-02-13 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_r168058685 ## File path: src/kvstore/kvstore_dist.h ## @@ -272,16 +282,35 @@ class KVStoreDist : public KVStoreLocal {

[GitHub] DickJC123 opened a new pull request #9791: CI test randomness 3

2018-02-13 Thread GitBox
DickJC123 opened a new pull request #9791: CI test randomness 3 URL: https://github.com/apache/incubator-mxnet/pull/9791 ## Description ## This is a rebasing and partial squashing of the stale "ci test randomness2" PR #8526 . It's based on the premise that the CI should test the

[GitHub] solin319 commented on issue #9782: problem in local kvstore

2018-02-13 Thread GitBox
solin319 commented on issue #9782: problem in local kvstore URL: https://github.com/apache/incubator-mxnet/issues/9782#issuecomment-365464602 We train resnet50 in imageclassification with kvstore=local.

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

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

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

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

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

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

[GitHub] parallelgithub closed issue #9781: Where to find argument definition with String type

2018-02-13 Thread GitBox
parallelgithub closed issue #9781: Where to find argument definition with String type URL: https://github.com/apache/incubator-mxnet/issues/9781 This is an automated message from the Apache Git Service. To respond to the

[GitHub] parallelgithub commented on a change in pull request #9771: Modify NDArrayIter constructor to receive tuple (i.e. dict in Python)?

2018-02-13 Thread GitBox
parallelgithub commented on a change in pull request #9771: Modify NDArrayIter constructor to receive tuple (i.e. dict in Python)? URL: https://github.com/apache/incubator-mxnet/pull/9771#discussion_r168052820 ## File path:

[GitHub] mbaijal commented on issue #8913: MXNet 1.0.0 Release Feedback: Source Headers and License Files

2018-02-13 Thread GitBox
mbaijal commented on issue #8913: MXNet 1.0.0 Release Feedback: Source Headers and License Files URL: https://github.com/apache/incubator-mxnet/issues/8913#issuecomment-365458977 @lupesko yes, I am tracking these and other issues in [this

[GitHub] mbaijal commented on issue #8913: MXNet 1.0.0 Release Feedback: Source Headers and License Files

2018-02-13 Thread GitBox
mbaijal commented on issue #8913: MXNet 1.0.0 Release Feedback: Source Headers and License Files URL: https://github.com/apache/incubator-mxnet/issues/8913#issuecomment-365458977 @lupesko yes, I am tracking these and other issues in [this

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

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

[GitHub] lupesko commented on issue #8913: MXNet 1.0.0 Release Feedback: Source Headers and License Files

2018-02-13 Thread GitBox
lupesko commented on issue #8913: MXNet 1.0.0 Release Feedback: Source Headers and License Files URL: https://github.com/apache/incubator-mxnet/issues/8913#issuecomment-365458375 @mbaijal are you or anyone else following up on this? 1.1.0 is going out soon...

[GitHub] stonedl3 commented on issue #9766: DeepLearning on Imagenet with mxnet issues translating .lst to .rec files

2018-02-13 Thread GitBox
stonedl3 commented on issue #9766: DeepLearning on Imagenet with mxnet issues translating .lst to .rec files URL: https://github.com/apache/incubator-mxnet/issues/9766#issuecomment-365457452 I am traveling for the next couple of days. I will post the script when I return. I have not

[GitHub] lupesko commented on issue #9781: Where to find argument definition with String type

2018-02-13 Thread GitBox
lupesko commented on issue #9781: Where to find argument definition with String type URL: https://github.com/apache/incubator-mxnet/issues/9781#issuecomment-365454884 For MXNet questions, please use [discuss.mxnet.io](https://discuss.mxnet.io/) Thanks! @sandeep-krishnamurthy can

[GitHub] szha commented on issue #6872: HDF5 DataIter Python

2018-02-13 Thread GitBox
szha commented on issue #6872: HDF5 DataIter Python URL: https://github.com/apache/incubator-mxnet/issues/6872#issuecomment-365452982 @apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and needs triage. For general "how-to" questions, our

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

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

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

2018-02-13 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_r168042933 ## File path: src/kvstore/kvstore_dist_server.h ## @@ -378,25 +379,46 @@ class KVStoreDistServer { if

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

2018-02-13 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_r168042933 ## File path: src/kvstore/kvstore_dist_server.h ## @@ -378,25 +379,46 @@ class KVStoreDistServer { if

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

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

[GitHub] szha opened a new pull request #9790: make array.reshape compatible with numpy

2018-02-13 Thread GitBox
szha opened a new pull request #9790: make array.reshape compatible with numpy URL: https://github.com/apache/incubator-mxnet/pull/9790 ## Description ## Allow n ints as input to array.reshape. ## Checklist ## ### Essentials ### - [x] Passed code style checking (`make lint`)

[GitHub] rf987 closed issue #9787: net.collect_params() omits Dropout layers

2018-02-13 Thread GitBox
rf987 closed issue #9787: net.collect_params() omits Dropout layers URL: https://github.com/apache/incubator-mxnet/issues/9787 This is an automated message from the Apache Git Service. To respond to the message, please log

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

2018-02-13 Thread GitBox
marcoabreu 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_r168025671 ## File path: tests/python/unittest/test_metric.py ## @@ -56,18 +55,51 @@ def

[GitHub] sxjscience commented on issue #9586: mx.metric F1 is using numpy logic

2018-02-13 Thread GitBox
sxjscience commented on issue #9586: mx.metric F1 is using numpy logic URL: https://github.com/apache/incubator-mxnet/issues/9586#issuecomment-365429204 OK, I'll PR after it's merged. This is an automated message from the

[GitHub] szha commented on issue #9586: mx.metric F1 is using numpy logic

2018-02-13 Thread GitBox
szha commented on issue #9586: mx.metric F1 is using numpy logic URL: https://github.com/apache/incubator-mxnet/issues/9586#issuecomment-365429078 @sxjscience awesome. Would you propose a PR after #9777 is merged? If/when you do, remember to report the benchmark test results from #9705

[GitHub] sxjscience commented on issue #9586: mx.metric F1 is using numpy logic

2018-02-13 Thread GitBox
sxjscience commented on issue #9586: mx.metric F1 is using numpy logic URL: https://github.com/apache/incubator-mxnet/issues/9586#issuecomment-365427676 Bring discussion to the correct place. I've implemented an ndarray version of F1 score when doing the experiments and I've included my

[GitHub] marcoabreu commented on issue #9616: Removing a broken tutorial from the nightly tests

2018-02-13 Thread GitBox
marcoabreu commented on issue #9616: Removing a broken tutorial from the nightly tests URL: https://github.com/apache/incubator-mxnet/pull/9616#issuecomment-365428986 Thanks Aaron, since @eric-haibin-lin is on vacation, I'd like @mbaijal opinion on this. Have you reviewed this case and

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

2018-02-13 Thread GitBox
marcoabreu 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-365428085 Good idea, please mention me in the other ones so I can label them accordingly

[GitHub] sxjscience commented on issue #9586: mx.metric F1 is using numpy logic

2018-02-13 Thread GitBox
sxjscience commented on issue #9586: mx.metric F1 is using numpy logic URL: https://github.com/apache/incubator-mxnet/issues/9586#issuecomment-365427676 Bring discussion to the correct place. I've implemented an ndarray version of F1 score when doing the experiments and this is my code

[GitHub] sxjscience commented on issue #9589: mx.metric should support top-K version of TP FP TN FN Precision Recall F1

2018-02-13 Thread GitBox
sxjscience commented on issue #9589: mx.metric should support top-K version of TP FP TN FN Precision Recall F1 URL: https://github.com/apache/incubator-mxnet/issues/9589#issuecomment-365426968 Also, we need to add F1 score for multi-label classification.

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

2018-02-13 Thread GitBox
szha commented on issue #9777: [MX-9588] Add micro averaging strategy for F1 metric URL: https://github.com/apache/incubator-mxnet/pull/9777#issuecomment-365426202 @sethah do you mind running the related test from #9705 and report the before/after change results here?

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

2018-02-13 Thread GitBox
szha commented on issue #9777: [MX-9588] Add micro averaging strategy for F1 metric URL: https://github.com/apache/incubator-mxnet/pull/9777#issuecomment-365425756 @sxjscience let's track the multi-label case in #9589 This

[incubator-mxnet] branch master updated: Some small typo fixes. (#9788)

2018-02-13 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/master by this push: new 45e8555 Some small typo fixes.

[GitHub] szha closed pull request #9788: Some small typo fixes.

2018-02-13 Thread GitBox
szha closed pull request #9788: Some small typo fixes. URL: https://github.com/apache/incubator-mxnet/pull/9788 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

[GitHub] sxjscience commented on issue #9787: net.collect_params() omits Dropout layers

2018-02-13 Thread GitBox
sxjscience commented on issue #9787: net.collect_params() omits Dropout layers URL: https://github.com/apache/incubator-mxnet/issues/9787#issuecomment-365417024 @rf987 You can directly use `print(net)` or `logging.info(str(net))`

[GitHub] sxjscience commented on issue #9787: net.collect_params() omits Dropout layers

2018-02-13 Thread GitBox
sxjscience commented on issue #9787: net.collect_params() omits Dropout layers URL: https://github.com/apache/incubator-mxnet/issues/9787#issuecomment-365417024 @rf987 You may directly try `print(net)` This is an automated

[GitHub] rf987 commented on issue #9787: net.collect_params() omits Dropout layers

2018-02-13 Thread GitBox
rf987 commented on issue #9787: net.collect_params() omits Dropout layers URL: https://github.com/apache/incubator-mxnet/issues/9787#issuecomment-365416037 When I open old models, I use the string output of net.collect_params() to remind me of what the network architecture was.

[GitHub] szha commented on issue #9787: net.collect_params() omits Dropout layers

2018-02-13 Thread GitBox
szha commented on issue #9787: net.collect_params() omits Dropout layers URL: https://github.com/apache/incubator-mxnet/issues/9787#issuecomment-365413675 That's because Dropout layer doesn't have a parameter right now.

[GitHub] szha commented on issue #9789: add activations doc

2018-02-13 Thread GitBox
szha commented on issue #9789: add activations doc URL: https://github.com/apache/incubator-mxnet/pull/9789#issuecomment-365412551 Updated doc can be found at http://mxnet-doc.s3-accelerate.dualstack.amazonaws.com/api/python/gluon/nn.html#activation-layers

[GitHub] szha opened a new pull request #9789: add activations doc

2018-02-13 Thread GitBox
szha opened a new pull request #9789: add activations doc URL: https://github.com/apache/incubator-mxnet/pull/9789 ## Description ## add doc section for gluon activations. ## Checklist ## ### Essentials ### - [x] To the my best knowledge, examples are either not affected by

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

2018-02-13 Thread GitBox
anirudh2290 commented on issue #9772: ndarray indexing issues URL: https://github.com/apache/incubator-mxnet/issues/9772#issuecomment-365348247 Issue is that isinstance is not working as expected by the code and returns false in python3. Workaround is to force the type of i to be int

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

2018-02-13 Thread GitBox
rahul003 commented on issue #8373: distribute training in fp16 URL: https://github.com/apache/incubator-mxnet/pull/8373#issuecomment-365386805 @solin319 Which machines did you run the above numbers on? Let us try to come up with an easier interface for this so we can use this on the latest

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

2018-02-13 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_r16759 ## File path: python/mxnet/metric.py ## @@ -475,8 +475,84 @@ def update(self,

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

2018-02-13 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_r167998887 ## File path: python/mxnet/metric.py ## @@ -475,8 +475,84 @@ def update(self, labels,

[GitHub] KellenSunderland opened a new pull request #9788: Some small typo fixes.

2018-02-13 Thread GitBox
KellenSunderland opened a new pull request #9788: Some small typo fixes. URL: https://github.com/apache/incubator-mxnet/pull/9788 Some small typo fixes. This is an automated message from the Apache Git Service. To respond to

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

2018-02-13 Thread GitBox
sxjscience commented on issue #9777: [MX-9588] Add micro averaging strategy for F1 metric URL: https://github.com/apache/incubator-mxnet/pull/9777#issuecomment-365397351 Also, one thing to note is that when the output has only one-label, the micro F1 is equivalent to accuracy

[GitHub] aaronmarkham opened a new pull request #9616: Removing a broken tutorial from the nightly tests

2018-02-13 Thread GitBox
aaronmarkham opened a new pull request #9616: Removing a broken tutorial from the nightly tests URL: https://github.com/apache/incubator-mxnet/pull/9616 ## Description ## The predict image tutorial is broken: https://github.com/apache/incubator-mxnet/issues/9532 It is causing the

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

2018-02-13 Thread GitBox
sxjscience commented on issue #9777: [MX-9588] Add micro averaging strategy for F1 metric URL: https://github.com/apache/incubator-mxnet/pull/9777#issuecomment-365397351 Also, one thing to note is that when the output has only one-label, the micro F1 is equivalent to accuracy

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

2018-02-13 Thread GitBox
sxjscience commented on issue #9777: [MX-9588] Add micro averaging strategy for F1 metric URL: https://github.com/apache/incubator-mxnet/pull/9777#issuecomment-365397351 Also, one thing to note is that when the output has only one-label, the micro F1 is equivalent to accuracy

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

2018-02-13 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_r167993481 ## File path: python/mxnet/metric.py ## @@ -475,8 +475,84 @@ def update(self,

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

2018-02-13 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_r167992849 ## File path: python/mxnet/metric.py ## @@ -475,8 +475,84 @@ def update(self,

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

2018-02-13 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_r167989462 ## File path: python/mxnet/metric.py ## @@ -475,8 +475,84 @@ def update(self, labels,

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

2018-02-13 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_r167989350 ## File path: python/mxnet/metric.py ## @@ -475,8 +475,84 @@ def update(self, labels,

[GitHub] rf987 opened a new issue #9787: net.collect_params() optims Dropout layers

2018-02-13 Thread GitBox
rf987 opened a new issue #9787: net.collect_params() optims Dropout layers URL: https://github.com/apache/incubator-mxnet/issues/9787 Why does net.collect_params() not make mention of Dropout layers ? This is an automated

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

2018-02-13 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_r167986170 ## File path: python/mxnet/metric.py ## @@ -475,8 +475,84 @@ def update(self, labels,

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

2018-02-13 Thread GitBox
rahul003 commented on issue #8373: distribute training in fp16 URL: https://github.com/apache/incubator-mxnet/pull/8373#issuecomment-365386805 @solin319 Which machines did you run the above numbers on? Let us try to come up with an easier interface for this so we can use this on the latest

[GitHub] Bestehorn commented on issue #2860: NDArray header in SAVE and LOAD function:

2018-02-13 Thread GitBox
Bestehorn commented on issue #2860: NDArray header in SAVE and LOAD function: URL: https://github.com/apache/incubator-mxnet/issues/2860#issuecomment-365384590 I have seen this error when I saved a model on a Linux machine and then loaded the model in Windows (or vice versa).

[GitHub] piiswrong commented on issue #7938: instance norm and reflection padding

2018-02-13 Thread GitBox
piiswrong commented on issue #7938: instance norm and reflection padding URL: https://github.com/apache/incubator-mxnet/pull/7938#issuecomment-365378948 @zhanghang1989 @szha The instance norm operator needs an axis argument to be consistent with batchnorm reflectionpad2d's

[GitHub] piiswrong commented on a change in pull request #9738: Add Support for int64

2018-02-13 Thread GitBox
piiswrong commented on a change in pull request #9738: Add Support for int64 URL: https://github.com/apache/incubator-mxnet/pull/9738#discussion_r167975658 ## File path: src/operator/mxnet_op.h ## @@ -204,6 +204,15 @@ inline int get_num_threads(const int N) { }

[GitHub] piiswrong commented on a change in pull request #9738: Add Support for int64

2018-02-13 Thread GitBox
piiswrong commented on a change in pull request #9738: Add Support for int64 URL: https://github.com/apache/incubator-mxnet/pull/9738#discussion_r167975658 ## File path: src/operator/mxnet_op.h ## @@ -204,6 +204,15 @@ inline int get_num_threads(const int N) { }

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

2018-02-13 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_r167971594 ## 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-13 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_r167971498 ## File path: python/mxnet/metric.py ## @@ -475,8 +475,84 @@ def update(self, labels,

[incubator-mxnet] branch master updated: Better Exception Handling for Operators (#9681)

2018-02-13 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 7b24137 Better Exception Handling for

[GitHub] piiswrong closed issue #8835: Python crashes (core-dump) instead of a graceful error message when GPU context is used on a CPU-only instance (EC2 x1.32xlarge)

2018-02-13 Thread GitBox
piiswrong closed issue #8835: Python crashes (core-dump) instead of a graceful error message when GPU context is used on a CPU-only instance (EC2 x1.32xlarge) URL: https://github.com/apache/incubator-mxnet/issues/8835 This

[GitHub] piiswrong closed issue #9567: nd.argmax cause "Kernel Died" error in Jupyter Notebook

2018-02-13 Thread GitBox
piiswrong closed issue #9567: nd.argmax cause "Kernel Died" error in Jupyter Notebook URL: https://github.com/apache/incubator-mxnet/issues/9567 This is an automated message from the Apache Git Service. To respond to the

[GitHub] piiswrong closed pull request #9681: Better Exception Handling for Operators

2018-02-13 Thread GitBox
piiswrong closed pull request #9681: Better Exception Handling for Operators URL: https://github.com/apache/incubator-mxnet/pull/9681 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 closed issue #9131: random_uniform causes VM to crash

2018-02-13 Thread GitBox
piiswrong closed issue #9131: random_uniform causes VM to crash URL: https://github.com/apache/incubator-mxnet/issues/9131 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] piiswrong closed issue #7335: Exception in threads kills entire process

2018-02-13 Thread GitBox
piiswrong closed issue #7335: Exception in threads kills entire process URL: https://github.com/apache/incubator-mxnet/issues/7335 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] piiswrong closed issue #9380: nd.stack causes abort for differently sized arrays

2018-02-13 Thread GitBox
piiswrong closed issue #9380: nd.stack causes abort for differently sized arrays URL: https://github.com/apache/incubator-mxnet/issues/9380 This is an automated message from the Apache Git Service. To respond to the message,

[incubator-mxnet] branch master updated: Using the "global_pool" option to avoid an unmatching input shape of (#9783)

2018-02-13 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 f57073e Using the "global_pool" option

[GitHub] piiswrong closed pull request #9783: Fixing a symbol file for image-classification example

2018-02-13 Thread GitBox
piiswrong closed pull request #9783: Fixing a symbol file for image-classification example URL: https://github.com/apache/incubator-mxnet/pull/9783 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:

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

2018-02-13 Thread GitBox
anirudh2290 commented on issue #9750: Any ideas how to implement Z-Buffer algorithm? URL: https://github.com/apache/incubator-mxnet/issues/9750#issuecomment-365364303 This seems more like Q/discussion topic. Please ask on discuss.mxnet.io

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

2018-02-13 Thread GitBox
anirudh2290 commented on issue #9750: Any ideas how to implement Z-Buffer algorithm? URL: https://github.com/apache/incubator-mxnet/issues/9750#issuecomment-365364303 This seems more like Q/discussion topic. Please ask on discuss.mxnet.io.

[GitHub] rahul003 commented on issue #9782: problem in local kvstore

2018-02-13 Thread GitBox
rahul003 commented on issue #9782: problem in local kvstore URL: https://github.com/apache/incubator-mxnet/issues/9782#issuecomment-365363075 Do you have some profiling output regarding this? What kind of network are you using?

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

2018-02-13 Thread GitBox
anirudh2290 commented on issue #9764: Large batch size does not improve predict speed URL: https://github.com/apache/incubator-mxnet/issues/9764#issuecomment-365359799 Can you provide some more information: Do you have a minimum reproducible example ? Do you have any benchmark numbers ?

[GitHub] cjolivier01 closed pull request #9785: remove superfluous cmake error output

2018-02-13 Thread GitBox
cjolivier01 closed pull request #9785: remove superfluous cmake error output URL: https://github.com/apache/incubator-mxnet/pull/9785 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

[incubator-mxnet] branch master updated: remove error output (#9785)

2018-02-13 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 9b7058d remove error output

[GitHub] anirudh2290 commented on issue #9766: DeepLearning on Imagenet with mxnet issues translating .lst to .rec files

2018-02-13 Thread GitBox
anirudh2290 commented on issue #9766: DeepLearning on Imagenet with mxnet issues translating .lst to .rec files URL: https://github.com/apache/incubator-mxnet/issues/9766#issuecomment-365358116 Can you please also provide the script that you ran to reproduce the issue ? I see that you

[GitHub] pishen commented on a change in pull request #9771: Modify NDArrayIter constructor to receive tuple (i.e. dict in Python)?

2018-02-13 Thread GitBox
pishen commented on a change in pull request #9771: Modify NDArrayIter constructor to receive tuple (i.e. dict in Python)? URL: https://github.com/apache/incubator-mxnet/pull/9771#discussion_r167953138 ## File path:

[GitHub] cjolivier01 closed pull request #9786: Add C++ modules to CODEOWNERS

2018-02-13 Thread GitBox
cjolivier01 closed pull request #9786: Add C++ modules to CODEOWNERS URL: https://github.com/apache/incubator-mxnet/pull/9786 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[incubator-mxnet] branch master updated: Add C++ modules to CODEOWNERS (#9786)

2018-02-13 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 8a7b18f Add C++ modules to

[GitHub] sxjscience commented on a change in pull request #9688: Adapt operators from PyTorch, will keep adding

2018-02-13 Thread GitBox
sxjscience commented on a change in pull request #9688: Adapt operators from PyTorch, will keep adding URL: https://github.com/apache/incubator-mxnet/pull/9688#discussion_r167952142 ## File path: src/operator/bilinear_upsample.cc ## @@ -0,0 +1,185 @@ +/* + * Licensed to

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

2018-02-13 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_r167951825 ## File path: example/ssd/detect/detector.py ## @@ -136,31 +133,52 @@ class names

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

2018-02-13 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_r167951698 ## File path: example/ssd/detect/detector.py ## @@ -136,31 +133,52 @@ class names

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

2018-02-13 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_r167951600 ## File path: example/ssd/detect/detector.py ## @@ -43,6 +43,7 @@ class Detector(object):

[GitHub] cjolivier01 commented on issue #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc.

2018-02-13 Thread GitBox
cjolivier01 commented on issue #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc. URL: https://github.com/apache/incubator-mxnet/pull/8972#issuecomment-360906975 CI broken can't build This is

  1   2   >