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

2018-03-03 Thread GitBox
reminisce commented on issue #9552: [REQUEST FOR REVIEW | DO NOT MERGE] Model Quantization with Calibration URL: https://github.com/apache/incubator-mxnet/pull/9552#issuecomment-370205130 @wentingj Yes, I noticed that too. I think there is still value of doing this. We can always keep impr

[GitHub] thinksanky commented on issue #55: Fixing 404 issue caused due to missing how_to directory

2018-03-03 Thread GitBox
thinksanky commented on issue #55: Fixing 404 issue caused due to missing how_to directory URL: https://github.com/apache/incubator-mxnet-site/pull/55#issuecomment-370204760 @yzhliu This is an automated message from the Apa

[GitHub] thinksanky opened a new pull request #55: Fixing 404 issue caused due to missing how_to directory

2018-03-03 Thread GitBox
thinksanky opened a new pull request #55: Fixing 404 issue caused due to missing how_to directory URL: https://github.com/apache/incubator-mxnet-site/pull/55 ## Description ## * Added the how_to folder with all the missing files and redirected to faq/. * Example redirection - https

[GitHub] reminisce commented on a change in pull request #9887: Non-blocking row_sparse_pull

2018-03-03 Thread GitBox
reminisce commented on a change in pull request #9887: Non-blocking row_sparse_pull URL: https://github.com/apache/incubator-mxnet/pull/9887#discussion_r172038187 ## File path: tests/python/unittest/test_kvstore.py ## @@ -76,7 +76,7 @@ def check_row_sparse_pull(kv, count)

[GitHub] reminisce commented on a change in pull request #9981: [fix issue#9976] The assignment problem in NDArray

2018-03-03 Thread GitBox
reminisce commented on a change in pull request #9981: [fix issue#9976] The assignment problem in NDArray URL: https://github.com/apache/incubator-mxnet/pull/9981#discussion_r172037661 ## File path: tests/python/unittest/test_ndarray.py ## @@ -1099,6 +1099,39 @@ def test_a

[GitHub] reminisce commented on a change in pull request #9981: [fix issue#9976] The assignment problem in NDArray

2018-03-03 Thread GitBox
reminisce commented on a change in pull request #9981: [fix issue#9976] The assignment problem in NDArray URL: https://github.com/apache/incubator-mxnet/pull/9981#discussion_r172037688 ## File path: tests/python/unittest/test_ndarray.py ## @@ -1099,6 +1099,39 @@ def test_a

[GitHub] reminisce commented on a change in pull request #9981: [fix issue#9976] The assignment problem in NDArray

2018-03-03 Thread GitBox
reminisce commented on a change in pull request #9981: [fix issue#9976] The assignment problem in NDArray URL: https://github.com/apache/incubator-mxnet/pull/9981#discussion_r172037698 ## File path: tests/python/unittest/test_ndarray.py ## @@ -1099,6 +1099,39 @@ def test_a

[GitHub] Feywell commented on issue #9572: Return a NaN when using operator ( ** ) on Windows version with GPU

2018-03-03 Thread GitBox
Feywell commented on issue #9572: Return a NaN when using operator ( ** ) on Windows version with GPU URL: https://github.com/apache/incubator-mxnet/issues/9572#issuecomment-370198874 @cgraywang I find the issue still exist. I upgraded the mxnet-cu80 windows version to > 1.1.0b2

[GitHub] wkcn commented on a change in pull request #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version)

2018-03-03 Thread GitBox
wkcn commented on a change in pull request #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#discussion_r172035107 ## File path: src/operator/contrib/proposal.cu ## @@ -553

[GitHub] precedenceguo commented on a change in pull request #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version)

2018-03-03 Thread GitBox
precedenceguo commented on a change in pull request #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#discussion_r172034936 ## File path: src/operator/contrib/proposal.cu ##

[GitHub] precedenceguo commented on a change in pull request #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version)

2018-03-03 Thread GitBox
precedenceguo commented on a change in pull request #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#discussion_r172034936 ## File path: src/operator/contrib/proposal.cu ##

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

2018-03-03 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-370193815 @reminisce But MKLDNN int8 kernel has some performance benefit. Besides, If we fuse conv and relu into

