[GitHub] piiswrong closed pull request #8254: Fix softmax_cross_entropy

2017-12-12 Thread GitBox
piiswrong closed pull request #8254: Fix softmax_cross_entropy URL: https://github.com/apache/incubator-mxnet/pull/8254 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] piiswrong commented on issue #8254: Fix softmax_cross_entropy

2017-12-12 Thread GitBox
piiswrong commented on issue #8254: Fix softmax_cross_entropy URL: https://github.com/apache/incubator-mxnet/pull/8254#issuecomment-351210798 closing due to inactive This is an automated message from the Apache Git Service.

[GitHub] piiswrong commented on issue #8218: Update RandomCropAug

2017-12-12 Thread GitBox
piiswrong commented on issue #8218: Update RandomCropAug URL: https://github.com/apache/incubator-mxnet/pull/8218#issuecomment-351211149 Could you rebase to master? This is an automated message from the Apache Git Service.

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2017-12-12 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r156510303 ## File path: src/operator/nn/activation-inl.h ## @@ -61,158 +62,127 @@ struct ActivationParam : public

[GitHub] piiswrong commented on issue #8114: fix memcost example

2017-12-12 Thread GitBox
piiswrong commented on issue #8114: fix memcost example URL: https://github.com/apache/incubator-mxnet/pull/8114#issuecomment-351211258 @yajiedesign Could you update? This is an automated message from the Apache Git Service.

[GitHub] piiswrong commented on issue #8275: [WIP] Implement 'same' convolutions

2017-12-12 Thread GitBox
piiswrong commented on issue #8275: [WIP] Implement 'same' convolutions URL: https://github.com/apache/incubator-mxnet/pull/8275#issuecomment-351211510 closing due to outdated. Please rebase after https://github.com/apache/incubator-mxnet/pull/8302 is merged if you are still interested.

[GitHub] piiswrong commented on issue #8313: Ci test randomness

2017-12-12 Thread GitBox
piiswrong commented on issue #8313: Ci test randomness URL: https://github.com/apache/incubator-mxnet/pull/8313#issuecomment-351211734 Sorry for having overlooked this. @DickJC123 Is it still relevant? This is an automated

[GitHub] madjam commented on a change in pull request #9030: Fix Gan

2017-12-12 Thread GitBox
madjam commented on a change in pull request #9030: Fix Gan URL: https://github.com/apache/incubator-mxnet/pull/9030#discussion_r156505249 ## File path: docs/tutorials/index.md ## @@ -5,7 +5,7 @@ Gluon is the high-level interface for MXNet. It is more intuitive and easier

[GitHub] madjam commented on a change in pull request #9030: Fix Gan

2017-12-12 Thread GitBox
madjam commented on a change in pull request #9030: Fix Gan URL: https://github.com/apache/incubator-mxnet/pull/9030#discussion_r156508105 ## File path: docs/tutorials/unsupervised_learning/gan.md ## @@ -47,31 +47,31 @@ To complete this tutorial, you need: - Python 2.7,

[GitHub] madjam commented on a change in pull request #9030: Fix Gan

2017-12-12 Thread GitBox
madjam commented on a change in pull request #9030: Fix Gan URL: https://github.com/apache/incubator-mxnet/pull/9030#discussion_r156509616 ## File path: docs/tutorials/unsupervised_learning/gan.md ## @@ -160,16 +160,16 @@ g5 = mx.sym.Deconvolution(gact4, name='g5',

[GitHub] madjam commented on a change in pull request #9030: Fix Gan

2017-12-12 Thread GitBox
madjam commented on a change in pull request #9030: Fix Gan URL: https://github.com/apache/incubator-mxnet/pull/9030#discussion_r156509820 ## File path: docs/tutorials/unsupervised_learning/gan.md ## @@ -160,16 +160,16 @@ g5 = mx.sym.Deconvolution(gact4, name='g5',

[GitHub] madjam commented on a change in pull request #9030: Fix Gan

2017-12-12 Thread GitBox
madjam commented on a change in pull request #9030: Fix Gan URL: https://github.com/apache/incubator-mxnet/pull/9030#discussion_r156509374 ## File path: docs/tutorials/unsupervised_learning/gan.md ## @@ -117,22 +117,22 @@ class RandIter(mx.io.DataIter): return

