[GitHub] wentingj commented on issue #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-01-24 Thread GitBox
wentingj commented on issue #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#issuecomment-360378383 hi, @ZihengJiang, what are the layers using quantized version to get the accuracy from

[GitHub] szha commented on issue #9514: Language Modeling Datasets and Sampler

2018-01-24 Thread GitBox
szha commented on issue #9514: Language Modeling Datasets and Sampler URL: https://github.com/apache/incubator-mxnet/pull/9514#issuecomment-360378242 I updated this PR based on the latest change in text api naming. Also, I made the vocabulary as a property of the dataset for exchanging the

[GitHub] eric-haibin-lin commented on a change in pull request #9562: Broken Link Fixes

2018-01-24 Thread GitBox
eric-haibin-lin commented on a change in pull request #9562: Broken Link Fixes URL: https://github.com/apache/incubator-mxnet/pull/9562#discussion_r163760316 ## File path: docs/faq/bucketing.md ## @@ -84,7 +84,7 @@ and then randomly chooses sequences from that bucket to

[GitHub] eric-haibin-lin commented on a change in pull request #9562: Broken Link Fixes

2018-01-24 Thread GitBox
eric-haibin-lin commented on a change in pull request #9562: Broken Link Fixes URL: https://github.com/apache/incubator-mxnet/pull/9562#discussion_r163760376 ## File path: docs/faq/add_op_in_backend.md ## @@ -0,0 +1,608 @@ +# A Beginner's Guide to Implementing Operators in

[GitHub] Rikorose commented on issue #9531: KeyError: in mx.nd.array.empty()

2018-01-24 Thread GitBox
Rikorose commented on issue #9531: KeyError: in mx.nd.array.empty() URL: https://github.com/apache/incubator-mxnet/issues/9531#issuecomment-360371878 As I said, this error didn't occur consistently. Now I cannot reproduce it anymore. Anyway here is my code using numpy array since I do

[GitHub] thinksanky opened a new pull request #9562: Broken Link Fixes

2018-01-24 Thread GitBox
thinksanky opened a new pull request #9562: Broken Link Fixes URL: https://github.com/apache/incubator-mxnet/pull/9562 ## Description ## - Fixed broken links that were pointing to how_to and now pointing to faq. - Added 2 files that were still pointing to how_to.

[GitHub] benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2018-01-24 Thread GitBox
benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-360368130 Done. This is an automated message from the

[GitHub] aaronmarkham commented on issue #9535: Tutorials index page update

2018-01-24 Thread GitBox
aaronmarkham commented on issue #9535: Tutorials index page update URL: https://github.com/apache/incubator-mxnet/pull/9535#issuecomment-360366173 Updates are applied to the preview here: http://ec2-34-204-183-27.compute-1.amazonaws.com/tutorials/index.html The preview also includes

[GitHub] thinksanky commented on issue #9548: updated release version to 1.0.1 on the mainpage and re-arranged news?

2018-01-24 Thread GitBox
thinksanky commented on issue #9548: updated release version to 1.0.1 on the mainpage and re-arranged news? URL: https://github.com/apache/incubator-mxnet/pull/9548#issuecomment-360359436 https://user-images.githubusercontent.com/31976455/35371175-55019514-0147-11e8-9cb9-f68f5811b26e.png;>

[GitHub] Soonhwan-Kwon opened a new pull request #9561: fix CreateOp of slice_channel to accept dtypes of uint8 and int32

2018-01-24 Thread GitBox
Soonhwan-Kwon opened a new pull request #9561: fix CreateOp of slice_channel to accept dtypes of uint8 and int32 URL: https://github.com/apache/incubator-mxnet/pull/9561 ## Description ## To deal with quantization in future, operators like slice_channel(split) should have capability

[GitHub] eric-haibin-lin commented on issue #9558: Signum with grad compression

2018-01-24 Thread GitBox
eric-haibin-lin commented on issue #9558: Signum with grad compression URL: https://github.com/apache/incubator-mxnet/pull/9558#issuecomment-360356840 @rahul003 This is an automated message from the Apache Git Service. To

[GitHub] DickJC123 opened a new pull request #9560: Expand gpu-kernel-launch synchronous error checking.

2018-01-24 Thread GitBox
DickJC123 opened a new pull request #9560: Expand gpu-kernel-launch synchronous error checking. URL: https://github.com/apache/incubator-mxnet/pull/9560 Background: @zhreshold and @szha discovered that if MXNet were built without a suitable kernel binary or PTX for the GPU in the system,

