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

2018-01-04 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] cookingbear opened a new issue #9317: why's the function asnumpy() so slow?

2018-01-04 Thread GitBox
cookingbear opened a new issue #9317: why's the function asnumpy() so slow? URL: https://github.com/apache/incubator-mxnet/issues/9317 I tried the speech_recognition example but found out that the last step below(pred = pred.asnumpy()) slow down the process. why's that? `def upda

[GitHub] adaaaaaa commented on issue #8671: Discussion and troubleshooting on PyPI (pip) installation

2018-01-04 Thread GitBox
adaa commented on issue #8671: Discussion and troubleshooting on PyPI (pip) installation URL: https://github.com/apache/incubator-mxnet/issues/8671#issuecomment-355477966 cu90?cu91???benchmark?? This is an automated

[GitHub] jeremiedb commented on issue #4817: Fine-tune with R API

2018-01-04 Thread GitBox
jeremiedb commented on issue #4817: Fine-tune with R API URL: https://github.com/apache/incubator-mxnet/issues/4817#issuecomment-355477273 @lichen11, please see answer in #7968 This is an automated message from the Apache Git

[GitHub] jeremiedb commented on issue #7968: [R] Transfer Learning using VGG-16

2018-01-04 Thread GitBox
jeremiedb commented on issue #7968: [R] Transfer Learning using VGG-16 URL: https://github.com/apache/incubator-mxnet/issues/7968#issuecomment-355477039 Just made a few tests with different ResNet models and I also experienced crashes. Issue appears tied with a memory that isn't re

