[GitHub] [incubator-mxnet] ChaiBapchya commented on pull request #18326: [R] Fix incorrect copyto usage & incorrect website title for Symbol API in R

2020-05-14 Thread GitBox
ChaiBapchya commented on pull request #18326: URL: https://github.com/apache/incubator-mxnet/pull/18326#issuecomment-629044610 @mxnet-bot run ci [unix-cpu] This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-mxnet] ChaiBapchya commented on issue #18330: test_metric_performance

2020-05-14 Thread GitBox
ChaiBapchya commented on issue #18330: URL: https://github.com/apache/incubator-mxnet/issues/18330#issuecomment-629044532 http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Funix-cpu/detail/PR-18326/1/pipeline #18326 unrelated PR

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18326: [R] Fix incorrect copyto usage & incorrect website title for Symbol API in R

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18326: URL: https://github.com/apache/incubator-mxnet/pull/18326#issuecomment-629044650 Jenkins CI successfully triggered : [unix-cpu] This is an automated message from the Apache Git

[incubator-mxnet] branch master updated (4d4cbd5 -> fec534a)

2020-05-14 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git. from 4d4cbd5 xfail_when_nonstandard_decimal_separator for test_metric.py (#18312) add fec534a Fix

[GitHub] [incubator-mxnet-ci] ChaiBapchya commented on a change in pull request #25: Serverless implementation for jenkins pipeline monitor lambda

2020-05-14 Thread GitBox
ChaiBapchya commented on a change in pull request #25: URL: https://github.com/apache/incubator-mxnet-ci/pull/25#discussion_r425581784 ## File path: services/jenkins-pipeline-monitor/handler.py ## @@ -0,0 +1,134 @@ +import os +import boto3 +import json +import logging +import

[incubator-mxnet] branch master updated: Fix missing MKLDNN headers (#18310)

2020-05-14 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen 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 fec534a Fix missing MKLDNN headers

[GitHub] [incubator-mxnet] leezu closed issue #18120: MKLDNN header missing in recent mxnet nightly static builds

2020-05-14 Thread GitBox
leezu closed issue #18120: URL: https://github.com/apache/incubator-mxnet/issues/18120 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [incubator-mxnet] leezu merged pull request #18310: Fix MKLDNN headers missing in nightly builds

2020-05-14 Thread GitBox
leezu merged pull request #18310: URL: https://github.com/apache/incubator-mxnet/pull/18310 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [incubator-mxnet-ci] ChaiBapchya commented on a change in pull request #25: Serverless implementation for jenkins pipeline monitor lambda

2020-05-14 Thread GitBox
ChaiBapchya commented on a change in pull request #25: URL: https://github.com/apache/incubator-mxnet-ci/pull/25#discussion_r425576336 ## File path: services/jenkins-pipeline-monitor/handler.py ## @@ -0,0 +1,134 @@ +import os +import boto3 +import json +import logging +import

[GitHub] [incubator-mxnet-ci] ChaiBapchya commented on a change in pull request #25: Serverless implementation for jenkins pipeline monitor lambda

2020-05-14 Thread GitBox
ChaiBapchya commented on a change in pull request #25: URL: https://github.com/apache/incubator-mxnet-ci/pull/25#discussion_r425575843 ## File path: services/jenkins-pipeline-monitor/deploy_lambda.sh ## @@ -0,0 +1,16 @@ +set -e + +echo "Deployment stage (test, prod)" +read

[GitHub] [incubator-mxnet] eric-haibin-lin commented on a change in pull request #17749: Fix races in block scope

2020-05-14 Thread GitBox
eric-haibin-lin commented on a change in pull request #17749: URL: https://github.com/apache/incubator-mxnet/pull/17749#discussion_r425564767 ## File path: python/mxnet/gluon/block.py ## @@ -91,22 +92,22 @@ def create(prefix, params, hint): def __enter__(self):

[GitHub] [incubator-mxnet] mseth10 closed pull request #18318: Revert "Changes to mxnet.metric (#18083)"

2020-05-14 Thread GitBox
mseth10 closed pull request #18318: URL: https://github.com/apache/incubator-mxnet/pull/18318 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-mxnet] Alicia1529 commented on pull request #17672: [Numpy] more support for boolean indexing/assign and change on implementation

2020-05-14 Thread GitBox
Alicia1529 commented on pull request #17672: URL: https://github.com/apache/incubator-mxnet/pull/17672#issuecomment-629014274 > @Alicia1529 , can you help to back port this PR to v1.7.x and v1.x branches as well, which will be included in the upcoming 1.7.0 release? As the issue is

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18328: Fix memory leaks in Gluon

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18328: URL: https://github.com/apache/incubator-mxnet/pull/18328#issuecomment-629011853 Jenkins CI successfully triggered : [centos-gpu, windows-gpu] This is an automated message from the

[GitHub] [incubator-mxnet] leezu commented on pull request #18328: Fix memory leaks in Gluon

2020-05-14 Thread GitBox
leezu commented on pull request #18328: URL: https://github.com/apache/incubator-mxnet/pull/18328#issuecomment-629011820 @mxnet-bot run ci [centos-gpu, windows-gpu] This is an automated message from the Apache Git Service.

[GitHub] [incubator-mxnet] chinakook closed issue #18170: /usr/bin/ld fail when build

2020-05-14 Thread GitBox
chinakook closed issue #18170: URL: https://github.com/apache/incubator-mxnet/issues/18170 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [incubator-mxnet] chinakook commented on issue #18170: /usr/bin/ld fail when build

2020-05-14 Thread GitBox
chinakook commented on issue #18170: URL: https://github.com/apache/incubator-mxnet/issues/18170#issuecomment-628999182 @ptrendx Thank for your reply. It's solved by using right version of gfortran in Ubuntu 20.04. This is

[GitHub] [incubator-mxnet] ys2843 commented on a change in pull request #18288: Website global search feature

2020-05-14 Thread GitBox
ys2843 commented on a change in pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#discussion_r425533141 ## File path: docs/static_site/src/assets/js/globalSearch.js ## @@ -0,0 +1,152 @@ +/*! + * Licensed to the Apache Software Foundation (ASF)

[GitHub] [incubator-mxnet] ys2843 edited a comment on pull request #18288: Website global search feature

2020-05-14 Thread GitBox
ys2843 edited a comment on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-628990327 > Looks good. > > * Should we use offsite js files or host locally? > * Should the API key be in a js file are can it be in a config file so we keep

[GitHub] [incubator-mxnet] ys2843 edited a comment on pull request #18288: Website global search feature

2020-05-14 Thread GitBox
ys2843 edited a comment on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-628990327 > Looks good. > > * Should we use offsite js files or host locally? > * Should the API key be in a js file are can it be in a config file so we keep

[GitHub] [incubator-mxnet] ys2843 commented on pull request #18288: Website global search feature

2020-05-14 Thread GitBox
ys2843 commented on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-628990327 > Looks good. > > * Should we use offsite js files or host locally? > * Should the API key be in a js file are can it be in a config file so we keep this

[GitHub] [incubator-mxnet] leezu commented on pull request #18310: Fix MKLDNN headers missing in nightly builds

2020-05-14 Thread GitBox
leezu commented on pull request #18310: URL: https://github.com/apache/incubator-mxnet/pull/18310#issuecomment-628983396 @mxnet-bot run ci [unix-cpu] This is an automated message from the Apache Git Service. To

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18310: Fix MKLDNN headers missing in nightly builds

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18310: URL: https://github.com/apache/incubator-mxnet/pull/18310#issuecomment-628983414 Jenkins CI successfully triggered : [unix-cpu] This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] ys2843 commented on a change in pull request #18288: Website global search feature

2020-05-14 Thread GitBox
ys2843 commented on a change in pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#discussion_r425524968 ## File path: docs/static_site/src/_includes/head.html ## @@ -5,14 +5,15 @@

[GitHub] [incubator-mxnet] leezu closed issue #18329: 'mxnet' has no attribute 'metric'

2020-05-14 Thread GitBox
leezu closed issue #18329: URL: https://github.com/apache/incubator-mxnet/issues/18329 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [incubator-mxnet] leezu commented on issue #18329: 'mxnet' has no attribute 'metric'

2020-05-14 Thread GitBox
leezu commented on issue #18329: URL: https://github.com/apache/incubator-mxnet/issues/18329#issuecomment-628977295 MXNet 2 moves the metric API to mx.gluon.metric gluoncv is not yet compatible with MXNet 2, but you can use MXNet 1.x branch Closing as this is not an issue in

[GitHub] [incubator-mxnet] eric-haibin-lin commented on a change in pull request #18306: Add a timeout to the storage profiler in case mem_counters_ is not yet initialized

2020-05-14 Thread GitBox
eric-haibin-lin commented on a change in pull request #18306: URL: https://github.com/apache/incubator-mxnet/pull/18306#discussion_r425521065 ## File path: src/profiler/storage_profiler.h ## @@ -52,6 +54,12 @@ class DeviceStorageProfiler { if

[GitHub] [incubator-mxnet] leezu opened a new issue #18330: test_metric_performance

2020-05-14 Thread GitBox
leezu opened a new issue #18330: URL: https://github.com/apache/incubator-mxnet/issues/18330 http://jenkins.mxnet-ci.amazon-ml.com/blue/rest/organizations/jenkins/pipelines/mxnet-validation/pipelines/unix-cpu/branches/PR-18284/runs/6/nodes/364/steps/731/log/?start=0 ```

[GitHub] [incubator-mxnet] heaseny opened a new issue #18329: 'mxnet' has no attribute 'metric'

2020-05-14 Thread GitBox
heaseny opened a new issue #18329: URL: https://github.com/apache/incubator-mxnet/issues/18329 ## Description On master branch, when execute the below commands, python -c "import gluoncv" or python -c "import mxnet as mx; print(mx.metric)" , an error raised with message: 'mxnet' has

[GitHub] [incubator-mxnet] leezu commented on pull request #18284: Fix parse operator attributes in new FFI

2020-05-14 Thread GitBox
leezu commented on pull request #18284: URL: https://github.com/apache/incubator-mxnet/pull/18284#issuecomment-628976260 @mxnet-bot run ci [unix-cpu, centos-cpu] This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18284: Fix parse operator attributes in new FFI

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18284: URL: https://github.com/apache/incubator-mxnet/pull/18284#issuecomment-628976282 Jenkins CI successfully triggered : [unix-cpu, centos-cpu] This is an automated message from the

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18315: Fix coredumps

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18315: URL: https://github.com/apache/incubator-mxnet/pull/18315#issuecomment-628976057 Jenkins CI successfully triggered : [unix-cpu] This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] leezu commented on pull request #18315: Fix coredumps

2020-05-14 Thread GitBox
leezu commented on pull request #18315: URL: https://github.com/apache/incubator-mxnet/pull/18315#issuecomment-628976016 @mxnet-bot run ci [unix-cpu] This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-mxnet] leezu edited a comment on issue #18314: ThreadedEngine: Segfault

2020-05-14 Thread GitBox
leezu edited a comment on issue #18314: URL: https://github.com/apache/incubator-mxnet/issues/18314#issuecomment-628969701 Let's close this issue as it's specific to the use of gc.DEBUG_SAVEALL. The leak is fixed in https://github.com/apache/incubator-mxnet/pull/18328

[GitHub] [incubator-mxnet] heaseny commented on pull request #18289: avoid skipping validation stage when data read iterator exists

2020-05-14 Thread GitBox
heaseny commented on pull request #18289: URL: https://github.com/apache/incubator-mxnet/pull/18289#issuecomment-628970117 @szha Can you help to review this pull request, and merge it if there is no problem? Thanks a lot.

[GitHub] [incubator-mxnet] leezu commented on issue #18314: ThreadedEngine: Segfault

2020-05-14 Thread GitBox
leezu commented on issue #18314: URL: https://github.com/apache/incubator-mxnet/issues/18314#issuecomment-628969701 Let's close this issue as it's specific to the use of gc.DEBUG_SAVEALL This is an automated message from

[GitHub] [incubator-mxnet] leezu closed issue #18314: ThreadedEngine: Segfault

2020-05-14 Thread GitBox
leezu closed issue #18314: URL: https://github.com/apache/incubator-mxnet/issues/18314 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18328: Fix memory leaks in Gluon

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18328: URL: https://github.com/apache/incubator-mxnet/pull/18328#issuecomment-628965083 Hey @leezu , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following

[GitHub] [incubator-mxnet] leezu opened a new pull request #18328: Fix memory leaks in Gluon

2020-05-14 Thread GitBox
leezu opened a new pull request #18328: URL: https://github.com/apache/incubator-mxnet/pull/18328 ## Description ## Previously the _BlockScope keeps references to the parameter ndarrays, preventing memory from being freed if a Block is not used anymore. Among other problems, this

[GitHub] [incubator-mxnet] JustinhoCHN commented on issue #17923: can not install mxnet on jetson nano, jetpack 4.3, mxnet 1.4

2020-05-14 Thread GitBox
JustinhoCHN commented on issue #17923: URL: https://github.com/apache/incubator-mxnet/issues/17923#issuecomment-628964002 > Do you still have the image which you used on your Jetson Nano? Nvidia keeps on updating their images regularly and I ended up with CUDA 10.2 :/ > > Edit:

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

2020-05-14 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 eefa875 Bump the

[GitHub] [incubator-mxnet] aaronmarkham commented on a change in pull request #18288: Website global search feature

2020-05-14 Thread GitBox
aaronmarkham commented on a change in pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#discussion_r425488410 ## File path: docs/static_site/src/_includes/head.html ## @@ -5,14 +5,15 @@

[GitHub] [incubator-mxnet] ChaiBapchya opened a new pull request #18327: update init.py with appropriate comments

2020-05-14 Thread GitBox
ChaiBapchya opened a new pull request #18327: URL: https://github.com/apache/incubator-mxnet/pull/18327 ## Description ## init.py imports submodules within mxnet binary. It includes incorrect comment & incomplete comments. This PR adds & updates accordingly. ## Checklist ##

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18327: update init.py with appropriate comments

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18327: URL: https://github.com/apache/incubator-mxnet/pull/18327#issuecomment-628941994 Hey @ChaiBapchya , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18326: [R] Fix incorrect copyto usage & incorrect website title for Symbol API in R

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18326: URL: https://github.com/apache/incubator-mxnet/pull/18326#issuecomment-628939287 Hey @ChaiBapchya , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the

[GitHub] [incubator-mxnet] ChaiBapchya opened a new pull request #18326: [R] Fix incorrect copyto usage & incorrect website title for Symbol API in R

2020-05-14 Thread GitBox
ChaiBapchya opened a new pull request #18326: URL: https://github.com/apache/incubator-mxnet/pull/18326 ## Description ## 1. `mx.nd.copyto` doesnt exist 2. website title shows ndarray instead of symbol ## Checklist ## ### Essentials ### Please feel free to remove

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18325: Change SGD with momentum to include momentum correction by default

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18325: URL: https://github.com/apache/incubator-mxnet/pull/18325#issuecomment-628926177 Hey @ptrendx , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following

[GitHub] [incubator-mxnet] ptrendx opened a new pull request #18325: Change SGD with momentum to include momentum correction by default

2020-05-14 Thread GitBox
ptrendx opened a new pull request #18325: URL: https://github.com/apache/incubator-mxnet/pull/18325 ## Description ## Fixes #15533 Notable change needed to happen to LARS optimizer, since it actually requires the lack of momentum correction, so I had to emulate it in the

[GitHub] [incubator-mxnet] apeforest commented on issue #10673: Segmentation Fault when using large mx.nd.array

2020-05-14 Thread GitBox
apeforest commented on issue #10673: URL: https://github.com/apache/incubator-mxnet/issues/10673#issuecomment-628915089 Please build with USE_INT64_TENSOR_SIZE flag. Closing the issue This is an automated message from the

[GitHub] [incubator-mxnet] apeforest closed issue #10673: Segmentation Fault when using large mx.nd.array

2020-05-14 Thread GitBox
apeforest closed issue #10673: URL: https://github.com/apache/incubator-mxnet/issues/10673 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [incubator-mxnet] eric-haibin-lin commented on issue #14022: Segmentation fault during inference with quantization

2020-05-14 Thread GitBox
eric-haibin-lin commented on issue #14022: URL: https://github.com/apache/incubator-mxnet/issues/14022#issuecomment-628910827 closing this for now. please feel free to comment/reopen if you come across this issue again

[GitHub] [incubator-mxnet] eric-haibin-lin closed issue #14022: Segmentation fault during inference with quantization

2020-05-14 Thread GitBox
eric-haibin-lin closed issue #14022: URL: https://github.com/apache/incubator-mxnet/issues/14022 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-mxnet] eric-haibin-lin closed issue #11846: Bug: quantization Layers seg fault instead of displaying proper error message

2020-05-14 Thread GitBox
eric-haibin-lin closed issue #11846: URL: https://github.com/apache/incubator-mxnet/issues/11846 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-mxnet] eric-haibin-lin commented on issue #11846: Bug: quantization Layers seg fault instead of displaying proper error message

2020-05-14 Thread GitBox
eric-haibin-lin commented on issue #11846: URL: https://github.com/apache/incubator-mxnet/issues/11846#issuecomment-628910582 closing this for now since mxnet by default is built with mkldnn This is an automated message

[GitHub] [incubator-mxnet] eric-haibin-lin commented on issue #10724: Flaky test MKLDNN test_bucket_module

2020-05-14 Thread GitBox
eric-haibin-lin commented on issue #10724: URL: https://github.com/apache/incubator-mxnet/issues/10724#issuecomment-628910311 closing this for now. Please feel free to comment/reopen if the issue comes up again This is an

[GitHub] [incubator-mxnet] eric-haibin-lin closed issue #10724: Flaky test MKLDNN test_bucket_module

2020-05-14 Thread GitBox
eric-haibin-lin closed issue #10724: URL: https://github.com/apache/incubator-mxnet/issues/10724 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-mxnet] eric-haibin-lin closed issue #11756: [MXNET-677] int8 quantization does not work on toy mnist dataset

2020-05-14 Thread GitBox
eric-haibin-lin closed issue #11756: URL: https://github.com/apache/incubator-mxnet/issues/11756 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-mxnet] eric-haibin-lin commented on issue #11756: [MXNET-677] int8 quantization does not work on toy mnist dataset

2020-05-14 Thread GitBox
eric-haibin-lin commented on issue #11756: URL: https://github.com/apache/incubator-mxnet/issues/11756#issuecomment-628909891 Closing this issue for now. Please feel free to reopen if you come across this again This is an

[GitHub] [incubator-mxnet] eric-haibin-lin commented on issue #10673: Segmentation Fault when using large mx.nd.array

2020-05-14 Thread GitBox
eric-haibin-lin commented on issue #10673: URL: https://github.com/apache/incubator-mxnet/issues/10673#issuecomment-628910048 @access2rohit @apeforest can we close this issue now? This is an automated message from the

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18315: Fix coredumps

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18315: URL: https://github.com/apache/incubator-mxnet/pull/18315#issuecomment-628895708 Jenkins CI successfully triggered : [unix-cpu, centos-cpu] This is an automated message from the

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18310: Fix MKLDNN headers missing in nightly builds

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18310: URL: https://github.com/apache/incubator-mxnet/pull/18310#issuecomment-628895957 Jenkins CI successfully triggered : [unix-cpu] This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] leezu commented on pull request #18315: Fix coredumps

2020-05-14 Thread GitBox
leezu commented on pull request #18315: URL: https://github.com/apache/incubator-mxnet/pull/18315#issuecomment-628895670 @mxnet-bot run ci [unix-cpu, centos-cpu] This is an automated message from the Apache Git Service. To

[GitHub] [incubator-mxnet] leezu commented on pull request #18310: Fix MKLDNN headers missing in nightly builds

2020-05-14 Thread GitBox
leezu commented on pull request #18310: URL: https://github.com/apache/incubator-mxnet/pull/18310#issuecomment-628895929 @mxnet-bot run ci [unix-cpu] This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-mxnet] leezu commented on pull request #18284: Fix parse operator attributes in new FFI

2020-05-14 Thread GitBox
leezu commented on pull request #18284: URL: https://github.com/apache/incubator-mxnet/pull/18284#issuecomment-628895574 @mxnet-bot run ci [unix-cpu, centos-cpu] This is an automated message from the Apache Git Service. To

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18284: Fix parse operator attributes in new FFI

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18284: URL: https://github.com/apache/incubator-mxnet/pull/18284#issuecomment-628895621 Jenkins CI successfully triggered : [unix-cpu, centos-cpu] This is an automated message from the

[GitHub] [incubator-mxnet] leezu commented on pull request #18323: Fix race condition in unittest by mark as serial tests

2020-05-14 Thread GitBox
leezu commented on pull request #18323: URL: https://github.com/apache/incubator-mxnet/pull/18323#issuecomment-628890535 Would creating the rec-file at a temporary path then solve the issue while preserving parallelization? You can take the `tmp_path` argument in your test function to

[incubator-mxnet] branch master updated: xfail_when_nonstandard_decimal_separator for test_metric.py (#18312)

2020-05-14 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen 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 4d4cbd5

[incubator-mxnet] branch master updated: xfail_when_nonstandard_decimal_separator for test_metric.py (#18312)

2020-05-14 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen 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 4d4cbd5

[GitHub] [incubator-mxnet] leezu merged pull request #18312: xfail_when_nonstandard_decimal_separator for test_metric.py

2020-05-14 Thread GitBox
leezu merged pull request #18312: URL: https://github.com/apache/incubator-mxnet/pull/18312 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [incubator-mxnet] rajkinra23 commented on issue #13288: OpenCV 4.0 Compatibility

2020-05-14 Thread GitBox
rajkinra23 commented on issue #13288: URL: https://github.com/apache/incubator-mxnet/issues/13288#issuecomment-628874551 > I still have linking errors in creating `bin/im2rec` with OpenCV 4.0.1 (MXNet HEAD:

[GitHub] [incubator-mxnet] ArmageddonKnight commented on a change in pull request #18228: [MXNET-1450] Improve the backward mirroring implementation

2020-05-14 Thread GitBox
ArmageddonKnight commented on a change in pull request #18228: URL: https://github.com/apache/incubator-mxnet/pull/18228#discussion_r425317983 ## File path: python/mxnet/rnn/rnn_cell.py ## @@ -459,6 +459,11 @@ def __call__(self, inputs, states):

[GitHub] [incubator-mxnet] ys2843 edited a comment on pull request #18288: Website global search feature

2020-05-14 Thread GitBox
ys2843 edited a comment on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-628867702 > Search feature looks great. > Nice to see we leveraging third-party to solve our search problem. > Q1. This feature is currently only for landing page

[GitHub] [incubator-mxnet] ys2843 commented on pull request #18288: Website global search feature

2020-05-14 Thread GitBox
ys2843 commented on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-628867702 > Search feature looks great. > Nice to see we leveraging third-party to solve our search problem. > Q1. This feature is currently only for landing page right?

[GitHub] [incubator-mxnet] anirudh2290 commented on issue #18314: ThreadedEngine: Segfault

2020-05-14 Thread GitBox
anirudh2290 commented on issue #18314: URL: https://github.com/apache/incubator-mxnet/issues/18314#issuecomment-628845269 > We see that after `del net` is called, two ndarrays are finalized thanks to reference counting. 4 arrays are only finalized after a full garbage collection.

[GitHub] [incubator-mxnet] leezu commented on issue #18314: ThreadedEngine: Segfault

2020-05-14 Thread GitBox
leezu commented on issue #18314: URL: https://github.com/apache/incubator-mxnet/issues/18314#issuecomment-628833322 That's because I set gc.set_debug(gc.DEBUG_SAVEALL). I want to preserve all objects found by gc, because I need to figure out where the reference cycle that prevented them

[GitHub] [incubator-mxnet] zhreshold commented on pull request #18323: Fix race condition in unittest by mark as serial tests

2020-05-14 Thread GitBox
zhreshold commented on pull request #18323: URL: https://github.com/apache/incubator-mxnet/pull/18323#issuecomment-628832894 If multiple tests want to create the rec file at the same time, the rec file may be corrupted This

[GitHub] [incubator-mxnet] anirudh2290 commented on issue #18314: ThreadedEngine: Segfault

2020-05-14 Thread GitBox
anirudh2290 commented on issue #18314: URL: https://github.com/apache/incubator-mxnet/issues/18314#issuecomment-628831199 From https://docs.python.org/3/library/gc.html#gc.garbage , "Changed in version 3.4: Following PEP 442, objects with a __del__() method don’t end up in gc.garbage

[GitHub] [incubator-mxnet] leezu commented on issue #18314: ThreadedEngine: Segfault

2020-05-14 Thread GitBox
leezu commented on issue #18314: URL: https://github.com/apache/incubator-mxnet/issues/18314#issuecomment-628829305 I can confirm that __del__ is called on the object, but it's still kept around in Python afterwards. In practice that may only make problems when debugging memory leaks

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #18324: [OpPerf] Add example of using opperf with internal op locally

2020-05-14 Thread GitBox
ChaiBapchya commented on a change in pull request #18324: URL: https://github.com/apache/incubator-mxnet/pull/18324#discussion_r425364020 ## File path: benchmark/opperf/README.md ## @@ -167,6 +167,41 @@ Output for the above benchmark run, on a CPU machine, would look

[GitHub] [incubator-mxnet] ChaiBapchya edited a comment on pull request #18288: Website global search feature

2020-05-14 Thread GitBox
ChaiBapchya edited a comment on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-628826234 Search feature looks great. Nice to see we leveraging third-party to solve our search problem. Q1. This feature is currently only for landing page

[GitHub] [incubator-mxnet] ChaiBapchya commented on pull request #18288: Website global search feature

2020-05-14 Thread GitBox
ChaiBapchya commented on pull request #18288: URL: https://github.com/apache/incubator-mxnet/pull/18288#issuecomment-628826234 Search feature looks great. Nice to see we leveraging third-party to solve our search problem. Q1. This feature is currently only for landing page right?

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #18324: [OpPerf] Add example of using opperf with internal op locally

2020-05-14 Thread GitBox
access2rohit commented on a change in pull request #18324: URL: https://github.com/apache/incubator-mxnet/pull/18324#discussion_r425359944 ## File path: benchmark/opperf/README.md ## @@ -167,6 +167,41 @@ Output for the above benchmark run, on a CPU machine, would look

[GitHub] [incubator-mxnet] leezu commented on pull request #18323: Fix race condition in unittest by mark as serial tests

2020-05-14 Thread GitBox
leezu commented on pull request #18323: URL: https://github.com/apache/incubator-mxnet/pull/18323#issuecomment-628822907 Could you clarify where the race occurs? This is an automated message from the Apache Git Service. To

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

2020-05-14 Thread aaronmarkham
This is an automated email from the ASF dual-hosted git repository. aaronmarkham 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 fca503b Bump the

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18324: [OpPerf] Add example of using opperf with internal op locally

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18324: URL: https://github.com/apache/incubator-mxnet/pull/18324#issuecomment-628821763 Hey @ChaiBapchya , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the

[GitHub] [incubator-mxnet] ChaiBapchya opened a new pull request #18324: [OpPerf] Add example of using opperf with internal op locally

2020-05-14 Thread GitBox
ChaiBapchya opened a new pull request #18324: URL: https://github.com/apache/incubator-mxnet/pull/18324 ## Description ## Currently opperf supports for mx.nd operators alone. However, locally, one can test internal ops. Documenting it in readme. @access2rohit

[GitHub] [incubator-mxnet] leezu commented on issue #18314: ThreadedEngine: Segfault

2020-05-14 Thread GitBox
leezu commented on issue #18314: URL: https://github.com/apache/incubator-mxnet/issues/18314#issuecomment-628821443 The mutex in above core dump is actually non existant. The problem here is that the ffi call to MXNDArraySyncCopyToCPU passes a handle whose Chunk was cleaned up. So the

[GitHub] [incubator-mxnet] anirudh2290 commented on issue #18314: ThreadedEngine: Segfault

2020-05-14 Thread GitBox
anirudh2290 commented on issue #18314: URL: https://github.com/apache/incubator-mxnet/issues/18314#issuecomment-628821250 not sure completely what it is, but it could be that the one of the objects in garbage is a ndarray and when you call print it calls its __repr__ and asnumpy, the

[GitHub] [incubator-mxnet] zhreshold commented on issue #18322: test_recordimage_dataset

2020-05-14 Thread GitBox
zhreshold commented on issue #18322: URL: https://github.com/apache/incubator-mxnet/issues/18322#issuecomment-628816780 This is very likely caused by parallel test. See https://github.com/apache/incubator-mxnet/pull/18323

[GitHub] [incubator-mxnet] zhreshold opened a new pull request #18323: Fix race condition in unittest by mark as serial tests

2020-05-14 Thread GitBox
zhreshold opened a new pull request #18323: URL: https://github.com/apache/incubator-mxnet/pull/18323 ## Description ## Fix #18322 ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. - [ ] The PR title starts with

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18323: Fix race condition in unittest by mark as serial tests

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18323: URL: https://github.com/apache/incubator-mxnet/pull/18323#issuecomment-628816516 Hey @zhreshold , Thanks for submitting the PR All tests are already queued to run once. If tests fail, you can trigger one or more tests again with the following

[GitHub] [incubator-mxnet] leezu removed a comment on pull request #18313: [Fix][Numpy][WIP] fix mix numpy scalar and MXNet numpy ndarray

2020-05-14 Thread GitBox
leezu removed a comment on pull request #18313: URL: https://github.com/apache/incubator-mxnet/pull/18313#issuecomment-628793435 We can also consider not exposing the "real" inplace operation This is an automated message

[GitHub] [incubator-mxnet] leezu commented on pull request #18313: [Fix][Numpy][WIP] fix mix numpy scalar and MXNet numpy ndarray

2020-05-14 Thread GitBox
leezu commented on pull request #18313: URL: https://github.com/apache/incubator-mxnet/pull/18313#issuecomment-628793435 We can also consider not exposing the "real" inplace operation This is an automated message from the

[GitHub] [incubator-mxnet] ArmageddonKnight commented on a change in pull request #18228: [MXNET-1450] Improve the backward mirroring implementation

2020-05-14 Thread GitBox
ArmageddonKnight commented on a change in pull request #18228: URL: https://github.com/apache/incubator-mxnet/pull/18228#discussion_r425320086 ## File path: docs/static_site/src/pages/api/faq/env_var.md ## @@ -189,7 +189,7 @@ $env:MXNET_STORAGE_FALLBACK_LOG_VERBOSE=0 - The

[GitHub] [incubator-mxnet] ArmageddonKnight commented on a change in pull request #18228: [MXNET-1450] Improve the backward mirroring implementation

2020-05-14 Thread GitBox
ArmageddonKnight commented on a change in pull request #18228: URL: https://github.com/apache/incubator-mxnet/pull/18228#discussion_r425320086 ## File path: docs/static_site/src/pages/api/faq/env_var.md ## @@ -189,7 +189,7 @@ $env:MXNET_STORAGE_FALLBACK_LOG_VERBOSE=0 - The

[GitHub] [incubator-mxnet] ArmageddonKnight commented on a change in pull request #18228: [MXNET-1450] Improve the backward mirroring implementation

2020-05-14 Thread GitBox
ArmageddonKnight commented on a change in pull request #18228: URL: https://github.com/apache/incubator-mxnet/pull/18228#discussion_r425317983 ## File path: python/mxnet/rnn/rnn_cell.py ## @@ -459,6 +459,11 @@ def __call__(self, inputs, states):

[GitHub] [incubator-mxnet] ptrendx commented on issue #18280: [Performance Regression] GPU memory increase for training and inference models

2020-05-14 Thread GitBox
ptrendx commented on issue #18280: URL: https://github.com/apache/incubator-mxnet/issues/18280#issuecomment-628785129 `xxd` is Linux only I believe, so the better approach is actually to just be able to generate the file with `R"delimiter(` and `)delimiter"` around the code, which should

[GitHub] [incubator-mxnet] D-Roberts commented on pull request #18197: [Numpy] Add qr backward part 2 for wide matrices with m < n

2020-05-14 Thread GitBox
D-Roberts commented on pull request #18197: URL: https://github.com/apache/incubator-mxnet/pull/18197#issuecomment-628768250 @mxnet-bot run ci [centos-cpu, unix-cpu, windows-gpu] This is an automated message from the Apache

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18197: [Numpy] Add qr backward part 2 for wide matrices with m < n

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18197: URL: https://github.com/apache/incubator-mxnet/pull/18197#issuecomment-628768324 Jenkins CI successfully triggered : [unix-cpu, windows-gpu, centos-cpu] This is an automated

[GitHub] [incubator-mxnet] rondogency commented on pull request #18309: Revert PR 17767 for fixing GPU memory usage regression (#18283)

2020-05-14 Thread GitBox
rondogency commented on pull request #18309: URL: https://github.com/apache/incubator-mxnet/pull/18309#issuecomment-628762127 @mxnet-bot run ci [unix-gpu] This is an automated message from the Apache Git Service. To respond

  1   2   >