[GitHub] zhreshold commented on issue #9555: 0 to the power of 0 is nan on GPU

2018-01-24 Thread GitBox
zhreshold commented on issue #9555: 0 to the power of 0 is nan on GPU URL: https://github.com/apache/incubator-mxnet/issues/9555#issuecomment-360349327 It's mathmatically undefined behavior but should made consistant with numpy and cpu impl. So I guess it's CUDA `powf` function to blame?

[GitHub] sxjscience commented on issue #9555: 0 to the power of 0 is nan on GPU

2018-01-24 Thread GitBox
sxjscience commented on issue #9555: 0 to the power of 0 is nan on GPU URL: https://github.com/apache/incubator-mxnet/issues/9555#issuecomment-360346281 I think nan is the correct behavior as 0 power 0 is undefined. This is

[GitHub] mbaijal opened a new pull request #9559: License Adds - some more

2018-01-24 Thread GitBox
mbaijal opened a new pull request #9559: License Adds - some more URL: https://github.com/apache/incubator-mxnet/pull/9559 ## Description ## Fixing Licenses some more @bhavinthaker @marcoabreu Please review Note: For details, Please refer to this wiki -

[GitHub] yuxiangw opened a new pull request #9558: Signum with grad compression

2018-01-24 Thread GitBox
yuxiangw opened a new pull request #9558: Signum with grad compression URL: https://github.com/apache/incubator-mxnet/pull/9558 ## Description ## Added a new grad compression scheme. It keeps track of the momentum on every worker and sends the sign vector of the momentum. This is

[GitHub] yuewu001 opened a new issue #9557: update_on_kvstore error setting with multiple machines

2018-01-24 Thread GitBox
yuewu001 opened a new issue #9557: update_on_kvstore error setting with multiple machines URL: https://github.com/apache/incubator-mxnet/issues/9557 When I was training with multiple machines, i found that [model.py:_create_kvstore

[GitHub] thinksanky commented on issue #9548: updated release version to 1.0.1 on the mainpage and re-arranged news?

2018-01-24 Thread GitBox
thinksanky commented on issue #9548: updated release version to 1.0.1 on the mainpage and re-arranged news? URL: https://github.com/apache/incubator-mxnet/pull/9548#issuecomment-360344889 https://user-images.githubusercontent.com/31976455/35368317-a85f484c-0137-11e8-86d0-40e45e3a51f4.png;>

[GitHub] mbaijal opened a new pull request #9556: [Review Required] Fixing Licenses in the Docs folder wherever possible

2018-01-24 Thread GitBox
mbaijal opened a new pull request #9556: [Review Required] Fixing Licenses in the Docs folder wherever possible URL: https://github.com/apache/incubator-mxnet/pull/9556 ## Description ## Fixing Licenses in the Docs/ folder wherever possible Please refer to this wiki -

[GitHub] aaronmarkham commented on issue #9535: Tutorials index page update

2018-01-24 Thread GitBox
aaronmarkham commented on issue #9535: Tutorials index page update URL: https://github.com/apache/incubator-mxnet/pull/9535#issuecomment-360172206 Preview: http://ec2-34-204-183-27.compute-1.amazonaws.com/tutorials/index.html

[GitHub] aaronmarkham commented on issue #9535: Tutorials index page update

2018-01-24 Thread GitBox
aaronmarkham commented on issue #9535: Tutorials index page update URL: https://github.com/apache/incubator-mxnet/pull/9535#issuecomment-360172206 I'm using a flakey python server for this, and also you can't seem to connect to it when also on VPN... so let me know if this works for you:

[GitHub] aaronmarkham commented on a change in pull request #9535: Tutorials index page update

2018-01-24 Thread GitBox
aaronmarkham commented on a change in pull request #9535: Tutorials index page update URL: https://github.com/apache/incubator-mxnet/pull/9535#discussion_r163735769 ## File path: docs/tutorials/index.md ## @@ -1,84 +1,319 @@ # Tutorials -## Gluon +MXNet has two primary

[GitHub] zhreshold opened a new issue #9555: 0 to the power of 0 is nan on GPU

2018-01-24 Thread GitBox
zhreshold opened a new issue #9555: 0 to the power of 0 is nan on GPU URL: https://github.com/apache/incubator-mxnet/issues/9555 A simple bug on gpu only ``` import mxnet as mx import numpy as np print(mx.nd.array([0, 1e-30], ctx=mx.cpu()) ** 0) # [1. 1.]