[GitHub] madjam commented on a change in pull request #9030: Fix Gan

2017-12-12 Thread GitBox
madjam commented on a change in pull request #9030: Fix Gan URL: https://github.com/apache/incubator-mxnet/pull/9030#discussion_r156508217 ## File path: docs/tutorials/unsupervised_learning/gan.md ## @@ -47,31 +47,31 @@ To complete this tutorial, you need: - Python 2.7,

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

2017-12-12 Thread GitBox
szha commented on issue #7938: instance norm and reflection padding URL: https://github.com/apache/incubator-mxnet/pull/7938#issuecomment-351211862 Actually @zhanghang1989 will join us soon ? This is an automated message

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2017-12-12 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r156510825 ## File path: src/operator/nn/activation.cc ## @@ -21,65 +21,128 @@ * Copyright (c) 2015 by Contributors *

[GitHub] piiswrong commented on issue #8198: Adding .png to the list of allowed extensions in the im2rec.py

2017-12-12 Thread GitBox
piiswrong commented on issue #8198: Adding .png to the list of allowed extensions in the im2rec.py URL: https://github.com/apache/incubator-mxnet/pull/8198#issuecomment-351212300 Closing due to moved to

[GitHub] piiswrong closed pull request #8198: Adding .png to the list of allowed extensions in the im2rec.py

2017-12-12 Thread GitBox
piiswrong closed pull request #8198: Adding .png to the list of allowed extensions in the im2rec.py URL: https://github.com/apache/incubator-mxnet/pull/8198 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

[GitHub] piiswrong opened a new pull request #9043: Adding .png to the list of allowed extensions in the im2rec.py

2017-12-12 Thread GitBox
piiswrong opened a new pull request #9043: Adding .png to the list of allowed extensions in the im2rec.py URL: https://github.com/apache/incubator-mxnet/pull/9043 ## Description ## (Brief description on what this PR is about) ## Checklist ## ### Essentials ### - [ ] Passed

[GitHub] piiswrong closed pull request #7929: Merge `mxbox` into `gluon.data`

2017-12-12 Thread GitBox
piiswrong closed pull request #7929: Merge `mxbox` into `gluon.data` URL: https://github.com/apache/incubator-mxnet/pull/7929 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] piiswrong commented on issue #7929: Merge `mxbox` into `gluon.data`

2017-12-12 Thread GitBox
piiswrong commented on issue #7929: Merge `mxbox` into `gluon.data` URL: https://github.com/apache/incubator-mxnet/pull/7929#issuecomment-351212440 closing due to inactive. Please see the vision branch. This is an automated

[GitHub] piiswrong commented on issue #8406: Fixed rtc error on Tesla / Jetson TX2

2017-12-12 Thread GitBox
piiswrong commented on issue #8406: Fixed rtc error on Tesla / Jetson TX2 URL: https://github.com/apache/incubator-mxnet/pull/8406#issuecomment-351212584 @KellenSunderland Ping. Any updates? This is an automated message from

[GitHub] piiswrong commented on issue #7363: Add tensorboard configure into ./common/fit.py and ./train_mnist.py

2017-12-12 Thread GitBox
piiswrong commented on issue #7363: Add tensorboard configure into ./common/fit.py and ./train_mnist.py URL: https://github.com/apache/incubator-mxnet/pull/7363#issuecomment-351212915 closing due to inactive. @zihaolucky Please reopen if this is still relevant. Please remove the log

[GitHub] piiswrong closed pull request #8485: fix parsing boolean values with argparse

2017-12-12 Thread GitBox
piiswrong closed pull request #8485: fix parsing boolean values with argparse URL: https://github.com/apache/incubator-mxnet/pull/8485 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] piiswrong commented on issue #8547: Revised validating process of a bounding box in ImageDetIter

2017-12-12 Thread GitBox
piiswrong commented on issue #8547: Revised validating process of a bounding box in ImageDetIter URL: https://github.com/apache/incubator-mxnet/pull/8547#issuecomment-351213155 @zhreshold Please have a look This is an