[GitHub] eric-haibin-lin opened a new pull request #9983: [WIP] language model with google's billion words dataset

2018-03-03 Thread GitBox
eric-haibin-lin opened a new pull request #9983: [WIP] language model with google's billion words dataset URL: https://github.com/apache/incubator-mxnet/pull/9983 ## Description ## Reproduced LSTMP 2048-512 baseline, achieving test ppl ~42 in https://arxiv.org/pdf/1602.02410.pdf The

[GitHub] marcoabreu commented on issue #9975: revert DiagrammeR compatibility to 0.9.0

2018-03-03 Thread GitBox
marcoabreu commented on issue #9975: revert DiagrammeR compatibility to 0.9.0 URL: https://github.com/apache/incubator-mxnet/pull/9975#issuecomment-370187701 Is this still compatible with 1.0.0? This is an automated message fr

[GitHub] ShownX commented on issue #9405: BucketingModule causes uncaught exception if symbol name not specified

2018-03-03 Thread GitBox
ShownX commented on issue #9405: BucketingModule causes uncaught exception if symbol name not specified URL: https://github.com/apache/incubator-mxnet/issues/9405#issuecomment-370181587 Facing the same problem. However, in Gluon, it cannot define a name ---

[GitHub] szha closed pull request #9633: Gluon image-classification example improvement

2018-03-03 Thread GitBox
szha closed pull request #9633: Gluon image-classification example improvement URL: https://github.com/apache/incubator-mxnet/pull/9633 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: Gluon image-classification example improvement (#9633)

2018-03-03 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 8780096 Gluon image-classification e

[GitHub] CoinCheung commented on issue #9978: Error with random generator

2018-03-03 Thread GitBox
CoinCheung commented on issue #9978: Error with random generator URL: https://github.com/apache/incubator-mxnet/issues/9978#issuecomment-370145440 @asitstands Ah, I see. Thanks a lot. I think this issue can be closed now. ---

[GitHub] CoinCheung closed issue #9978: Error with random generator

2018-03-03 Thread GitBox
CoinCheung closed issue #9978: Error with random generator URL: https://github.com/apache/incubator-mxnet/issues/9978 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub a