[GitHub] szha commented on a change in pull request #9485: Gluon vision transforms

2018-01-24 Thread GitBox
szha commented on a change in pull request #9485: Gluon vision transforms URL: https://github.com/apache/incubator-mxnet/pull/9485#discussion_r163735370 ## File path: python/mxnet/gluon/data/vision/datasets.py ## @@ -28,9 +28,9 @@ import warnings import numpy as np

[GitHub] thinksanky commented on issue #9548: updated release version to 1.0.1 on the mainpage and re-arranged news?

2018-01-24 Thread GitBox
thinksanky commented on issue #9548: updated release version to 1.0.1 on the mainpage and re-arranged news? URL: https://github.com/apache/incubator-mxnet/pull/9548#issuecomment-360338896 https://user-images.githubusercontent.com/31976455/35367267-5660de48-0132-11e8-8238-22542118d1f0.png;>

[GitHub] thinksanky commented on issue #9548: updated release version to 1.0.1 on the mainpage and re-arranged news?

2018-01-24 Thread GitBox
thinksanky commented on issue #9548: updated release version to 1.0.1 on the mainpage and re-arranged news? URL: https://github.com/apache/incubator-mxnet/pull/9548#issuecomment-360338896 https://user-images.githubusercontent.com/31976455/35367267-5660de48-0132-11e8-8238-22542118d1f0.png;>

[GitHub] rahul003 commented on a change in pull request #8915: NVLink communication pattern updated

2018-01-24 Thread GitBox
rahul003 commented on a change in pull request #8915: NVLink communication pattern updated URL: https://github.com/apache/incubator-mxnet/pull/8915#discussion_r163710038 ## File path: src/kvstore/comm.h ## @@ -670,35 +822,67 @@ class CommDevice : public Comm { using

[GitHub] rahul003 commented on a change in pull request #8915: NVLink communication pattern updated

2018-01-24 Thread GitBox
rahul003 commented on a change in pull request #8915: NVLink communication pattern updated URL: https://github.com/apache/incubator-mxnet/pull/8915#discussion_r163717663 ## File path: src/kvstore/comm.h ## @@ -670,35 +822,67 @@ class CommDevice : public Comm { using

[GitHub] rahul003 commented on a change in pull request #8915: NVLink communication pattern updated

2018-01-24 Thread GitBox
rahul003 commented on a change in pull request #8915: NVLink communication pattern updated URL: https://github.com/apache/incubator-mxnet/pull/8915#discussion_r163722287 ## File path: src/kvstore/comm.h ## @@ -526,101 +541,238 @@ class CommDevice : public Comm { }

[GitHub] rahul003 commented on a change in pull request #8915: NVLink communication pattern updated

2018-01-24 Thread GitBox
rahul003 commented on a change in pull request #8915: NVLink communication pattern updated URL: https://github.com/apache/incubator-mxnet/pull/8915#discussion_r163724995 ## File path: src/kvstore/comm.h ## @@ -526,101 +541,238 @@ class CommDevice : public Comm { }

[GitHub] mbaijal opened a new pull request #9554: [Review Required] Adding Apache License header to the perl-package

2018-01-24 Thread GitBox
mbaijal opened a new pull request #9554: [Review Required] Adding Apache License header to the perl-package URL: https://github.com/apache/incubator-mxnet/pull/9554 ## Description ## Added the ASF header to the Perl package ## Checklist ## ### Essentials ### - [ ] Passed

[GitHub] joeddav commented on issue #1432: How can I clear the memory usage?

2018-01-24 Thread GitBox
joeddav commented on issue #1432: How can I clear the memory usage? URL: https://github.com/apache/incubator-mxnet/issues/1432#issuecomment-360330159 This worked for me: ```python del mod gc.collect() # memory should be freed ```

[GitHub] joeddav commented on issue #1432: How can I clear the memory usage?

2018-01-24 Thread GitBox
joeddav commented on issue #1432: How can I clear the memory usage? URL: https://github.com/apache/incubator-mxnet/issues/1432#issuecomment-360330159 This worked for me: ``` del mod gc.collect() # memory should be freed ```

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

2018-01-24 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_r163727577 ## File path: perl-package/AI-MXNet/lib/AI/MXNet/Profiler.pm