[GitHub] piiswrong closed pull request #8547: Revised validating process of a bounding box in ImageDetIter

2017-12-12 Thread GitBox
piiswrong closed pull request #8547: Revised validating process of a bounding box in ImageDetIter URL: https://github.com/apache/incubator-mxnet/pull/8547 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

[GitHub] mun94 opened a new pull request #8547: Revised validating process of a bounding box in ImageDetIter

2017-12-12 Thread GitBox
mun94 opened a new pull request #8547: Revised validating process of a bounding box in ImageDetIter URL: https://github.com/apache/incubator-mxnet/pull/8547 ## Description ## The previous ImageDetIter validated a bounding box with two points only (left-top and right-bottom points).

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2017-12-12 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r156511872 ## File path: src/operator/nn/activation.cu ## @@ -31,39 +31,73 @@ namespace mxnet { namespace op { + +#if

[GitHub] piiswrong closed pull request #8313: Ci test randomness

2017-12-12 Thread GitBox
piiswrong closed pull request #8313: Ci test randomness URL: https://github.com/apache/incubator-mxnet/pull/8313 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[GitHub] piiswrong commented on issue #8526: Ci test randomness2

2017-12-12 Thread GitBox
piiswrong commented on issue #8526: Ci test randomness2 URL: https://github.com/apache/incubator-mxnet/pull/8526#issuecomment-351213340 @DickJC123 Ping. Is this a duplicate? I'm going to close the older one This is an

[GitHub] piiswrong commented on issue #7931: MKL-DNN integration: request for reviews

2017-12-12 Thread GitBox
piiswrong commented on issue #7931: MKL-DNN integration: request for reviews URL: https://github.com/apache/incubator-mxnet/pull/7931#issuecomment-351213633 closing since @zheng-da is making a new PR for this This is an

[GitHub] piiswrong commented on issue #8423: Re-implement segnet in MXnet

2017-12-12 Thread GitBox
piiswrong commented on issue #8423: Re-implement segnet in MXnet URL: https://github.com/apache/incubator-mxnet/pull/8423#issuecomment-351213948 @zhreshold @winstywang @solin319 Any updates on this? This is an automated

[GitHub] piiswrong closed pull request #8373: distribute training in fp16

2017-12-12 Thread GitBox
piiswrong closed pull request #8373: distribute training in fp16 URL: https://github.com/apache/incubator-mxnet/pull/8373 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

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

2017-12-12 Thread GitBox
piiswrong commented on issue #8373: distribute training in fp16 URL: https://github.com/apache/incubator-mxnet/pull/8373#issuecomment-351214158 Closing due to outdated. @rahul003 please guide @solin319 to merge this into the gradient compression framework.

[GitHub] piiswrong closed pull request #8877: add v1.0 release and fixed footer overlay on search results

2017-12-12 Thread GitBox
piiswrong closed pull request #8877: add v1.0 release and fixed footer overlay on search results URL: https://github.com/apache/incubator-mxnet/pull/8877 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

[GitHub] piiswrong commented on issue #8877: add v1.0 release and fixed footer overlay on search results

2017-12-12 Thread GitBox
piiswrong commented on issue #8877: add v1.0 release and fixed footer overlay on search results URL: https://github.com/apache/incubator-mxnet/pull/8877#issuecomment-351214273 closing due to outdated This is an automated

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2017-12-12 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r156510303 ## File path: src/operator/nn/activation-inl.h ## @@ -61,158 +62,127 @@ struct ActivationParam : public

[GitHub] piiswrong commented on issue #8484: MXNet-to-CoreML module: Fixed KeyError for Reshape

2017-12-12 Thread GitBox
piiswrong commented on issue #8484: MXNet-to-CoreML module: Fixed KeyError for Reshape URL: https://github.com/apache/incubator-mxnet/pull/8484#issuecomment-351214484 @Prasad9 Is this still relevant? Could you rebase and test on master?

[GitHub] piiswrong commented on issue #8787: add CapsNet example

