[GitHub] [incubator-mxnet] chinakook commented on issue #14421: Updating mxnet from 1.0.0, networks give different outputs

2019-03-18 Thread GitBox
chinakook commented on issue #14421: Updating mxnet from 1.0.0, networks give different outputs URL: https://github.com/apache/incubator-mxnet/issues/14421#issuecomment-474207449 How do you use your model to inference? If you are using mx.module, please make sure the parameter

[GitHub] [incubator-mxnet] chinakook edited a comment on issue #14459: Cannot release memory sometimes

2019-03-18 Thread GitBox
chinakook edited a comment on issue #14459: Cannot release memory sometimes URL: https://github.com/apache/incubator-mxnet/issues/14459#issuecomment-474205117 Please repeat Ctrl-C until you get back to the console. However, if your GPU is set to persistent mode, Ctrl-C cannot clear the

[GitHub] [incubator-mxnet] chinakook commented on issue #14459: Cannot release memory sometimes

2019-03-18 Thread GitBox
chinakook commented on issue #14459: Cannot release memory sometimes URL: https://github.com/apache/incubator-mxnet/issues/14459#issuecomment-474205117 Please repeat Ctrl-C until you get back to the console. This is an

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #14202: [Doc] Start the tutorials for MKL-DNN backend

2019-03-18 Thread GitBox
pengzhao-intel commented on issue #14202: [Doc] Start the tutorials for MKL-DNN backend URL: https://github.com/apache/incubator-mxnet/pull/14202#issuecomment-474204607 Thanks, @aaronmarkham I am merging now :) This is an

[GitHub] [incubator-mxnet] pengzhao-intel merged pull request #14202: [Doc] Start the tutorials for MKL-DNN backend

2019-03-18 Thread GitBox
pengzhao-intel merged pull request #14202: [Doc] Start the tutorials for MKL-DNN backend URL: https://github.com/apache/incubator-mxnet/pull/14202 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #14399: [Doc] Improve the document for MKL-DNN backend

2019-03-18 Thread GitBox
pengzhao-intel commented on issue #14399: [Doc] Improve the document for MKL-DNN backend URL: https://github.com/apache/incubator-mxnet/issues/14399#issuecomment-474204315 @yinghu5 This is an automated message from the

[GitHub] [incubator-mxnet] aaronmarkham commented on issue #14202: [Doc] Start the tutorials for MKL-DNN backend

2019-03-18 Thread GitBox
aaronmarkham commented on issue #14202: [Doc] Start the tutorials for MKL-DNN backend URL: https://github.com/apache/incubator-mxnet/pull/14202#issuecomment-474204129 I think the separate issue to track the docs needs is great! Go ahead with this one.

[GitHub] [incubator-mxnet] arcadiaphy edited a comment on issue #14451: fix custom operation in fork

2019-03-18 Thread GitBox
arcadiaphy edited a comment on issue #14451: fix custom operation in fork URL: https://github.com/apache/incubator-mxnet/pull/14451#issuecomment-47410 @YutingZhang The problem you mentioned may be related to GIL, correctly handling GIL in c++ extension is a tricky problem, easy to

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #14202: [Doc] Start the tutorials for MKL-DNN backend

2019-03-18 Thread GitBox
pengzhao-intel commented on issue #14202: [Doc] Start the tutorials for MKL-DNN backend URL: https://github.com/apache/incubator-mxnet/pull/14202#issuecomment-474202565 @aaronmarkham do you have any other suggestion for our plan? We're working on improving the MKLDNN documents.

[GitHub] [incubator-mxnet] ciyongch closed pull request #14407: Update imagenet quantization script for MKL-DNN

2019-03-18 Thread GitBox
ciyongch closed pull request #14407: Update imagenet quantization script for MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/14407 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-mxnet] ciyongch commented on issue #14407: Update imagenet quantization script for MKL-DNN

2019-03-18 Thread GitBox
ciyongch commented on issue #14407: Update imagenet quantization script for MKL-DNN URL: https://github.com/apache/incubator-mxnet/pull/14407#issuecomment-474201445 Since PR #14466 enables signed int8 inputs to quantized FullyConnected, no need to apply this patch. Close it now.

[GitHub] [incubator-mxnet] ciyongch commented on issue #14466: [MKL-DNN] Enable s8 support for inner product and 3d input with flatten=false

2019-03-18 Thread GitBox
ciyongch commented on issue #14466: [MKL-DNN] Enable s8 support for inner product and 3d input with flatten=false URL: https://github.com/apache/incubator-mxnet/pull/14466#issuecomment-474200907 LGTM :) This is an automated

[GitHub] [incubator-mxnet] ZhennanQin commented on a change in pull request #14466: [MKL-DNN] Enable s8 support for inner product and 3d input with flatten=false

2019-03-18 Thread GitBox
ZhennanQin commented on a change in pull request #14466: [MKL-DNN] Enable s8 support for inner product and 3d input with flatten=false URL: https://github.com/apache/incubator-mxnet/pull/14466#discussion_r266728473 ## File path:

[GitHub] [incubator-mxnet] ZhennanQin commented on a change in pull request #14466: [MKL-DNN] Enable s8 support for inner product and 3d input with flatten=false