[GitHub] eric-haibin-lin closed pull request #9551: add three pretrained english fasttext embedding

2018-01-24 Thread GitBox
eric-haibin-lin closed pull request #9551: add three pretrained english fasttext embedding URL: https://github.com/apache/incubator-mxnet/pull/9551 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: add three pretrained english fasttext embedding (#9551)

2018-01-24 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 e57426f add three pretrained english

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

2018-01-24 Thread GitBox
sergeykolychev 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_r163720464 ## File path: perl-package/AI-MXNet/lib/AI/MXNet/Profiler.pm

[GitHub] marcoabreu opened a new issue #9553: test_operator_gpu.test_correlation fails

2018-01-24 Thread GitBox
marcoabreu opened a new issue #9553: test_operator_gpu.test_correlation fails URL: https://github.com/apache/incubator-mxnet/issues/9553 http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/master/284/pipeline Job: Python3: MKLML-GPU Command:

[GitHub] zhreshold commented on issue #8430: NDArrayIter doesn't preserve dtype of labels

2018-01-24 Thread GitBox
zhreshold commented on issue #8430: NDArrayIter doesn't preserve dtype of labels URL: https://github.com/apache/incubator-mxnet/issues/8430#issuecomment-360321045 This seems to be related to a behavior that when converting nympy array to NDArray, it won't preserve dtype by default.

[GitHub] reminisce opened a new pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration

2018-01-24 Thread GitBox
reminisce opened a new pull request #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552 ## Description ## This PR implements model **quantization** by adopting the [TensorFlow

[GitHub] szha commented on issue #8430: NDArrayIter doesn't preserve dtype of labels

2018-01-24 Thread GitBox
szha commented on issue #8430: NDArrayIter doesn't preserve dtype of labels URL: https://github.com/apache/incubator-mxnet/issues/8430#issuecomment-360320083 @apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and needs triage. For general

