[GitHub] larroy commented on a change in pull request #9761: Don't use FIRST_CUDA on generators which don't support toolsets

2018-02-13 Thread GitBox
larroy commented on a change in pull request #9761: Don't use FIRST_CUDA on generators which don't support toolsets URL: https://github.com/apache/incubator-mxnet/pull/9761#discussion_r167871781 ## File path: CMakeLists.txt ## @@ -1,6 +1,8 @@

[GitHub] juliusshufan closed pull request #9305: Fixing two symbol files for CNN

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

2018-02-13 Thread GitBox
juliusshufan opened a new pull request #9783: Fixing a symbol file for image-classification example URL: https://github.com/apache/incubator-mxnet/pull/9783 ## Description ## This PR is trying to fix a runtime error when running benchmark_score.py on CPU machine. The existing

[GitHub] larroy commented on a change in pull request #9761: Don't use FIRST_CUDA on generators which don't support toolsets

2018-02-13 Thread GitBox
larroy commented on a change in pull request #9761: Don't use FIRST_CUDA on generators which don't support toolsets URL: https://github.com/apache/incubator-mxnet/pull/9761#discussion_r16787 ## File path: CMakeLists.txt ## @@ -1,6 +1,8 @@

[GitHub] larroy opened a new pull request #9784: Fix for the case where there are no detections

2018-02-13 Thread GitBox
larroy opened a new pull request #9784: Fix for the case where there are no detections URL: https://github.com/apache/incubator-mxnet/pull/9784 Use cv2.cvtColor instead of np to convert from BGR 2 RGB Fix context in detector ## Description ## (Brief description on what this PR

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

2018-02-13 Thread GitBox
larroy commented on issue #9784: Fix for the case where there are no detections URL: https://github.com/apache/incubator-mxnet/pull/9784#issuecomment-365271880 @zhreshold This is an automated message from the Apache Git

[GitHub] szha commented on issue #8606: How to quick share weight like the tensorflow using reused_params !!!

2018-02-13 Thread GitBox
szha commented on issue #8606: How to quick share weight like the tensorflow using reused_params !!! URL: https://github.com/apache/incubator-mxnet/issues/8606#issuecomment-365252214 @apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and needs

[GitHub] dabraude commented on issue #9573: Fix bug with symbol loading, and added function for loading NDArrays from memory.

2018-02-13 Thread GitBox
dabraude commented on issue #9573: Fix bug with symbol loading, and added function for loading NDArrays from memory. URL: https://github.com/apache/incubator-mxnet/pull/9573#issuecomment-365258896 Was at a conference, don't know where that build error was from but it seemed to fix itself

[GitHub] larroy commented on a change in pull request #9761: Don't use FIRST_CUDA on generators which don't support toolsets

2018-02-13 Thread GitBox
larroy commented on a change in pull request #9761: Don't use FIRST_CUDA on generators which don't support toolsets URL: https://github.com/apache/incubator-mxnet/pull/9761#discussion_r167869103 ## File path: CMakeLists.txt ## @@ -1,6 +1,8 @@

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

2018-02-13 Thread GitBox
juliusshufan commented on issue #9305: Fixing two symbol files for CNN URL: https://github.com/apache/incubator-mxnet/pull/9305#issuecomment-365261381 Close this PR, and replace by https://github.com/apache/incubator-mxnet/pull/9783

[GitHub] marcoabreu commented on issue #9672: CMake CUDA fixes + NCCL

2018-02-13 Thread GitBox
marcoabreu commented on issue #9672: CMake CUDA fixes + NCCL URL: https://github.com/apache/incubator-mxnet/pull/9672#issuecomment-365202084 Was this PR approved by anybody or was this a self merge? This is an automated

[GitHub] rimusolem commented on issue #9780: ps-lite build problems

2018-02-13 Thread GitBox
rimusolem commented on issue #9780: ps-lite build problems URL: https://github.com/apache/incubator-mxnet/issues/9780#issuecomment-365183471 However, now I got a new error. ``` libtool: link: g++ -Wall -Wwrite-strings -Woverloaded-virtual -Wno-sign-compare -fPIC -Wl,-rpath

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