[incubator-mxnet] branch master updated: sparse regression operators (#9625)

2018-03-03 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 dedfd2d sparse regression operators (#

[GitHub] eric-haibin-lin closed pull request #9625: sparse regression operators

2018-03-03 Thread GitBox
eric-haibin-lin closed pull request #9625: sparse regression operators URL: https://github.com/apache/incubator-mxnet/pull/9625 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] marcoabreu commented on issue #9981: [fix issue#9976] The assignment problem in NDArray

2018-03-03 Thread GitBox
marcoabreu commented on issue #9981: [fix issue#9976] The assignment problem in NDArray URL: https://github.com/apache/incubator-mxnet/pull/9981#issuecomment-370141962 @piiswrong This is an automated message from the Apache

[GitHub] asitstands commented on issue #9978: Error with random generator

2018-03-03 Thread GitBox
asitstands commented on issue #9978: Error with random generator URL: https://github.com/apache/incubator-mxnet/issues/9978#issuecomment-370140101 @CoinCheung That is the usual way that pseudo random number generators work. They generate a sequence of pseudo random numbers. So each call of

[GitHub] asitstands commented on issue #9978: Error with random generator

2018-03-03 Thread GitBox
asitstands commented on issue #9978: Error with random generator URL: https://github.com/apache/incubator-mxnet/issues/9978#issuecomment-370140101 @CoinCheung That is the usual way that pseudo random number generators work. They generate a sequence of pseudo random numbers. So each call of

[GitHub] asitstands commented on issue #9978: Error with random generator

2018-03-03 Thread GitBox
asitstands commented on issue #9978: Error with random generator URL: https://github.com/apache/incubator-mxnet/issues/9978#issuecomment-370140101 @CoinCheung That is the usual way that pseudo random number generators work. They generate a sequence of pseudo random numbers. So each call of

[GitHub] asitstands commented on issue #9978: Error with random generator

2018-03-03 Thread GitBox
asitstands commented on issue #9978: Error with random generator URL: https://github.com/apache/incubator-mxnet/issues/9978#issuecomment-370140101 @CoinCheung That is the usual way that pseudo random number generators work. They generate a sequence of pseudo random numbers. So each call of

[GitHub] dma100180 commented on issue #4527: MXNet for R on Windows installation fails due to latest update in DiagremmeR

2018-03-03 Thread GitBox
dma100180 commented on issue #4527: MXNet for R on Windows installation fails due to latest update in DiagremmeR URL: https://github.com/apache/incubator-mxnet/issues/4527#issuecomment-370139374 Hello, everything was fine with MXNET and an update has caused me to change the diagrammer.

[GitHub] wkcn commented on issue #9981: [fix issue#9976] The assignment problem in NDArray

2018-03-03 Thread GitBox
wkcn commented on issue #9981: [fix issue#9976] The assignment problem in NDArray URL: https://github.com/apache/incubator-mxnet/pull/9981#issuecomment-370139225 @marcoabreu Yes, I will add it soon :) This is an automated mes

[GitHub] marcoabreu commented on issue #9981: [fix issue#9976] The assignment problem in NDArray

2018-03-03 Thread GitBox
marcoabreu commented on issue #9981: [fix issue#9976] The assignment problem in NDArray URL: https://github.com/apache/incubator-mxnet/pull/9981#issuecomment-370138765 Hello, could you please add a test for this case? This is

[GitHub] CoinCheung commented on issue #9978: Error with random generator

2018-03-03 Thread GitBox
CoinCheung commented on issue #9978: Error with random generator URL: https://github.com/apache/incubator-mxnet/issues/9978#issuecomment-370138600 @asitstands Then why it changes when I run it in my command line? ``` $ python >>> import mxnet as mx >>> import num

[GitHub] mseeger commented on issue #9982: Unary ops logcdf_normal, derivlogcdf_normal for log CDF of standard n?

2018-03-03 Thread GitBox
mseeger commented on issue #9982: Unary ops logcdf_normal, derivlogcdf_normal for log CDF of standard n? URL: https://github.com/apache/incubator-mxnet/pull/9982#issuecomment-370138558 Sorry, this is just #9961 reopened, I deleted the remote branch.

[GitHub] mseeger opened a new pull request #9982: Unary ops logcdf_normal, derivlogcdf_normal for log CDF of standard n?

2018-03-03 Thread GitBox
mseeger opened a new pull request #9982: Unary ops logcdf_normal, derivlogcdf_normal for log CDF of standard n? URL: https://github.com/apache/incubator-mxnet/pull/9982 ?ormal * New unary ops * Improved unit tests for basic unary, binary ops ## Description ## (Brief

[GitHub] mseeger closed pull request #9961: Unary ops norm_logcdf, norm_derivlogcdf for log CDF of standard normal

2018-03-03 Thread GitBox
mseeger closed pull request #9961: Unary ops norm_logcdf, norm_derivlogcdf for log CDF of standard normal URL: https://github.com/apache/incubator-mxnet/pull/9961 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake o

[GitHub] wkcn commented on issue #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version)

2018-03-03 Thread GitBox
wkcn commented on issue #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#issuecomment-369813766 @pengzhao-intel Thank you! I will have a try. ~~I use `#pragma omp paraller for` for e

[GitHub] wkcn commented on issue #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version)

2018-03-03 Thread GitBox
wkcn commented on issue #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#issuecomment-370133350 @pengzhao-intel Here is the testing code. https://gist.github.com/wkcn/4a09c142bc9886b45b5

[GitHub] wkcn commented on issue #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version)

2018-03-03 Thread GitBox
wkcn commented on issue #9939: add multi proposal operator (cpu version) and fix the bug in proposal op (gpu version) URL: https://github.com/apache/incubator-mxnet/pull/9939#issuecomment-370133350 @pengzhao-intel Here is the testing code. https://gist.github.com/wkcn/4a09c142bc9886b45b5

[GitHub] wkcn opened a new pull request #9981: [fix issue#9976] The assignment problem in NDArray

2018-03-03 Thread GitBox
wkcn opened a new pull request #9981: [fix issue#9976] The assignment problem in NDArray URL: https://github.com/apache/incubator-mxnet/pull/9981 ## Description ## Fix issue[#9976](https://github.com/apache/incubator-mxnet/issues/9976) ```python >>> a = mx.nd.array(np.arange(12