[incubator-mxnet] branch master updated: Data-iterator tutorial made python3 compatible. (#9460)

2018-01-24 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 5166e57 Data-iterator tutorial made

[GitHub] piiswrong closed pull request #9460: Data-iterator tutorial made python3 compatible.

2018-01-24 Thread GitBox
piiswrong closed pull request #9460: Data-iterator tutorial made python3 compatible. URL: https://github.com/apache/incubator-mxnet/pull/9460 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

[GitHub] KellenSunderland commented on issue #9537: float16 have no performance improvemen in training cifar10

2018-01-24 Thread GitBox
KellenSunderland commented on issue #9537: float16 have no performance improvemen in training cifar10 URL: https://github.com/apache/incubator-mxnet/issues/9537#issuecomment-360312345 Hello @gzh1991, thanks for aaking this question, as I'm sure it's of interest to a few users.

[incubator-mxnet] branch master updated: Bugfix: kAddTo support for swap_axes. (#9541)

2018-01-24 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 5f9dd2c Bugfix: kAddTo support for

[GitHub] piiswrong closed pull request #9541: Bugfix: kAddTo support for swap_axes.

2018-01-24 Thread GitBox
piiswrong closed pull request #9541: Bugfix: kAddTo support for swap_axes. URL: https://github.com/apache/incubator-mxnet/pull/9541 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 opened a new pull request #9551: add three pretrained english fasttext embedding

2018-01-24 Thread GitBox
szha opened a new pull request #9551: add three pretrained english fasttext embedding URL: https://github.com/apache/incubator-mxnet/pull/9551 ## Description ## Add three word-embeddings from fasttext for english. ## Checklist ## ### Essentials ### - [x] Passed code style

[GitHub] eric-haibin-lin commented on a change in pull request #9535: Tutorials index page update

2018-01-24 Thread GitBox
eric-haibin-lin commented on a change in pull request #9535: Tutorials index page update URL: https://github.com/apache/incubator-mxnet/pull/9535#discussion_r163709852 ## File path: docs/tutorials/index.md ## @@ -1,84 +1,319 @@ # Tutorials -## Gluon +MXNet has two

[GitHub] eric-haibin-lin commented on a change in pull request #9535: Tutorials index page update

2018-01-24 Thread GitBox
eric-haibin-lin commented on a change in pull request #9535: Tutorials index page update URL: https://github.com/apache/incubator-mxnet/pull/9535#discussion_r163709740 ## File path: docs/tutorials/index.md ## @@ -1,84 +1,319 @@ # Tutorials -## Gluon +MXNet has two

[incubator-mxnet] branch master updated: Text APIs: interface re-design with name changes (#9534)

2018-01-24 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 a1f1321 Text APIs: interface

[GitHub] szha closed pull request #9534: Text APIs: interface re-design with name changes

2018-01-24 Thread GitBox
szha closed pull request #9534: Text APIs: interface re-design with name changes URL: https://github.com/apache/incubator-mxnet/pull/9534 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

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

2018-01-24 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-360298273 @hyandell Can you check out the Perl changes, in this commit, please?

[GitHub] lupesko commented on a change in pull request #9548: updated release version to 1.0.1 on the mainpage and re-arranged news?

2018-01-24 Thread GitBox
lupesko commented on a change in pull request #9548: updated release version to 1.0.1 on the mainpage and re-arranged news? URL: https://github.com/apache/incubator-mxnet/pull/9548#discussion_r163696739 ## File path: docs/_static/mxnet-theme/index.html ## @@ -17,27 +17,26

[GitHub] lupesko commented on a change in pull request #9548: updated release version to 1.0.1 on the mainpage and re-arranged news?

2018-01-24 Thread GitBox
lupesko commented on a change in pull request #9548: updated release version to 1.0.1 on the mainpage and re-arranged news? URL: https://github.com/apache/incubator-mxnet/pull/9548#discussion_r163696942 ## File path: docs/_static/mxnet-theme/index.html ## @@ -17,27 +17,26

[incubator-mxnet] branch master updated: Update __init__.py (#9549)

2018-01-24 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 323fdcb Update __init__.py (#9549)

[GitHub] eric-haibin-lin closed pull request #9549: Update model zoo vision __init__.py

2018-01-24 Thread GitBox
eric-haibin-lin closed pull request #9549: Update model zoo vision __init__.py URL: https://github.com/apache/incubator-mxnet/pull/9549 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] marcoabreu commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2018-01-24 Thread GitBox
marcoabreu commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-360284529 Great, thanks! Feel free to just make the change and push the commit, the CI will take care of the rest

[GitHub] marcoabreu closed issue #9546: Automatically test for addto for ALL operators

2018-01-24 Thread GitBox
marcoabreu closed issue #9546: Automatically test for addto for ALL operators URL: https://github.com/apache/incubator-mxnet/issues/9546 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] marcoabreu commented on issue #9546: Automatically test for addto for ALL operators

2018-01-24 Thread GitBox
marcoabreu commented on issue #9546: Automatically test for addto for ALL operators URL: https://github.com/apache/incubator-mxnet/issues/9546#issuecomment-360284243 Ah, thanks! Closing this issue then. This is an automated

[GitHub] marcoabreu commented on issue #9541: Bugfix: kAddTo support for swap_axes.

2018-01-24 Thread GitBox
marcoabreu commented on issue #9541: Bugfix: kAddTo support for swap_axes. URL: https://github.com/apache/incubator-mxnet/pull/9541#issuecomment-360243492 Tracked at https://github.com/apache/incubator-mxnet/issues/9497 @tdomhan would you mind adding a test in a second PR? It would

[GitHub] benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2018-01-24 Thread GitBox
benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-360282441 I did rebase. I wonder if just changing to ```scala val predLabel = if (pred.shape ==

[GitHub] szha commented on issue #9534: Text APIs: interface re-design with name changes

2018-01-24 Thread GitBox
szha commented on issue #9534: Text APIs: interface re-design with name changes URL: https://github.com/apache/incubator-mxnet/pull/9534#issuecomment-360280219 This PR looks good to me now. Since all review comments have been addressed, I'm merging this once CI passes. Thank @piiswrong

[GitHub] aaronmarkham commented on a change in pull request #9548: updated release version to 1.0.1 on the mainpage and re-arranged news?

2018-01-24 Thread GitBox
aaronmarkham commented on a change in pull request #9548: updated release version to 1.0.1 on the mainpage and re-arranged news? URL: https://github.com/apache/incubator-mxnet/pull/9548#discussion_r163681396 ## File path: docs/_static/mxnet-theme/index.html ## @@ -17,27

[GitHub] aaronmarkham commented on a change in pull request #9548: updated release version to 1.0.1 on the mainpage and re-arranged news?

2018-01-24 Thread GitBox
aaronmarkham commented on a change in pull request #9548: updated release version to 1.0.1 on the mainpage and re-arranged news? URL: https://github.com/apache/incubator-mxnet/pull/9548#discussion_r163681626 ## File path: docs/_static/mxnet-theme/index.html ## @@ -17,27

[GitHub] aaronmarkham commented on a change in pull request #9548: updated release version to 1.0.1 on the mainpage and re-arranged news?

2018-01-24 Thread GitBox
aaronmarkham commented on a change in pull request #9548: updated release version to 1.0.1 on the mainpage and re-arranged news? URL: https://github.com/apache/incubator-mxnet/pull/9548#discussion_r163681396 ## File path: docs/_static/mxnet-theme/index.html ## @@ -17,27

[GitHub] asmushetzel commented on issue #9546: Automatically test for addto for ALL operators

2018-01-24 Thread GitBox
asmushetzel commented on issue #9546: Automatically test for addto for ALL operators URL: https://github.com/apache/incubator-mxnet/issues/9546#issuecomment-360276475 There is a similar issue already open #9497 May be we should use that one (or close that but we should have one

[GitHub] eric-haibin-lin commented on a change in pull request #9540: refactor regression ops to nnvm interface

2018-01-24 Thread GitBox
eric-haibin-lin commented on a change in pull request #9540: refactor regression ops to nnvm interface URL: https://github.com/apache/incubator-mxnet/pull/9540#discussion_r163678029 ## File path: src/operator/regression_output-inl.h ## @@ -50,146 +50,90 @@ struct

[GitHub] eric-haibin-lin commented on a change in pull request #9540: refactor regression ops to nnvm interface

2018-01-24 Thread GitBox
eric-haibin-lin commented on a change in pull request #9540: refactor regression ops to nnvm interface URL: https://github.com/apache/incubator-mxnet/pull/9540#discussion_r163679077 ## File path: src/operator/regression_output.cc ## @@ -106,13 +135,40 @@ It is suitable

[GitHub] eric-haibin-lin commented on a change in pull request #9540: refactor regression ops to nnvm interface

2018-01-24 Thread GitBox
eric-haibin-lin commented on a change in pull request #9540: refactor regression ops to nnvm interface URL: https://github.com/apache/incubator-mxnet/pull/9540#discussion_r163678720 ## File path: src/operator/regression_output.cc ## @@ -18,86 +18,115 @@ */ /*! - *

[GitHub] eric-haibin-lin commented on a change in pull request #9540: refactor regression ops to nnvm interface

2018-01-24 Thread GitBox
eric-haibin-lin commented on a change in pull request #9540: refactor regression ops to nnvm interface URL: https://github.com/apache/incubator-mxnet/pull/9540#discussion_r163678059 ## File path: src/operator/regression_output-inl.h ## @@ -50,146 +50,90 @@ struct

[GitHub] szha commented on a change in pull request #9534: Text APIs: interface re-design with name changes

2018-01-24 Thread GitBox
szha commented on a change in pull request #9534: Text APIs: interface re-design with name changes URL: https://github.com/apache/incubator-mxnet/pull/9534#discussion_r163677150 ## File path: docs/api/python/index.md ## @@ -151,3 +142,13 @@ imported by running:

[GitHub] szha commented on a change in pull request #9534: Text APIs: interface re-design with name changes

2018-01-24 Thread GitBox
szha commented on a change in pull request #9534: Text APIs: interface re-design with name changes URL: https://github.com/apache/incubator-mxnet/pull/9534#discussion_r163676904 ## File path: docs/api/python/contrib/contrib.md ## @@ -0,0 +1,12 @@ +# Contrib APIs Review

[GitHub] szha opened a new pull request #9549: Update model zoo vision __init__.py

2018-01-24 Thread GitBox
szha opened a new pull request #9549: Update model zoo vision __init__.py URL: https://github.com/apache/incubator-mxnet/pull/9549 ## Description ## Remove reference to pytorch since the manifest already acknowledges the sources. ## Checklist ## ### Essentials ### - [x]

[GitHub] szha commented on a change in pull request #9548: updated release version to 1.0.1 on the mainpage and re-arranged news?

2018-01-24 Thread GitBox
szha commented on a change in pull request #9548: updated release version to 1.0.1 on the mainpage and re-arranged news? URL: https://github.com/apache/incubator-mxnet/pull/9548#discussion_r163673884 ## File path: docs/_static/mxnet-theme/index.html ## @@ -17,27 +17,26 @@

[GitHub] thinksanky commented on issue #9548: updated release version to 1.0.1 on the mainpage and re-arranged news?

2018-01-24 Thread GitBox
thinksanky commented on issue #9548: updated release version to 1.0.1 on the mainpage and re-arranged news? URL: https://github.com/apache/incubator-mxnet/pull/9548#issuecomment-360268024 https://user-images.githubusercontent.com/31976455/35356227-a5f18d28-0104-11e8-8d7f-9d8e231f717e.png;>

[GitHub] thinksanky commented on issue #9548: updated release version to 1.0.1 on the mainpage and re-arranged news?

2018-01-24 Thread GitBox
thinksanky commented on issue #9548: updated release version to 1.0.1 on the mainpage and re-arranged news? URL: https://github.com/apache/incubator-mxnet/pull/9548#issuecomment-360267282 Note - API Menu should also be collapsed under Docs Menu. I am working on that. Since I saw some

[GitHub] thinksanky opened a new pull request #9548: updated release version to 1.0.1 on the mainpage and re-arranged news?

2018-01-24 Thread GitBox
thinksanky opened a new pull request #9548: updated release version to 1.0.1 on the mainpage and re-arranged news? URL: https://github.com/apache/incubator-mxnet/pull/9548 ## Description ## This PR has the following changes: * Collapsed Tutorials under Docs Nav Menu. *

[GitHub] analog-cbarber opened a new issue #9547: Implementing new operators in an external module

2018-01-24 Thread GitBox
analog-cbarber opened a new issue #9547: Implementing new operators in an external module URL: https://github.com/apache/incubator-mxnet/issues/9547 Currently it is possible to implement new operators in C++/CUDA, but it requires you to have your own copy of the source tree and build it

[GitHub] marcoabreu commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2018-01-24 Thread GitBox
marcoabreu commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-360258410 @benqua would you rebasing? That way everybody can see the problem and other people might be able to

[GitHub] benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2018-01-24 Thread GitBox
benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-360257217 Hi, Sorry for the long delay. The problem is that while this change fix my problem on segmentation

[GitHub] piiswrong commented on issue #9534: Text APIs: interface re-design with name changes

2018-01-24 Thread GitBox
piiswrong commented on issue #9534: Text APIs: interface re-design with name changes URL: https://github.com/apache/incubator-mxnet/pull/9534#issuecomment-360245144 Please move doc to contrib. Otherwise LGTM This is an

[GitHub] marcoabreu commented on issue #9541: Bugfix: kAddTo support for swap_axes.

2018-01-24 Thread GitBox
marcoabreu commented on issue #9541: Bugfix: kAddTo support for swap_axes. URL: https://github.com/apache/incubator-mxnet/pull/9541#issuecomment-360243492 Tracked at https://github.com/apache/incubator-mxnet/issues/9546. @tdomhan would you mind adding a test in a second PR? It would

[GitHub] marcoabreu opened a new issue #9546: Automatically test for addto for ALL operators

2018-01-24 Thread GitBox
marcoabreu opened a new issue #9546: Automatically test for addto for ALL operators URL: https://github.com/apache/incubator-mxnet/issues/9546 See https://github.com/apache/incubator-mxnet/pull/9541#issuecomment-360240969

[GitHub] piiswrong commented on issue #9541: Bugfix: kAddTo support for swap_axes.

2018-01-24 Thread GitBox
piiswrong commented on issue #9541: Bugfix: kAddTo support for swap_axes. URL: https://github.com/apache/incubator-mxnet/pull/9541#issuecomment-360240969 I'm merging this because code freeze is today. Let's update tests later. @marcoabreu Please track this. We should actually test

[GitHub] analog-cbarber commented on issue #9536: zeros does not support int64

2018-01-24 Thread GitBox
analog-cbarber commented on issue #9536: zeros does not support int64 URL: https://github.com/apache/incubator-mxnet/issues/9536#issuecomment-360238422 BTW, are the list of officially supported dtypes documented anywhere? I cannot find it.

[GitHub] EmergentOrder commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2018-01-24 Thread GitBox
EmergentOrder commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-360238318 @benqua would appreciate if you could fix the conflict or rerun here so it can be merged. This fix

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

2018-01-24 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-360231723 @pengzhao-intel @TaoLv I openned up this issue to discuss:

[GitHub] cjolivier01 opened a new issue #9545: Profiling discussion

2018-01-24 Thread GitBox
cjolivier01 opened a new issue #9545: Profiling discussion URL: https://github.com/apache/incubator-mxnet/issues/9545 Openning a thread to discuss some items that came up in my Profiling PR, what aren;t directly related to the PR and may go on beyond the life of the PR

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

2018-01-24 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-360229323 @pengzhao-intel I think it'll also be good to add a Counter for the MKL free/allocs to track MKL

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

2018-01-24 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-360229016 @sergeykolychev Ping :) This is

[GitHub] sxjscience commented on issue #9513: get the names of different outputs of the same layer

2018-01-24 Thread GitBox
sxjscience commented on issue #9513: get the names of different outputs of the same layer URL: https://github.com/apache/incubator-mxnet/issues/9513#issuecomment-360227366 For the custom layers, the name of the outputs are controlled by `list_outputs()`. See

[GitHub] sxjscience closed issue #9478: Loss becomes NAN with larger, more layered network

2018-01-24 Thread GitBox
sxjscience closed issue #9478: Loss becomes NAN with larger, more layered network URL: https://github.com/apache/incubator-mxnet/issues/9478 This is an automated message from the Apache Git Service. To respond to the

[GitHub] sxjscience commented on issue #9531: KeyError: in mx.nd.array.empty()

2018-01-24 Thread GitBox
sxjscience commented on issue #9531: KeyError: in mx.nd.array.empty() URL: https://github.com/apache/incubator-mxnet/issues/9531#issuecomment-360226446 Would you post an example code that would trigger this error? This is

[GitHub] ashokei commented on issue #8918: Added in Large-Batch SGD with a warmup, and a LARS startegy. Also add?

2018-01-24 Thread GitBox
ashokei commented on issue #8918: Added in Large-Batch SGD with a warmup, and a LARS startegy. Also add? URL: https://github.com/apache/incubator-mxnet/pull/8918#issuecomment-358545377 @zhreshold i made the requested changes, please review. thank you,

[GitHub] sxjscience commented on a change in pull request #9540: refactor regression ops to nnvm interface

2018-01-24 Thread GitBox
sxjscience commented on a change in pull request #9540: refactor regression ops to nnvm interface URL: https://github.com/apache/incubator-mxnet/pull/9540#discussion_r163630374 ## File path: src/operator/regression_output.cc ## @@ -106,13 +135,40 @@ It is suitable for

[GitHub] sxjscience commented on issue #9533: Cannot find engine branch for minpy package

2018-01-24 Thread GitBox
sxjscience commented on issue #9533: Cannot find engine branch for minpy package URL: https://github.com/apache/incubator-mxnet/issues/9533#issuecomment-360220388 I think you may need to raise an issue in [dmlc/minpy](https://github.com/dmlc/minpy). Also, some features in minpy like

[GitHub] sxjscience commented on issue #9533: Cannot find engine branch for minpy package

2018-01-24 Thread GitBox
sxjscience commented on issue #9533: Cannot find engine branch for minpy package URL: https://github.com/apache/incubator-mxnet/issues/9533#issuecomment-360220388 I think you may need to raise an issue in minpy. Also, some features in minpy like

[GitHub] marcoabreu commented on issue #9537: float16 have no performance improvemen in training cifar10

2018-01-24 Thread GitBox
marcoabreu commented on issue #9537: float16 have no performance improvemen in training cifar10 URL: https://github.com/apache/incubator-mxnet/issues/9537#issuecomment-360215044 @KellenSunderland This is an automated

[GitHub] sxjscience commented on issue #9541: Bugfix: kAddTo support for swap_axes.

2018-01-24 Thread GitBox
sxjscience commented on issue #9541: Bugfix: kAddTo support for swap_axes. URL: https://github.com/apache/incubator-mxnet/pull/9541#issuecomment-360212548 Like my comments in https://github.com/apache/incubator-mxnet/pull/9495, we should also revise the test here

[GitHub] marcoabreu commented on issue #9542: Dockerfile to create Jetson TX1 and TX2 compatible builds.

2018-01-24 Thread GitBox
marcoabreu commented on issue #9542: Dockerfile to create Jetson TX1 and TX2 compatible builds. URL: https://github.com/apache/incubator-mxnet/pull/9542#issuecomment-360211452 I wouldn't mind seeing this added to the Jenkinsfile during build-stage

  1   2   >