2019-03-18 Thread GitBox
ZhennanQin commented on a change in pull request #14466: [MKL-DNN] Enable s8 support for inner product and 3d input with flatten=false URL: https://github.com/apache/incubator-mxnet/pull/14466#discussion_r266728459 ## File path:

[GitHub] [incubator-mxnet] arcadiaphy commented on issue #14451: fix custom operation in fork

2019-03-18 Thread GitBox
arcadiaphy commented on issue #14451: fix custom operation in fork URL: https://github.com/apache/incubator-mxnet/pull/14451#issuecomment-47410 @YutingZhang I think the problem you mentioned is related to GIL, correctly handling GIL in c++ extension is a tricky problem, easy to cause

[GitHub] [incubator-mxnet] arcadiaphy commented on issue #14451: fix custom operation in fork

2019-03-18 Thread GitBox
arcadiaphy commented on issue #14451: fix custom operation in fork URL: https://github.com/apache/incubator-mxnet/pull/14451#issuecomment-474181960 @anirudh2290 The program blocks because of GIL, additional handling should be added to correctly run callbacks. ``` #include

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #14466: [MKL-DNN] Enable s8 support for inner product and 3d input with flatten=false

2019-03-18 Thread GitBox
pengzhao-intel commented on issue #14466: [MKL-DNN] Enable s8 support for inner product and 3d input with flatten=false URL: https://github.com/apache/incubator-mxnet/pull/14466#issuecomment-474179721 @reminisce @anirudh2290 @ZhennanQin @ciyongch please help take a review :)

[GitHub] [incubator-mxnet] ymjiang edited a comment on issue #13757: Set ImageNet data augmentation by default

2019-03-18 Thread GitBox
ymjiang edited a comment on issue #13757: Set ImageNet data augmentation by default URL: https://github.com/apache/incubator-mxnet/pull/13757#issuecomment-474179103 @karan6181 @anirudhacharya Sorry for my delay. I committed two new changes to enable data-augmentation with command-line

[GitHub] [incubator-mxnet] ymjiang commented on issue #13757: Set ImageNet data augmentation by default

2019-03-18 Thread GitBox
ymjiang commented on issue #13757: Set ImageNet data augmentation by default URL: https://github.com/apache/incubator-mxnet/pull/13757#issuecomment-474179103 @karan6181 @anirudhacharya Sorry for my delay. I committed two new changes to enable data-augmentation with command-line argument.

[GitHub] [incubator-mxnet] xinyu-intel opened a new pull request #14466: [MKL-DNN] Enable s8 support for inner product and 3d input with flatten=false

2019-03-18 Thread GitBox
xinyu-intel opened a new pull request #14466: [MKL-DNN] Enable s8 support for inner product and 3d input with flatten=false URL: https://github.com/apache/incubator-mxnet/pull/14466 ## Description ## **Major changes:** - Integrate mkl-dnn signed int8 inner product; - support

[GitHub] [incubator-mxnet] xinyu-intel closed pull request #13643: Remove mklml pre-install in ci

2019-03-18 Thread GitBox
xinyu-intel closed pull request #13643: Remove mklml pre-install in ci URL: https://github.com/apache/incubator-mxnet/pull/13643 This is an automated message from the Apache Git Service. To respond to the message, please log

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2019-03-18 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new bec1eb6 Bump the publish

[GitHub] [incubator-mxnet] ZhennanQin commented on a change in pull request #14277: Enhance PartitionGraph

2019-03-18 Thread GitBox
ZhennanQin commented on a change in pull request #14277: Enhance PartitionGraph URL: https://github.com/apache/incubator-mxnet/pull/14277#discussion_r266699111 ## File path: src/operator/subgraph/subgraph_property.h ## @@ -200,7 +197,7 @@ typedef

[GitHub] [incubator-mxnet] marcoabreu closed issue #14458: Create a Dataset Using RecordIO

2019-03-18 Thread GitBox
marcoabreu closed issue #14458: Create a Dataset Using RecordIO URL: https://github.com/apache/incubator-mxnet/issues/14458 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-mxnet] karan6181 commented on issue #12641: Add docker file which includes sample jupyter notebook (#12611)

2019-03-18 Thread GitBox
karan6181 commented on issue #12641: Add docker file which includes sample jupyter notebook (#12611) URL: https://github.com/apache/incubator-mxnet/pull/12641#issuecomment-474155735 @gautamkmr Thank you for your contribution! Can you please provide the update on this PR since there is no

[GitHub] [incubator-mxnet] karan6181 commented on issue #12502: [NGRAPH] MXNet - nGraph initial integration

2019-03-18 Thread GitBox
karan6181 commented on issue #12502: [NGRAPH] MXNet - nGraph initial integration URL: https://github.com/apache/incubator-mxnet/pull/12502#issuecomment-474154995 @mbrookhart @ashokei Thank you for your contribution! can you please address the review comments? Thank you!

[GitHub] [incubator-mxnet] karan6181 commented on issue #12781: Fixed issue #12745

2019-03-18 Thread GitBox
karan6181 commented on issue #12781: Fixed issue #12745 URL: https://github.com/apache/incubator-mxnet/pull/12781#issuecomment-474154532 @LuckyPigeon Can you please address the review comments? @aaronmarkham Could you please review this PR again? Thank you!