2017-12-12 Thread GitBox
piiswrong commented on issue #8787: add CapsNet example URL: https://github.com/apache/incubator-mxnet/pull/8787#issuecomment-351215138 @Soonhwan-Kwon Yes. Could you rebase to master? needs CI to pass This is an automated

[GitHub] piiswrong commented on issue #8846: Batching improvements for GEMM/TRSM operators and full MKL usage docs.

2017-12-12 Thread GitBox
piiswrong commented on issue #8846: Batching improvements for GEMM/TRSM operators and full MKL usage docs. URL: https://github.com/apache/incubator-mxnet/pull/8846#issuecomment-351215260 @asmushetzel Any updates? This is an

[GitHub] piiswrong commented on issue #8829: Small fix default parameter `--thresh` on SSD example demo.py

2017-12-12 Thread GitBox
piiswrong commented on issue #8829: Small fix default parameter `--thresh` on SSD example demo.py URL: https://github.com/apache/incubator-mxnet/pull/8829#issuecomment-351215539 @zhreshold Updates? This is an automated

[GitHub] piiswrong commented on issue #8797: fix errors for reinterpret_cast usage

2017-12-12 Thread GitBox
piiswrong commented on issue #8797: fix errors for reinterpret_cast usage URL: https://github.com/apache/incubator-mxnet/pull/8797#issuecomment-351215386 Could you rebase? This is an automated message from the Apache Git

[GitHub] zhreshold commented on issue #8547: Revised validating process of a bounding box in ImageDetIter

2017-12-12 Thread GitBox
zhreshold commented on issue #8547: Revised validating process of a bounding box in ImageDetIter URL: https://github.com/apache/incubator-mxnet/pull/8547#issuecomment-351215556 @mun94 If you are referring to non-standard rectangles, it requires much more than this fix

[GitHub] DickJC123 commented on issue #8526: Ci test randomness2

2017-12-12 Thread GitBox
DickJC123 commented on issue #8526: Ci test randomness2 URL: https://github.com/apache/incubator-mxnet/pull/8526#issuecomment-351215660 Yes, the 'randomness2' is an update to my thinking, so the old one can be closed. I can rebase this PR if you are interested in accepting it. The

[GitHub] piiswrong commented on issue #8664: fix VGG error in benchmark_score

2017-12-12 Thread GitBox
piiswrong commented on issue #8664: fix VGG error in benchmark_score URL: https://github.com/apache/incubator-mxnet/pull/8664#issuecomment-351215644 Could you rebase? This is an automated message from the Apache Git Service.

[GitHub] piiswrong commented on issue #8894: Mobilenet

2017-12-12 Thread GitBox
piiswrong commented on issue #8894: Mobilenet URL: https://github.com/apache/incubator-mxnet/pull/8894#issuecomment-351215867 @zhreshold Could you follow up? This is an automated message from the Apache Git Service. To

[GitHub] zhreshold commented on issue #8829: Small fix default parameter `--thresh` on SSD example demo.py

2017-12-12 Thread GitBox
zhreshold commented on issue #8829: Small fix default parameter `--thresh` on SSD example demo.py URL: https://github.com/apache/incubator-mxnet/pull/8829#issuecomment-351216284 I will update with argparse options, hard coded default values are no longer a good practice. Closing for this

[GitHub] zhreshold closed pull request #8829: Small fix default parameter `--thresh` on SSD example demo.py

2017-12-12 Thread GitBox
zhreshold closed pull request #8829: Small fix default parameter `--thresh` on SSD example demo.py URL: https://github.com/apache/incubator-mxnet/pull/8829 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

[GitHub] marcoabreu commented on issue #8526: Ci test randomness2

2017-12-12 Thread GitBox
marcoabreu commented on issue #8526: Ci test randomness2 URL: https://github.com/apache/incubator-mxnet/pull/8526#issuecomment-351216613 So this PR is complete? I'll see if I got time to look into it more closely. It would be great if you could rebase it in order to test everything on

[GitHub] piiswrong commented on issue #8912: add Gluon PReLU activation layer

2017-12-12 Thread GitBox
piiswrong commented on issue #8912: add Gluon PReLU activation layer URL: https://github.com/apache/incubator-mxnet/pull/8912#issuecomment-351216735 Could you add unittest and rebase to master? see tests/python/unittests/test_gluon_xxx.py for example