[incubator-mxnet] branch master updated: standard update for sparse sgd_mom_update (#9189)

2018-01-04 Thread haibin
This is an automated email from the ASF dual-hosted git repository. haibin 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 df9f79a standard update for sparse sgd

[GitHub] eric-haibin-lin closed pull request #9189: standard update for sparse sgd_mom_update

2018-01-04 Thread GitBox
eric-haibin-lin closed pull request #9189: standard update for sparse sgd_mom_update URL: https://github.com/apache/incubator-mxnet/pull/9189 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 thi

[GitHub] feevos opened a new issue #9316: bug in parameter default values of gluon.loss.Loss

2018-01-04 Thread GitBox
feevos opened a new issue #9316: bug in parameter default values of gluon.loss.Loss URL: https://github.com/apache/incubator-mxnet/issues/9316 ## Description (I think!) there is a bug in the default values of the parameters ```weight``` and ```batch_axis``` of the module ```gluon.lo

[GitHub] piiswrong commented on a change in pull request #9283: Fix custom op multi-gpu scaling

2018-01-04 Thread GitBox
piiswrong commented on a change in pull request #9283: Fix custom op multi-gpu scaling URL: https://github.com/apache/incubator-mxnet/pull/9283#discussion_r159808070 ## File path: src/operator/custom/custom-inl.h ## @@ -63,11 +64,80 @@ class Registry { return nullptr;

[GitHub] anirudh2290 commented on a change in pull request #9283: Fix custom op multi-gpu scaling

2018-01-04 Thread GitBox
anirudh2290 commented on a change in pull request #9283: Fix custom op multi-gpu scaling URL: https://github.com/apache/incubator-mxnet/pull/9283#discussion_r159801209 ## File path: src/operator/custom/custom-inl.h ## @@ -63,11 +64,80 @@ class Registry { return nullpt

[GitHub] Imshepherd commented on issue #8042: Convert network written in json format to mxnet symbol code?

2018-01-04 Thread GitBox
Imshepherd commented on issue #8042: Convert network written in json format to mxnet symbol code? URL: https://github.com/apache/incubator-mxnet/issues/8042#issuecomment-355457738 https://github.com/Imshepherd/MxNetR-Convert-json-to-symbol Example code which I wrote for R.

[GitHub] ZiyueHuang commented on issue #9310: model using contrib.SparseEmbedding returns inconsistent result between runs

2018-01-04 Thread GitBox
ZiyueHuang commented on issue #9310: model using contrib.SparseEmbedding returns inconsistent result between runs URL: https://github.com/apache/incubator-mxnet/issues/9310#issuecomment-355453091 Not sure. I have tried MF example using `mx.random.seed(0)` and `np.random.seed(0)`, the resu

[GitHub] lupesko commented on issue #9315: Refactor mxnet-scala code/tests/examples/docs to use "org.apache.mxnet" instead of "ml.dmlc.mxnet".

2018-01-04 Thread GitBox
lupesko commented on issue #9315: Refactor mxnet-scala code/tests/examples/docs to use "org.apache.mxnet" instead of "ml.dmlc.mxnet". URL: https://github.com/apache/incubator-mxnet/issues/9315#issuecomment-355450484 Looks good - thanks for picking this up! @nswamy please apply label

[GitHub] juliusshufan commented on a change in pull request #9305: Fixing two symbol files for CNN

2018-01-04 Thread GitBox
juliusshufan commented on a change in pull request #9305: Fixing two symbol files for CNN URL: https://github.com/apache/incubator-mxnet/pull/9305#discussion_r159794950 ## File path: example/image-classification/symbols/googlenet.py ## @@ -65,7 +65,7 @@ def get_symbol(num_

[GitHub] lichen11 commented on issue #4817: Fine-tune with R API

2018-01-04 Thread GitBox
lichen11 commented on issue #4817: Fine-tune with R API URL: https://github.com/apache/incubator-mxnet/issues/4817#issuecomment-355448008 Hi, I recently attempted fine-tuning ResNet101 based on your comments. However, when I initiate training (using GPU), R outputs the following msg:

[GitHub] juliusshufan commented on a change in pull request #9305: Fixing two symbol files for CNN

2018-01-04 Thread GitBox
juliusshufan commented on a change in pull request #9305: Fixing two symbol files for CNN URL: https://github.com/apache/incubator-mxnet/pull/9305#discussion_r159794950 ## File path: example/image-classification/symbols/googlenet.py ## @@ -65,7 +65,7 @@ def get_symbol(num_

[GitHub] juliusshufan commented on a change in pull request #9305: Fixing two symbol files for CNN

2018-01-04 Thread GitBox
juliusshufan commented on a change in pull request #9305: Fixing two symbol files for CNN URL: https://github.com/apache/incubator-mxnet/pull/9305#discussion_r159793640 ## File path: example/image-classification/symbols/vgg.py ## @@ -62,8 +62,12 @@ def get_symbol(num_class

[GitHub] szha commented on issue #9313: Fix CODEOWNERS syntax

2018-01-04 Thread GitBox
szha commented on issue #9313: Fix CODEOWNERS syntax URL: https://github.com/apache/incubator-mxnet/pull/9313#issuecomment-355440501 Relevant github doc https://help.github.com/articles/about-codeowners/ This is an automated m

[GitHub] szha closed pull request #9313: Fix CODEOWNERS syntax

2018-01-04 Thread GitBox
szha closed pull request #9313: Fix CODEOWNERS syntax URL: https://github.com/apache/incubator-mxnet/pull/9313 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 (fro

[incubator-mxnet] branch master updated: Fix CODEOWNERS syntax (#9313)

2018-01-04 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 9be50e0 Fix CODEOWNERS syntax (#9313

[GitHub] Roshrini opened a new issue #9315: Refactor mxnet-scala code/tests/examples/docs to use "org.apache.mxnet" instead of "ml.dmlc.mxnet".

2018-01-04 Thread GitBox
Roshrini opened a new issue #9315: Refactor mxnet-scala code/tests/examples/docs to use "org.apache.mxnet" instead of "ml.dmlc.mxnet". URL: https://github.com/apache/incubator-mxnet/issues/9315 ## Description Currently, MXNet-Scala library uses "ml.dmlc.mxnet" packages. As mxnet is unde

[GitHub] zheng-da commented on issue #8302: Refactor operators & MKLDNN

2018-01-04 Thread GitBox
zheng-da commented on issue #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#issuecomment-353705559 Remaining MKLDNN-related tasks before merging the PR. - [ ] Speedup inference with batch size 1. The goal is to reach the speed of old.mkldnn. Th

[GitHub] yuruofeifei commented on issue #8964: Fix example/module folder and remove duplicate examples

2018-01-04 Thread GitBox
yuruofeifei commented on issue #8964: Fix example/module folder and remove duplicate examples URL: https://github.com/apache/incubator-mxnet/pull/8964#issuecomment-355420085 @piiswrong @aaronmarkham can you please review? Remove some redundant tutorial files. ---

[GitHub] eric-haibin-lin opened a new issue #9314: nd.pick core dumps on large array

2018-01-04 Thread GitBox
eric-haibin-lin opened a new issue #9314: nd.pick core dumps on large array URL: https://github.com/apache/incubator-mxnet/issues/9314 ``` ubuntu@ip-172-31-13-31:~/mxnet$ python Python 2.7.12 (default, Nov 20 2017, 18:23:56) [GCC 5.4.0 20160609] on linux2 Type "help", "copyright"

[GitHub] marcoabreu opened a new pull request #9313: Fix CODEOWNERS syntax

2018-01-04 Thread GitBox
marcoabreu opened a new pull request #9313: Fix CODEOWNERS syntax URL: https://github.com/apache/incubator-mxnet/pull/9313 According to https://github.com/apache/incubator-mxnet, the notation /dir/ is necessary in order to cover all subdirectories. At the moment, only files on the first le

[GitHub] szha closed pull request #9312: test ci code owner

2018-01-04 Thread GitBox
szha closed pull request #9312: test ci code owner URL: https://github.com/apache/incubator-mxnet/pull/9312 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the

[incubator-mxnet] branch szha-patch-1 updated: Update README.md

2018-01-04 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch szha-patch-1 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/szha-patch-1 by this push: new bbc79bd Update README.md

[GitHub] szha opened a new pull request #9312: test ci code owner

2018-01-04 Thread GitBox
szha opened a new pull request #9312: test ci code owner URL: https://github.com/apache/incubator-mxnet/pull/9312 ## Description ## (Brief description on what this PR is about) @marcoabreu ## Checklist ## ### Essentials ### - [ ] Passed code style checking (`make lint`)

[incubator-mxnet] branch master updated: Add Marco de Abreu to CODEOWNERS related to CI (#9309)

2018-01-04 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 1602b12 Add Marco de Abreu to CODEOW

[GitHub] szha closed pull request #9309: Add Marco de Abreu to CODEOWNERS related to CI

2018-01-04 Thread GitBox
szha closed pull request #9309: Add Marco de Abreu to CODEOWNERS related to CI URL: https://github.com/apache/incubator-mxnet/pull/9309 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] szha commented on issue #9309: Add Marco de Abreu to CODEOWNERS related to CI

2018-01-04 Thread GitBox
szha commented on issue #9309: Add Marco de Abreu to CODEOWNERS related to CI URL: https://github.com/apache/incubator-mxnet/pull/9309#issuecomment-355406701 I agree that people who contributed and are familiar with the CI should be notified of changes to make sure the CI runs smoothly. Thi

[GitHub] machuck commented on issue #8962: Please add windows python package for 1.0.0 and 0.12.1 on PyPI

2018-01-04 Thread GitBox
machuck commented on issue #8962: Please add windows python package for 1.0.0 and 0.12.1 on PyPI URL: https://github.com/apache/incubator-mxnet/issues/8962#issuecomment-355403734 find all versions by: `pip search mxnet` For cpu only version `pip install keras-mxnet` -

[GitHub] aaronmarkham commented on issue #9145: Docs build fails on master

2018-01-04 Thread GitBox
aaronmarkham commented on issue #9145: Docs build fails on master URL: https://github.com/apache/incubator-mxnet/issues/9145#issuecomment-355402284 The docs are built in a Docker container. With the latest build from master, Docker hits a memory limit which causes the build to fail, and th

[GitHub] aaronmarkham closed issue #9145: Docs build fails on master

2018-01-04 Thread GitBox
aaronmarkham closed issue #9145: Docs build fails on master URL: https://github.com/apache/incubator-mxnet/issues/9145 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] sxjscience commented on issue #9310: model using contrib.SparseEmbedding returns inconsistent result between runs

2018-01-04 Thread GitBox
sxjscience commented on issue #9310: model using contrib.SparseEmbedding returns inconsistent result between runs URL: https://github.com/apache/incubator-mxnet/issues/9310#issuecomment-355398375 The result of summation would depend on the order of the additions because float32 has limite

[GitHub] eric-haibin-lin commented on issue #9310: model using contrib.SparseEmbedding returns inconsistent result between runs

2018-01-04 Thread GitBox
eric-haibin-lin commented on issue #9310: model using contrib.SparseEmbedding returns inconsistent result between runs URL: https://github.com/apache/incubator-mxnet/issues/9310#issuecomment-355397586 Why would the result of `atomicAdd ` non-deterministic? Isn't it atomic? ---

[GitHub] sxjscience commented on issue #9310: model using contrib.SparseEmbedding returns inconsistent result between runs

2018-01-04 Thread GitBox
sxjscience commented on issue #9310: model using contrib.SparseEmbedding returns inconsistent result between runs URL: https://github.com/apache/incubator-mxnet/issues/9310#issuecomment-355397275 OK, I've missed the GPU part. The result of atomicAdd would be non-deterministic. However, th

[GitHub] eric-haibin-lin commented on issue #9310: model using contrib.SparseEmbedding returns inconsistent result between runs

2018-01-04 Thread GitBox
eric-haibin-lin commented on issue #9310: model using contrib.SparseEmbedding returns inconsistent result between runs URL: https://github.com/apache/incubator-mxnet/issues/9310#issuecomment-355395924 The GPU kernel is using atomic add : https://github.com/apache/incubator-mxnet/blob/mast

[GitHub] cbalioglu opened a new pull request #9311: Prepend LD_LIBRARY_PATH to dll_path in find_lib_path

2018-01-04 Thread GitBox
cbalioglu opened a new pull request #9311: Prepend LD_LIBRARY_PATH to dll_path in find_lib_path URL: https://github.com/apache/incubator-mxnet/pull/9311 ## Description ## The paths contained in the LD_LIBRARY_PATH environment variable should have the highest priority while searching for

[GitHub] sxjscience commented on issue #9310: model using contrib.SparseEmbedding returns inconsistent result between runs

2018-01-04 Thread GitBox
sxjscience commented on issue #9310: model using contrib.SparseEmbedding returns inconsistent result between runs URL: https://github.com/apache/incubator-mxnet/issues/9310#issuecomment-355395149 Looks like again the atomicAdd problem https://github.com/apache/incubator-mxnet/blob/master/

[incubator-mxnet] branch v1.0.0 updated: Fix the gradient of gather_nd (#9200)

2018-01-04 Thread sxjscience
This is an automated email from the ASF dual-hosted git repository. sxjscience pushed a commit to branch v1.0.0 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/v1.0.0 by this push: new 5585393 Fix the gradient of gather

[GitHub] marcoabreu commented on issue #9296: Disable test_operator.test_laop_3 (temp fix #9295)

2018-01-04 Thread GitBox
marcoabreu commented on issue #9296: Disable test_operator.test_laop_3 (temp fix #9295) URL: https://github.com/apache/incubator-mxnet/pull/9296#issuecomment-355388353 Totally agree and that's why I have asked Steffen to put this on the usability sprint for last month. Unfortunately, these

[incubator-mxnet] branch v1.0.0 updated: bugfix for parallel rand generator on multi-gpu (#9300)

2018-01-04 Thread liuyizhi
This is an automated email from the ASF dual-hosted git repository. liuyizhi pushed a commit to branch v1.0.0 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/v1.0.0 by this push: new e4ff7c5 bugfix for parallel rand gen

[GitHub] lichen11 commented on issue #7968: [R] Transfer Learning using VGG-16

2018-01-04 Thread GitBox
lichen11 commented on issue #7968: [R] Transfer Learning using VGG-16 URL: https://github.com/apache/incubator-mxnet/issues/7968#issuecomment-355379685 Hi, I recently attempted transfer learning on ResNet101. I only retrain the last fully connected layer. resnet101<- mx.model.l

[GitHub] pracheer commented on issue #8339: data iterators tutorial errors

2018-01-04 Thread GitBox
pracheer commented on issue #8339: data iterators tutorial errors URL: https://github.com/apache/incubator-mxnet/issues/8339#issuecomment-355379267 This is due to the behavior of zip function which was changed in python 3. Will try to send a PR for this in a couple of days. ---

[GitHub] szha commented on issue #9284: error on the print(net.weight)

2018-01-04 Thread GitBox
szha commented on issue #9284: error on the print(net.weight) URL: https://github.com/apache/incubator-mxnet/issues/9284#issuecomment-355378955 Yes, this has been fixed in the latest version https://github.com/apache/incubator-mxnet/blob/master/python/mxnet/gluon/parameter.py#L120

[incubator-mxnet] branch master updated: Fix the gradient of gather_nd (#9200)

2018-01-04 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 d918868 Fix the gradient of gather_nd (#

[GitHub] piiswrong closed pull request #9200: Fix the gradient of gather_nd

2018-01-04 Thread GitBox
piiswrong closed pull request #9200: Fix the gradient of gather_nd URL: https://github.com/apache/incubator-mxnet/pull/9200 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

[incubator-mxnet] branch master updated: Add label_from_zero_one argument to LogisticLoss (#9265)

2018-01-04 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 c3616ed Add label_from_zero_one argument

[GitHub] piiswrong closed pull request #9265: Add label_from_zero_one argument to LogisticLoss

2018-01-04 Thread GitBox
piiswrong closed pull request #9265: Add label_from_zero_one argument to LogisticLoss URL: https://github.com/apache/incubator-mxnet/pull/9265 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 th

[GitHub] piiswrong closed pull request #9290: add epoch parameter to export() of HybridBlock

2018-01-04 Thread GitBox
piiswrong closed pull request #9290: add epoch parameter to export() of HybridBlock URL: https://github.com/apache/incubator-mxnet/pull/9290 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

[incubator-mxnet] branch master updated: add epoch parameter to export() of HybridBlock (#9290)

2018-01-04 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 4988535 add epoch parameter to export()

[GitHub] gamenn commented on issue #9294: Trouble installing on Windows

2018-01-04 Thread GitBox
gamenn commented on issue #9294: Trouble installing on Windows URL: https://github.com/apache/incubator-mxnet/issues/9294#issuecomment-355189033 @zhreshold What exactly are the path variable names MXNet looks for? And what exactly are the dlls it needs? I verified that the path on my machi

[GitHub] gamenn commented on issue #9294: Trouble installing on Windows

2018-01-04 Thread GitBox
gamenn commented on issue #9294: Trouble installing on Windows URL: https://github.com/apache/incubator-mxnet/issues/9294#issuecomment-355189033 @zhreshold What exactly are the path variable names MXNet looks for? And what exactly are the dlls it needs? I verified that the path on my machi

[GitHub] eric-haibin-lin commented on issue #9284: error on the print(net.weight)

2018-01-04 Thread GitBox
eric-haibin-lin commented on issue #9284: error on the print(net.weight) URL: https://github.com/apache/incubator-mxnet/issues/9284#issuecomment-355369790 @szha was this fixed? This is an automated message from the Apache Git

[GitHub] piiswrong commented on issue #9296: Disable test_operator.test_laop_3 (temp fix #9295)

2018-01-04 Thread GitBox
piiswrong commented on issue #9296: Disable test_operator.test_laop_3 (temp fix #9295) URL: https://github.com/apache/incubator-mxnet/pull/9296#issuecomment-355369010 We've removed many tests, but AFAIK none has been fixed and reintroduced. This doesn't look good.

[GitHub] piiswrong commented on a change in pull request #9305: Fixing two symbol files for CNN

2018-01-04 Thread GitBox
piiswrong commented on a change in pull request #9305: Fixing two symbol files for CNN URL: https://github.com/apache/incubator-mxnet/pull/9305#discussion_r159729629 ## File path: example/image-classification/symbols/vgg.py ## @@ -62,8 +62,12 @@ def get_symbol(num_classes,

[GitHub] piiswrong commented on a change in pull request #9305: Fixing two symbol files for CNN

2018-01-04 Thread GitBox
piiswrong commented on a change in pull request #9305: Fixing two symbol files for CNN URL: https://github.com/apache/incubator-mxnet/pull/9305#discussion_r159729510 ## File path: example/image-classification/symbols/googlenet.py ## @@ -65,7 +65,7 @@ def get_symbol(num_cla

[GitHub] eric-haibin-lin opened a new issue #9310: model using contrib.SparseEmbedding returns inconsistent result between runs

2018-01-04 Thread GitBox
eric-haibin-lin opened a new issue #9310: model using contrib.SparseEmbedding returns inconsistent result between runs URL: https://github.com/apache/incubator-mxnet/issues/9310 Note: Providing complete information in the most concise form is the best way to get help. This issue template s

[GitHub] KellenSunderland commented on issue #9055: Re-enable all op segments when in batch mode

2018-01-04 Thread GitBox
KellenSunderland commented on issue #9055: Re-enable all op segments when in batch mode URL: https://github.com/apache/incubator-mxnet/pull/9055#issuecomment-355358324 Ok, don't merge just yet, I'll fix indentation. Edit: should be fixed.

[GitHub] gamenn commented on issue #9294: Trouble installing on Windows

2018-01-04 Thread GitBox
gamenn commented on issue #9294: Trouble installing on Windows URL: https://github.com/apache/incubator-mxnet/issues/9294#issuecomment-355189033 @zhreshold What exactly are the path variable names MXNet looks for? And what exactly are the dlls it needs? ---

[GitHub] yuxiangw commented on a change in pull request #9220: Signum optimizer

2018-01-04 Thread GitBox
yuxiangw commented on a change in pull request #9220: Signum optimizer URL: https://github.com/apache/incubator-mxnet/pull/9220#discussion_r159721991 ## File path: src/operator/optimizer_op.cc ## @@ -35,6 +35,74 @@ DMLC_REGISTER_PARAMETER(AdamParam); DMLC_REGISTER_PARAMETE

[GitHub] yuxiangw commented on a change in pull request #9220: Signum optimizer

2018-01-04 Thread GitBox
yuxiangw commented on a change in pull request #9220: Signum optimizer URL: https://github.com/apache/incubator-mxnet/pull/9220#discussion_r159722018 ## File path: src/operator/optimizer_op.cc ## @@ -35,6 +35,74 @@ DMLC_REGISTER_PARAMETER(AdamParam); DMLC_REGISTER_PARAMETE

[GitHub] yuxiangw commented on a change in pull request #9220: Signum optimizer

2018-01-04 Thread GitBox
yuxiangw commented on a change in pull request #9220: Signum optimizer URL: https://github.com/apache/incubator-mxnet/pull/9220#discussion_r159722025 ## File path: src/operator/optimizer_op.cc ## @@ -35,6 +35,74 @@ DMLC_REGISTER_PARAMETER(AdamParam); DMLC_REGISTER_PARAMETE

[GitHub] yuxiangw commented on a change in pull request #9220: Signum optimizer

2018-01-04 Thread GitBox
yuxiangw commented on a change in pull request #9220: Signum optimizer URL: https://github.com/apache/incubator-mxnet/pull/9220#discussion_r159722059 ## File path: src/operator/optimizer_op.cu ## @@ -28,6 +28,14 @@ namespace mxnet { namespace op { +NNVM_REGISTER_OP(sign

[GitHub] yuxiangw commented on a change in pull request #9220: Signum optimizer

2018-01-04 Thread GitBox
yuxiangw commented on a change in pull request #9220: Signum optimizer URL: https://github.com/apache/incubator-mxnet/pull/9220#discussion_r159721999 ## File path: src/operator/optimizer_op.cc ## @@ -35,6 +35,74 @@ DMLC_REGISTER_PARAMETER(AdamParam); DMLC_REGISTER_PARAMETE

[GitHub] yuxiangw commented on a change in pull request #9220: Signum optimizer

2018-01-04 Thread GitBox
yuxiangw commented on a change in pull request #9220: Signum optimizer URL: https://github.com/apache/incubator-mxnet/pull/9220#discussion_r159722050 ## File path: src/operator/optimizer_op.cc ## @@ -35,6 +35,74 @@ DMLC_REGISTER_PARAMETER(AdamParam); DMLC_REGISTER_PARAMETE

[GitHub] yuxiangw commented on a change in pull request #9220: Signum optimizer

2018-01-04 Thread GitBox
yuxiangw commented on a change in pull request #9220: Signum optimizer URL: https://github.com/apache/incubator-mxnet/pull/9220#discussion_r159722035 ## File path: src/operator/optimizer_op.cc ## @@ -35,6 +35,74 @@ DMLC_REGISTER_PARAMETER(AdamParam); DMLC_REGISTER_PARAMETE

[GitHub] yuxiangw commented on a change in pull request #9220: Signum optimizer

2018-01-04 Thread GitBox
yuxiangw commented on a change in pull request #9220: Signum optimizer URL: https://github.com/apache/incubator-mxnet/pull/9220#discussion_r159721868 ## File path: python/mxnet/optimizer.py ## @@ -529,6 +534,69 @@ def update_multi_precision(self, index, weight, grad, state

[GitHub] yuxiangw commented on a change in pull request #9220: Signum optimizer

2018-01-04 Thread GitBox
yuxiangw commented on a change in pull request #9220: Signum optimizer URL: https://github.com/apache/incubator-mxnet/pull/9220#discussion_r159721842 ## File path: python/mxnet/optimizer.py ## @@ -57,6 +58,10 @@ class Optimizer(object): The weight decay (or L2 regu

[GitHub] KellenSunderland commented on issue #9055: Re-enable all op segments when in batch mode

2018-01-04 Thread GitBox
KellenSunderland commented on issue #9055: Re-enable all op segments when in batch mode URL: https://github.com/apache/incubator-mxnet/pull/9055#issuecomment-355358324 Ok, don't merge just yet, I'll fix indentation. This is a

[GitHub] marcoabreu commented on issue #9300: bugfix for parallel rand generator on multi-gpu

2018-01-04 Thread GitBox
marcoabreu commented on issue #9300: bugfix for parallel rand generator on multi-gpu URL: https://github.com/apache/incubator-mxnet/pull/9300#issuecomment-355357167 The CUDA library is going to be needed anyways, right? So that shouldn't be an issue. Couldn't this be achieved by trying to

[GitHub] marcoabreu commented on issue #9296: Disable test_operator.test_laop_3 (temp fix #9295)

2018-01-04 Thread GitBox
marcoabreu commented on issue #9296: Disable test_operator.test_laop_3 (temp fix #9295) URL: https://github.com/apache/incubator-mxnet/pull/9296#issuecomment-355356394 I haven't had time to investigate this issue yet and did not contact the author. @KellenSunderland @larroy what do you guy

[GitHub] cjolivier01 commented on issue #9300: bugfix for parallel rand generator on multi-gpu

2018-01-04 Thread GitBox
cjolivier01 commented on issue #9300: bugfix for parallel rand generator on multi-gpu URL: https://github.com/apache/incubator-mxnet/pull/9300#issuecomment-355355959 I am not aware of an API call for that from python. One would need to be created, which would also force the CUDA library to

[GitHub] eric-haibin-lin commented on a change in pull request #9055: Re-enable all op segments when in batch mode

2018-01-04 Thread GitBox
eric-haibin-lin commented on a change in pull request #9055: Re-enable all op segments when in batch mode URL: https://github.com/apache/incubator-mxnet/pull/9055#discussion_r159717876 ## File path: src/executor/graph_executor.cc ## @@ -1406,13 +1416,33 @@ void GraphExecut

[GitHub] eric-haibin-lin commented on a change in pull request #9055: Re-enable all op segments when in batch mode

2018-01-04 Thread GitBox
eric-haibin-lin commented on a change in pull request #9055: Re-enable all op segments when in batch mode URL: https://github.com/apache/incubator-mxnet/pull/9055#discussion_r159717666 ## File path: src/executor/graph_executor.cc ## @@ -1370,20 +1380,20 @@ void GraphExecut

[GitHub] eric-haibin-lin commented on a change in pull request #9055: Re-enable all op segments when in batch mode

2018-01-04 Thread GitBox
eric-haibin-lin commented on a change in pull request #9055: Re-enable all op segments when in batch mode URL: https://github.com/apache/incubator-mxnet/pull/9055#discussion_r159717980 ## File path: src/executor/graph_executor.cc ## @@ -1348,17 +1348,27 @@ void GraphExecut

[GitHub] cjolivier01 commented on issue #9296: Disable test_operator.test_laop_3 (temp fix #9295)

2018-01-04 Thread GitBox
cjolivier01 commented on issue #9296: Disable test_operator.test_laop_3 (temp fix #9295) URL: https://github.com/apache/incubator-mxnet/pull/9296#issuecomment-355354373 This is an lapack test, right? If so, is it on the lapack author's to-do list? Has any investigation been done? I'm co

[GitHub] eric-haibin-lin commented on a change in pull request #9220: Signum optimizer

2018-01-04 Thread GitBox
eric-haibin-lin commented on a change in pull request #9220: Signum optimizer URL: https://github.com/apache/incubator-mxnet/pull/9220#discussion_r159717200 ## File path: src/operator/optimizer_op-inl.h ## @@ -1088,6 +1090,146 @@ inline void FtrlUpdateEx(const nnvm::NodeAtt

commits@mxnet.incubator.apache.org

2018-01-04 Thread GitBox
eric-haibin-lin closed pull request #9303: fix wide&deep URL: https://github.com/apache/incubator-mxnet/pull/9303 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: fix w&d (#9303)

2018-01-04 Thread haibin
This is an automated email from the ASF dual-hosted git repository. haibin 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 5e587e5 fix w&d (#9303) 5e587e5 is des

[incubator-mxnet] branch master updated: Increase timeout for GIT on windows (Fix #9297) (#9298)

2018-01-04 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 469955d Increase timeout for GIT

[GitHub] cjolivier01 closed issue #9297: git checkout on windows slaves times out

2018-01-04 Thread GitBox
cjolivier01 closed issue #9297: git checkout on windows slaves times out URL: https://github.com/apache/incubator-mxnet/issues/9297 This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] cjolivier01 closed pull request #9298: Increase timeout for GIT on windows (Fix #9297)

2018-01-04 Thread GitBox
cjolivier01 closed pull request #9298: Increase timeout for GIT on windows (Fix #9297) URL: https://github.com/apache/incubator-mxnet/pull/9298 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 t

[GitHub] marcoabreu commented on issue #9300: bugfix for parallel rand generator on multi-gpu

2018-01-04 Thread GitBox
marcoabreu commented on issue #9300: bugfix for parallel rand generator on multi-gpu URL: https://github.com/apache/incubator-mxnet/pull/9300#issuecomment-355353106 Relying on nvidia-smi could cause issues as behaviour on windows and unix tend to differ. It should be possible to get the nu

[GitHub] marcoabreu opened a new pull request #9309: Add Marco de Abreu to CODEOWNERS related to CI

2018-01-04 Thread GitBox
marcoabreu opened a new pull request #9309: Add Marco de Abreu to CODEOWNERS related to CI URL: https://github.com/apache/incubator-mxnet/pull/9309 There have been unreviewed changes like https://github.com/apache/incubator-mxnet/commit/d0388d00d5959d96341129a40ab21264476dfda6 to CI-relat

[GitHub] yuxiangw commented on a change in pull request #9220: Signum optimizer

2018-01-04 Thread GitBox
yuxiangw commented on a change in pull request #9220: Signum optimizer URL: https://github.com/apache/incubator-mxnet/pull/9220#discussion_r159714768 ## File path: src/operator/optimizer_op-inl.h ## @@ -1088,6 +1090,146 @@ inline void FtrlUpdateEx(const nnvm::NodeAttrs& att

[GitHub] mwbyeon commented on issue #9308: batch size in inference

2018-01-04 Thread GitBox
mwbyeon commented on issue #9308: batch size in inference URL: https://github.com/apache/incubator-mxnet/issues/9308#issuecomment-355328275 This is an example. 1. Set the shape of input data with batch size. `(batch_size, channel, height, width)` https://github.com/apache/incub

[GitHub] mwbyeon commented on issue #9308: batch size in inference

2018-01-04 Thread GitBox
mwbyeon commented on issue #9308: batch size in inference URL: https://github.com/apache/incubator-mxnet/issues/9308#issuecomment-355328275 This is an example. 1. Set the shape of input data with batch size. `(batch_size, channel, height, width)` https://github.com/apache/incub

[GitHub] mwbyeon commented on issue #9308: batch size in inference

2018-01-04 Thread GitBox
mwbyeon commented on issue #9308: batch size in inference URL: https://github.com/apache/incubator-mxnet/issues/9308#issuecomment-355328275 This is an example. Fist, Set the shape of input data with batch size. `(batch_size, channel, height, width)` https://github.com/apache/in

[GitHub] KellenSunderland commented on issue #9055: Re-enable all op segments when in batch mode

2018-01-04 Thread GitBox
KellenSunderland commented on issue #9055: Re-enable all op segments when in batch mode URL: https://github.com/apache/incubator-mxnet/pull/9055#issuecomment-355328018 Rebased. Would also be great if you could have a look @larroy

[GitHub] gamenn commented on issue #9294: Trouble installing on Windows

2018-01-04 Thread GitBox
gamenn commented on issue #9294: Trouble installing on Windows URL: https://github.com/apache/incubator-mxnet/issues/9294#issuecomment-355189033 What exactly are the path variable names MXNet looks for? And what exactly are the dlls it needs?

[GitHub] cjolivier01 commented on issue #9300: bugfix for parallel rand generator on multi-gpu

2018-01-04 Thread GitBox
cjolivier01 commented on issue #9300: bugfix for parallel rand generator on multi-gpu URL: https://github.com/apache/incubator-mxnet/pull/9300#issuecomment-355314141 For example, run nvidia-smi -L and count number of lines returned which start with "GPU [0-9]+" or something like that

[GitHub] cjolivier01 commented on issue #9300: bugfix for parallel rand generator on multi-gpu

2018-01-04 Thread GitBox
cjolivier01 commented on issue #9300: bugfix for parallel rand generator on multi-gpu URL: https://github.com/apache/incubator-mxnet/pull/9300#issuecomment-355313475 Current test machines in CI have at least two GPUs. However, you will need to detect GPU count in the unit test because when

[incubator-mxnet] branch master updated: bugfix for parallel rand generator on multi-gpu (#9300)

2018-01-04 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 661bcef bugfix for parallel rand

[GitHub] cjolivier01 closed pull request #9300: bugfix for parallel rand generator on multi-gpu

2018-01-04 Thread GitBox
cjolivier01 closed pull request #9300: bugfix for parallel rand generator on multi-gpu URL: https://github.com/apache/incubator-mxnet/pull/9300 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 t

[GitHub] BenLag2906 opened a new issue #9308: batch size in inference

2018-01-04 Thread GitBox
BenLag2906 opened a new issue #9308: batch size in inference URL: https://github.com/apache/incubator-mxnet/issues/9308 I work with your version of mxnet under windows in c++. I have heard of it was possible to make inference/prediction with a batch size. This practise is usually onl

[GitHub] asmushetzel commented on issue #8456: setting OMP_NUM_THREADS doesn't improve performance with MKL2017

2018-01-04 Thread GitBox
asmushetzel commented on issue #8456: setting OMP_NUM_THREADS doesn't improve performance with MKL2017 URL: https://github.com/apache/incubator-mxnet/issues/8456#issuecomment-355293256 MKL does multithreading internally (also based on openmp, but not controlled by above environment variab

[GitHub] KeyKy closed issue #9289: How to get the symbol type?

2018-01-04 Thread GitBox
KeyKy closed issue #9289: How to get the symbol type? URL: https://github.com/apache/incubator-mxnet/issues/9289 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and us

[GitHub] KeyKy commented on issue #9289: How to get the symbol type?

2018-01-04 Thread GitBox
KeyKy commented on issue #9289: How to get the symbol type? URL: https://github.com/apache/incubator-mxnet/issues/9289#issuecomment-355279353 THX! This is an automated message from the Apache Git Service. To respond to the me

  1   2   >