[GitHub] [incubator-mxnet] karan6181 commented on issue #13132: enabled link checker and doc test during building the doc

2019-03-18 Thread GitBox
karan6181 commented on issue #13132: enabled link checker and doc test during building the doc URL: https://github.com/apache/incubator-mxnet/pull/13132#issuecomment-474154287 @mli Thank you for your contribution! Can you please look into this PR and close it or let us know if the changes

[GitHub] [incubator-mxnet] karan6181 commented on issue #13643: Remove mklml pre-install in ci

2019-03-18 Thread GitBox
karan6181 commented on issue #13643: Remove mklml pre-install in ci URL: https://github.com/apache/incubator-mxnet/pull/13643#issuecomment-474153891 @xinyu-intel Thank you for your contribution! Could you please address the review comments? Thanks

[GitHub] [incubator-mxnet] karan6181 commented on issue #13322: Change CUB submodule to track Nvidia CUB project.

2019-03-18 Thread GitBox
karan6181 commented on issue #13322: Change CUB submodule to track Nvidia CUB project. URL: https://github.com/apache/incubator-mxnet/pull/13322#issuecomment-474153543 @frankfliu Could you please look at the incomplete CI? It seems no updates from last couple of weeks. Can you please look

[GitHub] [incubator-mxnet] karan6181 commented on issue #13400: [MXNET-1229] use OpenBLAS, lapack & OpenCV from conan

2019-03-18 Thread GitBox
karan6181 commented on issue #13400: [MXNET-1229] use OpenBLAS, lapack & OpenCV from conan URL: https://github.com/apache/incubator-mxnet/pull/13400#issuecomment-474153201 @szha Could you please review this PR again? Thank you!

[GitHub] [incubator-mxnet] karan6181 commented on issue #13401: [MXNET-1227] Adding CornerPooling operator

2019-03-18 Thread GitBox
karan6181 commented on issue #13401: [MXNET-1227] Adding CornerPooling operator URL: https://github.com/apache/incubator-mxnet/pull/13401#issuecomment-474152750 @BigDeviltjj Can you please address the @anirudh2290 comments? Also, please resolve the branch conflicts. Thanks!

[GitHub] [incubator-mxnet] karan6181 commented on issue #13574: Replaced rand_r with std:: random generation

2019-03-18 Thread GitBox
karan6181 commented on issue #13574: Replaced rand_r with std:: random generation URL: https://github.com/apache/incubator-mxnet/pull/13574#issuecomment-474152407 @lebeg Did you get a chance to look at the RandGenerator implementation ? Can you please make the relevant code changes for

[GitHub] [incubator-mxnet] karan6181 commented on issue #13576: [MXNET-679] Improved CMakeLists.txt

2019-03-18 Thread GitBox
karan6181 commented on issue #13576: [MXNET-679] Improved CMakeLists.txt URL: https://github.com/apache/incubator-mxnet/pull/13576#issuecomment-474152050 @lebeg Could you please re-trigger the CI again since some tests are failing and can you please address the comments? Thank you!

[GitHub] [incubator-mxnet] karan6181 commented on issue #13715: Add C++ Predictor class for inference

2019-03-18 Thread GitBox
karan6181 commented on issue #13715: Add C++ Predictor class for inference URL: https://github.com/apache/incubator-mxnet/pull/13715#issuecomment-474151702 @stu1130 @ChaiBapchya @KellenSunderland Could you please review this PR again? Thank you!

[GitHub] [incubator-mxnet] karan6181 commented on issue #13757: Set ImageNet data augmentation by default

2019-03-18 Thread GitBox
karan6181 commented on issue #13757: Set ImageNet data augmentation by default URL: https://github.com/apache/incubator-mxnet/pull/13757#issuecomment-474151356 @ymjiang Could you please address the review comments made by @anirudhacharya. It seems no updates since last 2 weeks. Thanks!

[GitHub] [incubator-mxnet] karan6181 commented on issue #13763: [contrib][op] fix MultiBoxPrior confusing results if first ratio is not 1.0

2019-03-18 Thread GitBox
karan6181 commented on issue #13763: [contrib][op] fix MultiBoxPrior confusing results if first ratio is not 1.0 URL: https://github.com/apache/incubator-mxnet/pull/13763#issuecomment-474150843 @zhreshold Could you please provide an update on this PR? It seems no tracking for last 2

[GitHub] [incubator-mxnet] astonzhang opened a new pull request #14465: [MXNET-1361] Change 'The Straight Dope' to 'Dive into Deep Learning' on the website

2019-03-18 Thread GitBox
astonzhang opened a new pull request #14465: [MXNET-1361] Change 'The Straight Dope' to 'Dive into Deep Learning' on the website URL: https://github.com/apache/incubator-mxnet/pull/14465 ## Description ## Change *The Straight Dope* to *Dive into Deep Learning* on the website. JIRA:

[GitHub] [incubator-mxnet] karan6181 commented on issue #13906: [MXNET-703] Update onnx-tensorrt for int8/fp16 support