[GitHub] piiswrong closed pull request #8945: Add param 'num_filter' for 'BaseConvRNNCell'.

2017-12-12 Thread GitBox
piiswrong closed pull request #8945: Add param 'num_filter' for 'BaseConvRNNCell'. URL: https://github.com/apache/incubator-mxnet/pull/8945 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

[GitHub] piiswrong commented on issue #8958: [WIP] fix sgld optimizer and its related example

2017-12-12 Thread GitBox
piiswrong commented on issue #8958: [WIP] fix sgld optimizer and its related example URL: https://github.com/apache/incubator-mxnet/pull/8958#issuecomment-351217130 I thought we needed to fix the c++ random generator? This

[GitHub] piiswrong commented on issue #8945: Add param 'num_filter' for 'BaseConvRNNCell'.

2017-12-12 Thread GitBox
piiswrong commented on issue #8945: Add param 'num_filter' for 'BaseConvRNNCell'. URL: https://github.com/apache/incubator-mxnet/pull/8945#issuecomment-351216953 I think num_hidden is fine. Also this breaks backward compatibility

[GitHub] piiswrong commented on issue #8967: add shared storage in windows

2017-12-12 Thread GitBox
piiswrong commented on issue #8967: add shared storage in windows URL: https://github.com/apache/incubator-mxnet/pull/8967#issuecomment-351217191 Could you rebase? This is an automated message from the Apache Git Service. To

[GitHub] piiswrong commented on issue #8826: improve memory usage in sparse.dot

2017-12-12 Thread GitBox
piiswrong commented on issue #8826: improve memory usage in sparse.dot URL: https://github.com/apache/incubator-mxnet/pull/8826#issuecomment-351217383 updates? This is an automated message from the Apache Git Service. To

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

2017-12-12 Thread GitBox
piiswrong commented on issue #8964: Fix example/module folder and remove duplicate examples URL: https://github.com/apache/incubator-mxnet/pull/8964#issuecomment-351217474 could you rebase? This is an automated message from

[GitHub] piiswrong commented on issue #8942: Cloud install instructions - adding DLAMI docs link

2017-12-12 Thread GitBox
piiswrong commented on issue #8942: Cloud install instructions - adding DLAMI docs link URL: https://github.com/apache/incubator-mxnet/pull/8942#issuecomment-351217556 please rebase This is an automated message from the

[GitHub] zhreshold commented on issue #8894: Mobilenet

2017-12-12 Thread GitBox
zhreshold commented on issue #8894: Mobilenet URL: https://github.com/apache/incubator-mxnet/pull/8894#issuecomment-351217590 @dwSun One more qustion, is this change fully compatible with previously trained model? This is an

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

2017-12-12 Thread GitBox
piiswrong 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-351217873 @zhreshold This is an automated

[GitHub] piiswrong commented on issue #8107: "add warmup lr_scheduler" create a new pr

2017-12-12 Thread GitBox
piiswrong commented on issue #8107: "add warmup lr_scheduler" create a new pr URL: https://github.com/apache/incubator-mxnet/pull/8107#issuecomment-351218734 The document still looks cryptic. I don't quite understand why we need 3 more arguments. @astonzhang @szha Could you help

[GitHub] piiswrong commented on issue #8724: im2rec.py output labels.txt file for use in inference.

2017-12-12 Thread GitBox
piiswrong commented on issue #8724: im2rec.py output labels.txt file for use in inference. URL: https://github.com/apache/incubator-mxnet/pull/8724#issuecomment-351219096 Is the output consistent with imagenet's synset file?

[GitHub] piiswrong commented on issue #8414: Fix the strides in Gluon version of ResNet v1

2017-12-12 Thread GitBox
piiswrong commented on issue #8414: Fix the strides in Gluon version of ResNet v1 URL: https://github.com/apache/incubator-mxnet/pull/8414#issuecomment-351219313 @zhreshold @szha Please update the modelzoo after training with new io pipeline

[GitHub] piiswrong commented on issue #8639: [WIP] Gluon object detection