2018-02-13 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-365223393 Your change has the following issues, not providing backwards compatibility: ``` Type change:

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

2018-02-13 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-365201632 Since we are using semantic versioning, this would require a major version update. Thus, this

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

2018-02-13 Thread GitBox
parallelgithub 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-365221525 Thank you for replying. Would you please explain more about backwards compatibility? Since

[GitHub] solin319 opened a new issue #9782: problem in local kvstore

2018-02-13 Thread GitBox
solin319 opened a new issue #9782: problem in local kvstore URL: https://github.com/apache/incubator-mxnet/issues/9782 In MXNet V1.0 and V1.1-rc0, when training with kvstore='local' or kvstore='device' . The computation of update **can not overlap** the backward computation.

[GitHub] rimusolem commented on issue #9780: ps-lite build problems

2018-02-13 Thread GitBox
rimusolem commented on issue #9780: ps-lite build problems URL: https://github.com/apache/incubator-mxnet/issues/9780#issuecomment-365182406 Setting environment variables CFLAGS, CXXFLASGS and CPPFLAGS as "-O3" solves the problem.

[GitHub] rimusolem commented on issue #9780: ps-lite build problems

2018-02-13 Thread GitBox
rimusolem commented on issue #9780: ps-lite build problems URL: https://github.com/apache/incubator-mxnet/issues/9780#issuecomment-365183471 However, now I got a new error. ``` libtool: link: g++ -Wall -Wwrite-strings -Woverloaded-virtual -Wno-sign-compare -fPIC -Wl,-rpath

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

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

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

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

2018-02-13 Thread GitBox
cjolivier01 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_r167916969 ## File path:

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

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

[incubator-mxnet] branch master updated: add missed optimizer docs (#9779)

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 81dc02e add missed optimizer

[GitHub] cjolivier01 closed pull request #9779: add missed optimizer docs

2018-02-13 Thread GitBox
cjolivier01 closed pull request #9779: add missed optimizer docs URL: https://github.com/apache/incubator-mxnet/pull/9779 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] 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_r167922565 ## File path:

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

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

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

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

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

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

[GitHub] cjolivier01 opened a new pull request #9785: remove error output

2018-02-13 Thread GitBox
cjolivier01 opened a new pull request #9785: remove error output URL: https://github.com/apache/incubator-mxnet/pull/9785 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### - [ ] Passed code style checking (`make lint`) - [ ]

[incubator-mxnet] branch master updated: Update loss.md (#9778)

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 e64b7b8 Update loss.md (#9778)

[GitHub] cjolivier01 closed pull request #9778: Update loss.md

2018-02-13 Thread GitBox
cjolivier01 closed pull request #9778: Update loss.md URL: https://github.com/apache/incubator-mxnet/pull/9778 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use

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

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

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

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

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

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

[GitHub] cjolivier01 commented on issue #9761: Don't use FIRST_CUDA on generators which don't support toolsets

2018-02-13 Thread GitBox
cjolivier01 commented on issue #9761: Don't use FIRST_CUDA on generators which don't support toolsets URL: https://github.com/apache/incubator-mxnet/pull/9761#issuecomment-365294184 simply building isn?t what was broken ? cmake knowing how to build cuda files natively was broken by your

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

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

[GitHub] cjolivier01 commented on a change in pull request #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc.

2018-02-13 Thread GitBox
cjolivier01 commented on a change in pull request #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc. URL: https://github.com/apache/incubator-mxnet/pull/8972#discussion_r167925647 ## File path: python/mxnet/profiler.py ## @@ -54,5 +60,280 @@

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

[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

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

[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] cjolivier01 opened a new pull request #9786: Add C++ modules to CODEOWNERS

2018-02-13 Thread GitBox
cjolivier01 opened a new pull request #9786: Add C++ modules to CODEOWNERS URL: https://github.com/apache/incubator-mxnet/pull/9786 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### - [ ] Passed code style checking (`make

[GitHub] cjolivier01 commented on issue #9786: Add C++ modules to CODEOWNERS

2018-02-13 Thread GitBox
cjolivier01 commented on issue #9786: Add C++ modules to CODEOWNERS URL: https://github.com/apache/incubator-mxnet/pull/9786#issuecomment-365323929 Added myself to everything but would be great if other ppl can add themselves so I can take my name off of some and not get spammed so much :)

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

[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] 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 commented on a change in pull request #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc.

2018-02-13 Thread GitBox
cjolivier01 commented on a change in pull request #8972: Profiling enhancements, python API, vtune and chrome tracing objects, etc. URL: https://github.com/apache/incubator-mxnet/pull/8972#discussion_r167926714 ## File path: python/mxnet/profiler.py ## @@ -54,5 +60,280 @@

[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_r167944698 ## File path: tests/python/unittest/test_metric.py ## @@ -56,18 +55,51 @@ def

[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] anirudh2290 commented on issue #9763: Too many header files need to be included when using C++ api

2018-02-13 Thread GitBox
anirudh2290 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-365349065 @marcoabreu should we have a different tag for CPP_PACKAGE? I have seen a bunch of issues for the

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

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

2018-02-13 Thread GitBox
cjolivier01 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_r167938189 ## File path:

[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-365348395 I think the current way is fine. All we need to do is to provide a mx.metric version of sklearn.metrics.f1_score.

[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

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

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

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

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

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

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

[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] 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 I've included my

  1   2   >