2019-03-18 Thread GitBox
karan6181 commented on issue #13906: [MXNET-703] Update onnx-tensorrt for int8/fp16 support URL: https://github.com/apache/incubator-mxnet/pull/13906#issuecomment-474150308 @KellenSunderland can you please address the review comments? @Roshrini @vandanavk Can you please review it?

[GitHub] [incubator-mxnet] karan6181 commented on issue #13955: adding the gluon implementation of deformable convolution

2019-03-18 Thread GitBox
karan6181 commented on issue #13955: adding the gluon implementation of deformable convolution URL: https://github.com/apache/incubator-mxnet/pull/13955#issuecomment-474149937 @suyz526 Can you please check the CI failure? No updates since couple of weeks.

[GitHub] [incubator-mxnet] karan6181 commented on issue #14028: Cmake blas

2019-03-18 Thread GitBox
karan6181 commented on issue #14028: Cmake blas URL: https://github.com/apache/incubator-mxnet/pull/14028#issuecomment-474149586 @edisongustavo Thank you for your contributions! Can you please provide some updates in this PR? Thanks!

[GitHub] [incubator-mxnet] zachgk edited a comment on issue #14434: could i set multi kv_store in distribute training program?

2019-03-18 Thread GitBox
zachgk edited a comment on issue #14434: could i set multi kv_store in distribute training program? URL: https://github.com/apache/incubator-mxnet/issues/14434#issuecomment-474149386 @mxnet-label-bot add [Feature request]

[GitHub] [incubator-mxnet] zachgk commented on issue #14434: could i set multi kv_store in distribute training program?

2019-03-18 Thread GitBox
zachgk commented on issue #14434: could i set multi kv_store in distribute training program? URL: https://github.com/apache/incubator-mxnet/issues/14434#issuecomment-474149386 @mxnet-label-bot add [Feature] This is an

[GitHub] [incubator-mxnet] karan6181 commented on issue #14040: Reformat of TensorRT to use subgraph API

2019-03-18 Thread GitBox
karan6181 commented on issue #14040: Reformat of TensorRT to use subgraph API URL: https://github.com/apache/incubator-mxnet/pull/14040#issuecomment-474149318 @KellenSunderland and @anirudh2290 could you please review this PR again. Thanks !

[GitHub] [incubator-mxnet] zachgk commented on issue #14438: Fixing unintentional variable overloading

2019-03-18 Thread GitBox
zachgk commented on issue #14438: Fixing unintentional variable overloading URL: https://github.com/apache/incubator-mxnet/pull/14438#issuecomment-474148975 @zjost Cool trick: If you edit the PR description and add `resolves #14439`, then it will automatically resolve the issue when the PR

[GitHub] [incubator-mxnet] abhinavs95 commented on a change in pull request #14442: [MXNet-1349][WIP][Fit API]Add validation support and unit tests for fit() API

2019-03-18 Thread GitBox
abhinavs95 commented on a change in pull request #14442: [MXNet-1349][WIP][Fit API]Add validation support and unit tests for fit() API URL: https://github.com/apache/incubator-mxnet/pull/14442#discussion_r266688958 ## File path: python/mxnet/gluon/estimator/estimator.py ##

[GitHub] [incubator-mxnet] haojin2 commented on a change in pull request #14449: GELU Operator

2019-03-18 Thread GitBox
haojin2 commented on a change in pull request #14449: GELU Operator URL: https://github.com/apache/incubator-mxnet/pull/14449#discussion_r266688546 ## File path: src/operator/mshadow_op.h ## @@ -127,6 +131,20 @@ MXNET_UNARY_MATH_OP(softsign, a / (1.0f + math::fabs(a)));

[GitHub] [incubator-mxnet] zachgk commented on issue #14383: MXNET_BACKWARD_DO_MIRROR is broken

2019-03-18 Thread GitBox
zachgk commented on issue #14383: MXNET_BACKWARD_DO_MIRROR is broken URL: https://github.com/apache/incubator-mxnet/issues/14383#issuecomment-474148127 @mxnet-label-bot add [Gluon, Bug] This is an automated message from the

[GitHub] [incubator-mxnet] karan6181 commented on issue #14053: in-place reshape ops

2019-03-18 Thread GitBox
karan6181 commented on issue #14053: in-place reshape ops URL: https://github.com/apache/incubator-mxnet/pull/14053#issuecomment-474148069 @szha Thank you for your contributions! Is this PR work in progress? This is an

[GitHub] [incubator-mxnet] andrewfayres commented on issue #14458: Create a Dataset Using RecordIO

2019-03-18 Thread GitBox
andrewfayres commented on issue #14458: Create a Dataset Using RecordIO URL: https://github.com/apache/incubator-mxnet/issues/14458#issuecomment-474147629 @mxnet-label-bot add [Question, Example] This is an automated message

[GitHub] [incubator-mxnet] karan6181 commented on issue #14071: [MXNET-1269] Scoverage

2019-03-18 Thread GitBox
karan6181 commented on issue #14071: [MXNET-1269] Scoverage URL: https://github.com/apache/incubator-mxnet/pull/14071#issuecomment-474147486 @yzhliu and @nswamy Can you please review this PR. @zachgk Can you please look into the CI failure?