2017-12-12 Thread GitBox
piiswrong commented on issue #8639: [WIP] Gluon object detection URL: https://github.com/apache/incubator-mxnet/pull/8639#issuecomment-351219369 any updates? This is an automated message from the Apache Git Service. To

[GitHub] piiswrong closed pull request #8689: Mark tests that should only be run nightly.

2017-12-12 Thread GitBox
piiswrong closed pull request #8689: Mark tests that should only be run nightly. URL: https://github.com/apache/incubator-mxnet/pull/8689 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] mun94 commented on issue #8547: Revised validating process of a bounding box in ImageDetIter

2017-12-12 Thread GitBox
mun94 commented on issue #8547: Revised validating process of a bounding box in ImageDetIter URL: https://github.com/apache/incubator-mxnet/pull/8547#issuecomment-351219580 It's validating polygons (not necessarily a rectangle) by looking at the four points. How would you suggest fixing

[GitHub] piiswrong commented on issue #8689: Mark tests that should only be run nightly.

2017-12-12 Thread GitBox
piiswrong commented on issue #8689: Mark tests that should only be run nightly. URL: https://github.com/apache/incubator-mxnet/pull/8689#issuecomment-351219601 closing due to outdated This is an automated message from the

[GitHub] piiswrong commented on issue #8684: [BugFix][CoreML Converter] Dense layers w/o bias.

2017-12-12 Thread GitBox
piiswrong commented on issue #8684: [BugFix][CoreML Converter] Dense layers w/o bias. URL: https://github.com/apache/incubator-mxnet/pull/8684#issuecomment-351219729 Could you update? This is an automated message from the

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

2017-12-12 Thread GitBox
piiswrong 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-351219818 @javelinjs any updates? This is an

[GitHub] piiswrong commented on issue #8519: More details to the windows build process

2017-12-12 Thread GitBox
piiswrong commented on issue #8519: More details to the windows build process URL: https://github.com/apache/incubator-mxnet/pull/8519#issuecomment-351220066 Could you rebase again? Sorry about the bad CI. It should have been fixed now

[GitHub] piiswrong commented on issue #8519: More details to the windows build process

2017-12-12 Thread GitBox
piiswrong commented on issue #8519: More details to the windows build process URL: https://github.com/apache/incubator-mxnet/pull/8519#issuecomment-351220066 Could you rebase again? Sorry about the bad CI. It should have been fixed

[GitHub] piiswrong commented on issue #8652: Add note about deprecation of v1 versions of ImageRecordIter

2017-12-12 Thread GitBox
piiswrong commented on issue #8652: Add note about deprecation of v1 versions of ImageRecordIter URL: https://github.com/apache/incubator-mxnet/pull/8652#issuecomment-351220238 Could you rebase? This is an automated message

[GitHub] piiswrong commented on issue #8377: re-enable tests

2017-12-12 Thread GitBox
piiswrong commented on issue #8377: re-enable tests URL: https://github.com/apache/incubator-mxnet/pull/8377#issuecomment-351220172 @szha Any updates? This is an automated message from the Apache Git Service. To respond to

[GitHub] piiswrong commented on issue #9001: Disable test_optimizers:test_sgd (Temp fix #9000)

2017-12-12 Thread GitBox
piiswrong commented on issue #9001: Disable test_optimizers:test_sgd (Temp fix #9000) URL: https://github.com/apache/incubator-mxnet/pull/9001#issuecomment-351220404 updates? This is an automated message from the Apache Git