[GitHub] [incubator-mxnet] piyushghai commented on a change in pull request #14442: [MXNet-1349][WIP][Fit API]Add validation support and unit tests for fit() API

2019-03-18 Thread GitBox
piyushghai commented on a change in pull request #14442: [MXNet-1349][WIP][Fit API]Add validation support and unit tests for fit() API URL: https://github.com/apache/incubator-mxnet/pull/14442#discussion_r266687712 ## File path: tests/python/unittest/test_gluon_estimator.py

[GitHub] [incubator-mxnet] piyushghai commented on a change in pull request #14442: [MXNet-1349][WIP][Fit API]Add validation support and unit tests for fit() API

2019-03-18 Thread GitBox
piyushghai commented on a change in pull request #14442: [MXNet-1349][WIP][Fit API]Add validation support and unit tests for fit() API URL: https://github.com/apache/incubator-mxnet/pull/14442#discussion_r266687402 ## File path: python/mxnet/gluon/estimator/estimator.py ##

[GitHub] [incubator-mxnet] tlby commented on issue #14461: [MXNET-1359] Adds a multiclass-MCC metric derived from Pearson

2019-03-18 Thread GitBox
tlby commented on issue #14461: [MXNET-1359] Adds a multiclass-MCC metric derived from Pearson URL: https://github.com/apache/incubator-mxnet/pull/14461#issuecomment-474147085 hmm, actually `numpy.corrcoef()` may not be working how I need for the multiclass class. This work is based on

[GitHub] [incubator-mxnet] zachgk commented on issue #14447: Seg Fault while using Randomized relu activation function

2019-03-18 Thread GitBox
zachgk commented on issue #14447: Seg Fault while using Randomized relu activation function URL: https://github.com/apache/incubator-mxnet/issues/14447#issuecomment-474147028 @mxnet-label-bot add [Backend, Operator, Bug]

[GitHub] [incubator-mxnet] piyushghai commented on a change in pull request #14464: Fixed issue where the estimator was printing beyond the dataset size …

2019-03-18 Thread GitBox
piyushghai commented on a change in pull request #14464: Fixed issue where the estimator was printing beyond the dataset size … URL: https://github.com/apache/incubator-mxnet/pull/14464#discussion_r266687216 ## File path: python/mxnet/gluon/estimator/estimator.py ## @@

[GitHub] [incubator-mxnet] karan6181 commented on issue #14078: [MXNET-14075] repair example, enable fault tolerant build of R reference manual

2019-03-18 Thread GitBox
karan6181 commented on issue #14078: [MXNET-14075] repair example, enable fault tolerant build of R reference manual URL: https://github.com/apache/incubator-mxnet/pull/14078#issuecomment-474146857 @shadogray is the build still failing? I see no activities since Feb-15 on this PR. Can you

[GitHub] [incubator-mxnet] roywei commented on a change in pull request #14464: Fixed issue where the estimator was printing beyond the dataset size …

2019-03-18 Thread GitBox
roywei commented on a change in pull request #14464: Fixed issue where the estimator was printing beyond the dataset size … URL: https://github.com/apache/incubator-mxnet/pull/14464#discussion_r266686740 ## File path: python/mxnet/gluon/estimator/estimator.py ## @@ -242,7

[GitHub] [incubator-mxnet] andrewfayres commented on issue #14450: broken clojure test

2019-03-18 Thread GitBox
andrewfayres commented on issue #14450: broken clojure test URL: https://github.com/apache/incubator-mxnet/issues/14450#issuecomment-474146585 Issue was resolved with #14448 which disabled the tests. Those tests have a dependency on an external repository which was experiencing an outage.

[GitHub] [incubator-mxnet] zachgk commented on issue #14456: python import error

2019-03-18 Thread GitBox
zachgk commented on issue #14456: python import error URL: https://github.com/apache/incubator-mxnet/issues/14456#issuecomment-474146260 @mxnet-label-bot add [Build] This is an automated message from the Apache Git Service.

[GitHub] [incubator-mxnet] karan6181 commented on issue #14113: Enhance subgraph API

2019-03-18 Thread GitBox
karan6181 commented on issue #14113: Enhance subgraph API URL: https://github.com/apache/incubator-mxnet/pull/14113#issuecomment-474146156 @ZhennanQin Could you please re-trigger the CI and see why some of the checks are still waiting? Thanks !

[GitHub] [incubator-mxnet] piyushghai opened a new pull request #14464: Fixed issue where the estimator was printing beyond the dataset size …

2019-03-18 Thread GitBox
piyushghai opened a new pull request #14464: Fixed issue where the estimator was printing beyond the dataset size … URL: https://github.com/apache/incubator-mxnet/pull/14464 …for the last batch ## Description ## As title. @roywei @abhinavs95 @karan6181

[GitHub] [incubator-mxnet] piyushghai commented on issue #14464: Fixed issue where the estimator was printing beyond the dataset size …

2019-03-18 Thread GitBox
piyushghai commented on issue #14464: Fixed issue where the estimator was printing beyond the dataset size … URL: https://github.com/apache/incubator-mxnet/pull/14464#issuecomment-474146088 @mxnet-label-bot Add [Gluon] This

[GitHub] [incubator-mxnet] karan6181 commented on issue #14140: Doc Fix: added note about cuda9.2 requirement to Java example

2019-03-18 Thread GitBox
karan6181 commented on issue #14140: Doc Fix: added note about cuda9.2 requirement to Java example URL: https://github.com/apache/incubator-mxnet/pull/14140#issuecomment-474145921 @samskalicky Can you please address the review comments ? Thanks!

[GitHub] [incubator-mxnet] zachgk commented on issue #14457: [Flaky Test] Python 3: GPU WIN test_operator_gpu.test_multinomial_generator

2019-03-18 Thread GitBox
zachgk commented on issue #14457: [Flaky Test] Python 3: GPU WIN test_operator_gpu.test_multinomial_generator URL: https://github.com/apache/incubator-mxnet/issues/14457#issuecomment-474145714 @mxnet-label-bot add [Test, Flaky, Windows, Python]

[GitHub] [incubator-mxnet] karan6181 commented on issue #14196: Group Normalization

2019-03-18 Thread GitBox
karan6181 commented on issue #14196: Group Normalization URL: https://github.com/apache/incubator-mxnet/pull/14196#issuecomment-474145564 @zhanghang1989 Can you please address the review comments and try to merge this PR? Thanks!

[GitHub] [incubator-mxnet] zachgk commented on issue #14458: Create a Dataset Using RecordIO

2019-03-18 Thread GitBox
zachgk commented on issue #14458: Create a Dataset Using RecordIO URL: https://github.com/apache/incubator-mxnet/issues/14458#issuecomment-474145403 For questions, please submit them on MXNet discussion forum (https://discuss.mxnet.io), where it will get a wider audience and allow other

[GitHub] [incubator-mxnet] karan6181 commented on issue #14319: CI Changes for Codified Windows AMIs

2019-03-18 Thread GitBox
karan6181 commented on issue #14319: CI Changes for Codified Windows AMIs URL: https://github.com/apache/incubator-mxnet/pull/14319#issuecomment-474145226 @Roshrini Can you please close this PR since it is already been merged in this PR

[GitHub] [incubator-mxnet] piyushghai commented on a change in pull request #14462: [MXNET-1358][Fit API] Fit api tutorial

2019-03-18 Thread GitBox
piyushghai commented on a change in pull request #14462: [MXNET-1358][Fit API] Fit api tutorial URL: https://github.com/apache/incubator-mxnet/pull/14462#discussion_r266684296 ## File path: docs/tutorials/gluon/fit_api_tutorial.md ## @@ -0,0 +1,311 @@ + + + + + + + + + +

[GitHub] [incubator-mxnet] piyushghai commented on a change in pull request #14462: [MXNET-1358][Fit API] Fit api tutorial

2019-03-18 Thread GitBox
piyushghai commented on a change in pull request #14462: [MXNET-1358][Fit API] Fit api tutorial URL: https://github.com/apache/incubator-mxnet/pull/14462#discussion_r266684214 ## File path: docs/tutorials/gluon/fit_api_tutorial.md ## @@ -0,0 +1,311 @@ + + + + + + + + + +

[GitHub] [incubator-mxnet] lupesko commented on issue #10792: [MXNET-400] support string type for kvstore key in cpp-package

2019-03-18 Thread GitBox
lupesko commented on issue #10792: [MXNET-400] support string type for kvstore key in cpp-package URL: https://github.com/apache/incubator-mxnet/pull/10792#issuecomment-474144072 @nihui - since there was no feedback/follow up on feedback since May 2018, we should probably close the PR.

[GitHub] [incubator-mxnet] zachgk commented on issue #14459: Cannot release memory sometimes

2019-03-18 Thread GitBox
zachgk commented on issue #14459: Cannot release memory sometimes URL: https://github.com/apache/incubator-mxnet/issues/14459#issuecomment-474142486 @ShownX Can you provide some more information? Can you give the example or a minimal script that causes the problem? Can you also

[GitHub] [incubator-mxnet] zachgk removed a comment on issue #14459: Cannot release memory sometimes

2019-03-18 Thread GitBox
zachgk removed a comment on issue #14459: Cannot release memory sometimes URL: https://github.com/apache/incubator-mxnet/issues/14459#issuecomment-474142540 @mxnet-label-bot add [Memory, Exception Handling, Pending Requester Info]

[GitHub] [incubator-mxnet] zachgk commented on issue #14459: Cannot release memory sometimes

2019-03-18 Thread GitBox
zachgk commented on issue #14459: Cannot release memory sometimes URL: https://github.com/apache/incubator-mxnet/issues/14459#issuecomment-474142540 @mxnet-label-bot add [Memory, Exception Handling, Pending Requester Info]

[GitHub] [incubator-mxnet] abhinavs95 commented on a change in pull request #14462: [MXNET-1358][Fit API] Fit api tutorial

2019-03-18 Thread GitBox
abhinavs95 commented on a change in pull request #14462: [MXNET-1358][Fit API] Fit api tutorial URL: https://github.com/apache/incubator-mxnet/pull/14462#discussion_r266680655 ## File path: docs/tutorials/gluon/fit_api_tutorial.md ## @@ -0,0 +1,311 @@ + + + + + + + + + +