[incubator-mxnet] branch master updated: Remove redundant or unused imports (#9021)

2017-12-12 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 db15049 Remove redundant or unused

[GitHub] piiswrong closed pull request #9021: Remove redundant or unused imports

2017-12-12 Thread GitBox
piiswrong closed pull request #9021: Remove redundant or unused imports URL: https://github.com/apache/incubator-mxnet/pull/9021 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] marcoabreu commented on issue #9001: Disable test_optimizers:test_sgd (Temp fix #9000)

2017-12-12 Thread GitBox
marcoabreu commented on issue #9001: Disable test_optimizers:test_sgd (Temp fix #9000) URL: https://github.com/apache/incubator-mxnet/pull/9001#issuecomment-351220891 Rebuilding This is an automated message from the Apache

[GitHub] piiswrong closed pull request #9025: fix small memory leak of sparse embedding

2017-12-12 Thread GitBox
piiswrong closed pull request #9025: fix small memory leak of sparse embedding URL: https://github.com/apache/incubator-mxnet/pull/9025 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: fix small memory leak of sparse embedding (#9025)

2017-12-12 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 167871a fix small memory leak of sparse

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

2017-12-12 Thread GitBox
piiswrong 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_r156519228 ## File path: include/mxnet/c_api.h ## @@ -227,10 +227,138 @@

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2017-12-12 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r156519245 ## File path: src/ndarray/ndarray.cc ## @@ -64,17 +158,147 @@ nnvm::Symbol NDArray::get_autograd_symbol() const

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

2017-12-12 Thread GitBox
piiswrong 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_r156519270 ## File path: python/mxnet/profiler.py ## @@ -56,3 +53,235 @@

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2017-12-12 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r156519807 ## File path: src/ndarray/ndarray.cc ## @@ -64,17 +158,147 @@ nnvm::Symbol NDArray::get_autograd_symbol() const

[GitHub] zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN

2017-12-12 Thread GitBox
zheng-da commented on a change in pull request #8302: Refactor operators & MKLDNN URL: https://github.com/apache/incubator-mxnet/pull/8302#discussion_r156520457 ## File path: src/ndarray/ndarray.cc ## @@ -64,17 +158,147 @@ nnvm::Symbol NDArray::get_autograd_symbol() const

[GitHub] indhub commented on issue #9033: Reorganize the tutorials index page

2017-12-12 Thread GitBox
indhub commented on issue #9033: Reorganize the tutorials index page URL: https://github.com/apache/incubator-mxnet/pull/9033#issuecomment-351224605 @madjam This is an automated message from the Apache Git Service. To

[GitHub] zhreshold commented on a change in pull request #8547: Revised validating process of a bounding box in ImageDetIter

2017-12-12 Thread GitBox
zhreshold commented on a change in pull request #8547: Revised validating process of a bounding box in ImageDetIter URL: https://github.com/apache/incubator-mxnet/pull/8547#discussion_r156522021 ## File path: python/mxnet/image/detection.py ## @@ -726,10 +726,23 @@ def

[GitHub] zhreshold commented on a change in pull request #8547: Revised validating process of a bounding box in ImageDetIter

2017-12-12 Thread GitBox
zhreshold commented on a change in pull request #8547: Revised validating process of a bounding box in ImageDetIter URL: https://github.com/apache/incubator-mxnet/pull/8547#discussion_r156521772 ## File path: python/mxnet/image/detection.py ## @@ -726,10 +726,23 @@ def

[GitHub] zhreshold commented on a change in pull request #8547: Revised validating process of a bounding box in ImageDetIter

2017-12-12 Thread GitBox
zhreshold commented on a change in pull request #8547: Revised validating process of a bounding box in ImageDetIter URL: https://github.com/apache/incubator-mxnet/pull/8547#discussion_r156521729 ## File path: python/mxnet/image/detection.py ## @@ -726,10 +726,23 @@ def

[GitHub] chsin opened a new issue #9044: [scala] run_gan_mnist.sh fails

2017-12-12 Thread GitBox
chsin opened a new issue #9044: [scala] run_gan_mnist.sh fails URL: https://github.com/apache/incubator-mxnet/issues/9044 ## Description Was able to built libmxnet.so, and successfully build the Scala package, but run_gan_mnist.sh fails. ## Environment info (Required) I had to

[GitHub] larroy commented on issue #9016: Add googletest as a 3rdparty library

2017-12-12 Thread GitBox
larroy commented on issue #9016: Add googletest as a 3rdparty library URL: https://github.com/apache/incubator-mxnet/pull/9016#issuecomment-351226080 We discussed in the list to add it as a third party The library is very small, why is it better to have it as a submodule?

[GitHub] szha commented on a change in pull request #9016: Add googletest as a 3rdparty library

2017-12-12 Thread GitBox
szha commented on a change in pull request #9016: Add googletest as a 3rdparty library URL: https://github.com/apache/incubator-mxnet/pull/9016#discussion_r156526353 ## File path: .gitmodules ## @@ -1,9 +1,9 @@ [submodule "mshadow"] path = mshadow - url =

[incubator-mxnet] branch master updated: Adding .png to the list of allowed extensions in the im2rec.py (#9043)

2017-12-12 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 087a906 Adding .png to the list of

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

2017-12-12 Thread GitBox
Laurawly commented on a change in pull request #8915: NVLink communication pattern updated URL: https://github.com/apache/incubator-mxnet/pull/8915#discussion_r156526811 ## File path: src/kvstore/comm.h ## @@ -670,31 +715,75 @@ class CommDevice : public Comm { using

[GitHub] szha closed pull request #9043: Adding .png to the list of allowed extensions in the im2rec.py

2017-12-12 Thread GitBox
szha closed pull request #9043: Adding .png to the list of allowed extensions in the im2rec.py URL: https://github.com/apache/incubator-mxnet/pull/9043 This is an automated message from the Apache Git Service. To respond to

[incubator-mxnet] branch piiswrong-patch-2 deleted (was 4c1f05b)

2017-12-12 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a change to branch piiswrong-patch-2 in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. was 4c1f05b Adding .png to the list of allowed extensions in the im2rec.py The revisions that

[GitHub] eric-haibin-lin commented on issue #9042: Avoiding weight sharing among certain layers in BucketingModule?

2017-12-12 Thread GitBox
eric-haibin-lin commented on issue #9042: Avoiding weight sharing among certain layers in BucketingModule? URL: https://github.com/apache/incubator-mxnet/issues/9042#issuecomment-351233681 You can make `sym_gen(default_bucket_key)` returns a "master network" that contains both these two

[GitHub] eric-haibin-lin commented on issue #9042: Avoiding weight sharing among certain layers in BucketingModule?

2017-12-12 Thread GitBox
eric-haibin-lin commented on issue #9042: Avoiding weight sharing among certain layers in BucketingModule? URL: https://github.com/apache/incubator-mxnet/issues/9042#issuecomment-351233681 You can make `sym_gen(default_bucket_key)` returns a "master network" that contains all these FC

[GitHub] pracheer commented on issue #9045: Remove defunct NLP tutorial page

2017-12-12 Thread GitBox
pracheer commented on issue #9045: Remove defunct NLP tutorial page URL: https://github.com/apache/incubator-mxnet/pull/9045#issuecomment-351233712 Would it make more sense to write a basic tutorial on rnns using MXNet as part of this PR itself?

[GitHub] eric-haibin-lin commented on issue #9042: Avoiding weight sharing among certain layers in BucketingModule?

2017-12-12 Thread GitBox
eric-haibin-lin commented on issue #9042: Avoiding weight sharing among certain layers in BucketingModule? URL: https://github.com/apache/incubator-mxnet/issues/9042#issuecomment-351233681 You can make `sym_gen(default_bucket_key)` returns a "master network" that contains all these FC

[GitHub] larroy commented on a change in pull request #9035: [WIP] Build improvements, for users and for Jenkins

2017-12-12 Thread GitBox
larroy commented on a change in pull request #9035: [WIP] Build improvements, for users and for Jenkins URL: https://github.com/apache/incubator-mxnet/pull/9035#discussion_r156529917 ## File path: build.py ## @@ -0,0 +1,167 @@ +#!/usr/bin/env python +# -*- coding: utf-8

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

2017-12-12 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_r156529905 ## File path: include/mxnet/c_api.h ## @@ -227,10 +227,138 @@

[GitHub] larroy commented on issue #9033: Reorganize the tutorials index page

2017-12-12 Thread GitBox
larroy commented on issue #9033: Reorganize the tutorials index page URL: https://github.com/apache/incubator-mxnet/pull/9033#issuecomment-351234313 The build failed beause of a Jenkinsfile difference, but if a commiter can merge this without CI then please go ahead.

<    1   2   3   >