[GitHub] [incubator-mxnet] abhinavs95 commented on a change in pull request #14462: [MXNET-1358][Fit API] Fit api tutorial

2019-03-18 Thread GitBox
abhinavs95 commented on a change in pull request #14462: [MXNET-1358][Fit API] Fit api tutorial URL: https://github.com/apache/incubator-mxnet/pull/14462#discussion_r266682419 ## File path: docs/tutorials/gluon/fit_api_tutorial.md ## @@ -0,0 +1,311 @@ + + + + + + + + + +

[GitHub] [incubator-mxnet] zachgk commented on issue #14348: [WIP][MXNET-1302, MXNET-1331] Backport fix to v1.3.x branch

2019-03-18 Thread GitBox
zachgk commented on issue #14348: [WIP][MXNET-1302,MXNET-1331] Backport fix to v1.3.x branch URL: https://github.com/apache/incubator-mxnet/pull/14348#issuecomment-474137523 @perdasilva Can you help take a look at this? It seems to be failing on the Windows CI

[GitHub] [incubator-mxnet] piyushghai commented on issue #14462: [MXNET-1358][Fit API] Fit api tutorial

2019-03-18 Thread GitBox
piyushghai commented on issue #14462: [MXNET-1358][Fit API] Fit api tutorial URL: https://github.com/apache/incubator-mxnet/pull/14462#issuecomment-474135267 @mxnet-label-bot Add [Gluon, pr-awaiting-review] This is an

[GitHub] [incubator-mxnet] zachgk commented on issue #14463: Enforce determinism for backwards compatibility checker

2019-03-18 Thread GitBox
zachgk commented on issue #14463: Enforce determinism for backwards compatibility checker URL: https://github.com/apache/incubator-mxnet/pull/14463#issuecomment-474134710 @mxnet-label-bot add [CI] This is an automated

[GitHub] [incubator-mxnet] wkcn commented on a change in pull request #13886: Fixes for wine detection tutorial

2019-03-18 Thread GitBox
wkcn commented on a change in pull request #13886: Fixes for wine detection tutorial URL: https://github.com/apache/incubator-mxnet/pull/13886#discussion_r266675529 ## File path: docs/tutorials/embedded/wine_detector.md ## @@ -100,13 +101,14 @@ with open('synset.txt',

[GitHub] [incubator-mxnet] zachgk opened a new pull request #14463: Enforce determinism for backwards compatibility checker

2019-03-18 Thread GitBox
zachgk opened a new pull request #14463: Enforce determinism for backwards compatibility checker URL: https://github.com/apache/incubator-mxnet/pull/14463 ## Description ## Adds the enforce determinism setting when running the backwards compatibility checker to avoid flaky issues due

[GitHub] [incubator-mxnet] piyushghai opened a new pull request #14462: [MXNET-1358] Fit api tutorial

2019-03-18 Thread GitBox
piyushghai opened a new pull request #14462: [MXNET-1358] Fit api tutorial URL: https://github.com/apache/incubator-mxnet/pull/14462 ## Description ## Added a simple tutorial showcasing the fit api usage. ## Checklist ## ### Essentials ### Please feel free to remove

[GitHub] [incubator-mxnet] anirudh2290 commented on issue #14451: fix custom operation in fork

2019-03-18 Thread GitBox
anirudh2290 commented on issue #14451: fix custom operation in fork URL: https://github.com/apache/incubator-mxnet/pull/14451#issuecomment-474131945 I think this may be related to https://docs.python.org/3/c-api/init.html#non-python-created-threads . Quoting from the doc: >

[GitHub] [incubator-mxnet] YutingZhang commented on issue #14451: fix custom operation in fork

2019-03-18 Thread GitBox
YutingZhang commented on issue #14451: fix custom operation in fork URL: https://github.com/apache/incubator-mxnet/pull/14451#issuecomment-474129832 @arcadiaphy Does this also related to https://discuss.mxnet.io/t/custom-operator-cause-stuck-in-multicard-train/1452 Basically, a CustomOP

[GitHub] [incubator-mxnet] apeforest commented on issue #10002: General support of OPs for second-order gradient

2019-03-18 Thread GitBox
apeforest commented on issue #10002: General support of OPs for second-order gradient URL: https://github.com/apache/incubator-mxnet/issues/10002#issuecomment-474122803 @lonelykid We are actively working on the higher order gradient feature right now. We will update you once the PR is

[GitHub] [incubator-mxnet] tlby opened a new pull request #14461: [MXNET-1359] Adds a multiclass-MCC metric derived from Pearson

2019-03-18 Thread GitBox
tlby opened a new pull request #14461: [MXNET-1359] Adds a multiclass-MCC metric derived from Pearson URL: https://github.com/apache/incubator-mxnet/pull/14461 ## Description ## A multiclass metric equivalent to `mxnet.metric.MCC` can be derived from `mxnet.metric.PearsonCorrelation`

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2019-03-18 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new 40bfba8 Bump the publish

[GitHub] [incubator-mxnet] eric-haibin-lin commented on a change in pull request #14449: GELU Operator

2019-03-18 Thread GitBox
eric-haibin-lin commented on a change in pull request #14449: GELU Operator URL: https://github.com/apache/incubator-mxnet/pull/14449#discussion_r266635416 ## File path: src/operator/mshadow_op.h ## @@ -127,6 +131,20 @@ MXNET_UNARY_MATH_OP(softsign, a / (1.0f +

[GitHub] [incubator-mxnet] eric-haibin-lin closed issue #14358: Memory builds up when creating size-zero NDArray in a loop

2019-03-18 Thread GitBox
eric-haibin-lin closed issue #14358: Memory builds up when creating size-zero NDArray in a loop URL: https://github.com/apache/incubator-mxnet/issues/14358 This is an automated message from the Apache Git Service. To

[incubator-mxnet] branch master updated: Fix memory leak for size-zero ndarray (#14365)

2019-03-18 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 3ab1dec Fix memory leak for size-zero

[GitHub] [incubator-mxnet] eric-haibin-lin closed issue #13951: Gluon RNN memory leaks with extra variables

2019-03-18 Thread GitBox
eric-haibin-lin closed issue #13951: Gluon RNN memory leaks with extra variables URL: https://github.com/apache/incubator-mxnet/issues/13951 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-mxnet] eric-haibin-lin merged pull request #14365: Fix memory leak for size-zero ndarray

2019-03-18 Thread GitBox
eric-haibin-lin merged pull request #14365: Fix memory leak for size-zero ndarray URL: https://github.com/apache/incubator-mxnet/pull/14365 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-mxnet] haojin2 commented on issue #14449: GELU Operator

2019-03-18 Thread GitBox
haojin2 commented on issue #14449: GELU Operator URL: https://github.com/apache/incubator-mxnet/pull/14449#issuecomment-474090439 @szha Gluon test added, also fixed a problem with existing SELU test. This is an automated

[GitHub] [incubator-mxnet] abhinavs95 commented on a change in pull request #14405: [MXNet-1343][WIP][Fit API]Add CNN integration test for fit() API

2019-03-18 Thread GitBox
abhinavs95 commented on a change in pull request #14405: [MXNet-1343][WIP][Fit API]Add CNN integration test for fit() API URL: https://github.com/apache/incubator-mxnet/pull/14405#discussion_r266624361 ## File path: tests/nightly/estimator/test_estimator_cnn_gpu.py ## @@

[GitHub] [incubator-mxnet] abhinavs95 commented on a change in pull request #14405: [MXNet-1343][WIP][Fit API]Add CNN integration test for fit() API

2019-03-18 Thread GitBox
abhinavs95 commented on a change in pull request #14405: [MXNet-1343][WIP][Fit API]Add CNN integration test for fit() API URL: https://github.com/apache/incubator-mxnet/pull/14405#discussion_r266624285 ## File path: tests/nightly/estimator/test_estimator_cnn_cpu.py ## @@

[GitHub] [incubator-mxnet] abhinavs95 commented on a change in pull request #14405: [MXNet-1343][WIP][Fit API]Add CNN integration test for fit() API

2019-03-18 Thread GitBox
abhinavs95 commented on a change in pull request #14405: [MXNet-1343][WIP][Fit API]Add CNN integration test for fit() API URL: https://github.com/apache/incubator-mxnet/pull/14405#discussion_r266624147 ## File path: tests/nightly/estimator/test_estimator_cnn_cpu.py ## @@

[incubator-mxnet-site] branch asf-site updated: Bump the publish timestamp.

2019-03-18 Thread zhasheng
This is an automated email from the ASF dual-hosted git repository. zhasheng pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-mxnet-site.git The following commit(s) were added to refs/heads/asf-site by this push: new be7a732 Bump the publish

[GitHub] [incubator-mxnet] roywei commented on a change in pull request #14405: [MXNet-1343][WIP][Fit API]Add CNN integration test for fit() API

2019-03-18 Thread GitBox
roywei commented on a change in pull request #14405: [MXNet-1343][WIP][Fit API]Add CNN integration test for fit() API URL: https://github.com/apache/incubator-mxnet/pull/14405#discussion_r266584922 ## File path: tests/nightly/estimator/test_estimator_cnn_cpu.py ## @@ -0,0

[GitHub] [incubator-mxnet] roywei commented on a change in pull request #14405: [MXNet-1343][WIP][Fit API]Add CNN integration test for fit() API

2019-03-18 Thread GitBox
roywei commented on a change in pull request #14405: [MXNet-1343][WIP][Fit API]Add CNN integration test for fit() API URL: https://github.com/apache/incubator-mxnet/pull/14405#discussion_r266583109 ## File path: tests/nightly/estimator/test_estimator_cnn_gpu.py ## @@ -0,0

[GitHub] [incubator-mxnet] roywei commented on a change in pull request #14405: [MXNet-1343][WIP][Fit API]Add CNN integration test for fit() API

2019-03-18 Thread GitBox
roywei commented on a change in pull request #14405: [MXNet-1343][WIP][Fit API]Add CNN integration test for fit() API URL: https://github.com/apache/incubator-mxnet/pull/14405#discussion_r266585192 ## File path: tests/nightly/estimator/test_estimator_cnn_cpu.py ## @@ -0,0

  1   2   >