[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18316: [1.7.x] Backport of fix LSTM and GRU layers gradient calculations

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18316: URL: https://github.com/apache/incubator-mxnet/pull/18316#issuecomment-628439726 Hey @bgawrych , 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 c

[GitHub] [incubator-mxnet] bgawrych opened a new pull request #18316: [1.7.x] Backport of fix LSTM and GRU layers gradient calculations

2020-05-14 Thread GitBox
bgawrych opened a new pull request #18316: URL: https://github.com/apache/incubator-mxnet/pull/18316 ## Description ## Fix for LSTM and GRU layers without DNNL enabled give wrong gradients #17898 For bidiractional LSTM with number of layers > 2 input gradient calculation was incor

[GitHub] [incubator-mxnet] yzhliu commented on issue #18297: different dtype when calculating division compared with numpy

2020-05-14 Thread GitBox
yzhliu commented on issue #18297: URL: https://github.com/apache/incubator-mxnet/issues/18297#issuecomment-628443160 @JiangZhaoh will be helping to document. This is an automated message from the Apache Git Service. To respo

[GitHub] [incubator-mxnet] TriLoo edited a comment on issue #18295: /usr/lib64/libc.so.6 cause a mxnet segmentation.

2020-05-14 Thread GitBox
TriLoo edited a comment on issue #18295: URL: https://github.com/apache/incubator-mxnet/issues/18295#issuecomment-628443534 a wrong input caused this error, and this error is fixed now. thank. @leezu This is an automated me

[GitHub] [incubator-mxnet] TriLoo commented on issue #18295: /usr/lib64/libc.so.6 cause a mxnet segmentation.

2020-05-14 Thread GitBox
TriLoo commented on issue #18295: URL: https://github.com/apache/incubator-mxnet/issues/18295#issuecomment-628443534 a wrong input caused this error, and this error is fixed now. thank This is an automated message from the A

[GitHub] [incubator-mxnet] TriLoo closed issue #18295: /usr/lib64/libc.so.6 cause a mxnet segmentation.

2020-05-14 Thread GitBox
TriLoo closed issue #18295: URL: https://github.com/apache/incubator-mxnet/issues/18295 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] hzfan commented on a change in pull request #18197: [Numpy] Add qr backward part 2 for wide matrices with m < n

2020-05-14 Thread GitBox
hzfan commented on a change in pull request #18197: URL: https://github.com/apache/incubator-mxnet/pull/18197#discussion_r424862190 ## File path: src/operator/numpy/linalg/np_qr-inl.h ## @@ -514,15 +548,28 @@ struct qr_backward { template size_t QrBackwardWorkspaceSize(cons

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

2020-05-14 Thread GitBox
hzfan commented on a change in pull request #18197: URL: https://github.com/apache/incubator-mxnet/pull/18197#discussion_r424862190 ## File path: src/operator/numpy/linalg/np_qr-inl.h ## @@ -514,15 +548,28 @@ struct qr_backward { template size_t QrBackwardWorkspaceSize(cons

[GitHub] [incubator-mxnet] bgawrych opened a new pull request #18317: [1.x] Backport of fix LSTM and GRU layers gradient calculations

2020-05-14 Thread GitBox
bgawrych opened a new pull request #18317: URL: https://github.com/apache/incubator-mxnet/pull/18317 ## Description ## Fix for LSTM and GRU layers without DNNL enabled give wrong gradients #17898 For bidiractional LSTM with number of layers > 2 input gradient calculation was incor

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18317: [1.x] Backport of fix LSTM and GRU layers gradient calculations

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18317: URL: https://github.com/apache/incubator-mxnet/pull/18317#issuecomment-628452048 Hey @bgawrych , 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 c

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

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

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

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

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

2020-05-14 Thread GitBox
hgt312 commented on pull request #18313: URL: https://github.com/apache/incubator-mxnet/pull/18313#issuecomment-628473298 @yzhliu @sxjscience Any thoughts on the conversion rules? This is an automated message from the Apache

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

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

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

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

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

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

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

2020-05-14 Thread GitBox
sxjscience commented on pull request #18313: URL: https://github.com/apache/incubator-mxnet/pull/18313#issuecomment-628485821 ```python import jax.numpy as jnp import numpy as onp # jnp += onp a = jnp.ones((10,)) b = onp.ones((10,)) a += b print(type(a)) # onp +=

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

2020-05-14 Thread GitBox
sxjscience commented on pull request #18313: URL: https://github.com/apache/incubator-mxnet/pull/18313#issuecomment-628486640 In Jax, if any ndarray is a Jax ndarray, the result will also be a Jax ndarray. This is an automat

[GitHub] [incubator-mxnet] mseth10 commented on pull request #18083: Changes to mxnet.metric

2020-05-14 Thread GitBox
mseth10 commented on pull request #18083: URL: https://github.com/apache/incubator-mxnet/pull/18083#issuecomment-628494904 @acphile this PR fails nightly CD while running nightly python unit tests. The following tests fail: test_mcc, test_multilabel_f1, test_binary_f1 http://jenkins.

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

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18318: URL: https://github.com/apache/incubator-mxnet/pull/18318#issuecomment-628499438 Hey @mseth10 , 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 co

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

2020-05-14 Thread GitBox
mseth10 opened a new pull request #18318: URL: https://github.com/apache/incubator-mxnet/pull/18318 This reverts commit effbb8b7498e58259bafa6010b8d042bff8d239c. This fixes the nightly CD failures: http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/restricted-mxnet-c

[GitHub] [incubator-mxnet] mseth10 edited a comment on pull request #18083: Changes to mxnet.metric

2020-05-14 Thread GitBox
mseth10 edited a comment on pull request #18083: URL: https://github.com/apache/incubator-mxnet/pull/18083#issuecomment-628494904 @acphile this PR fails nightly CD while running nightly python unit tests. The following tests fail: test_mcc, test_multilabel_f1, test_binary_f1 http://j

[GitHub] [incubator-mxnet] AntiZpvoh opened a new pull request #18319: [numpy] symbolic advanced indexing

2020-05-14 Thread GitBox
AntiZpvoh opened a new pull request #18319: URL: https://github.com/apache/incubator-mxnet/pull/18319 ## Description ## add symbolic advanced indexing for DeepNumPy solve the issue https://github.com/apache/incubator-mxnet/issues/17327 ## Checklist ## ### Essentials ###

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18319: [numpy] symbolic advanced indexing

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18319: URL: https://github.com/apache/incubator-mxnet/pull/18319#issuecomment-628511276 Hey @AntiZpvoh , 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] bhavitvyamalik commented on issue #17923: can not install mxnet on jetson nano, jetpack 4.3, mxnet 1.4

2020-05-14 Thread GitBox
bhavitvyamalik commented on issue #17923: URL: https://github.com/apache/incubator-mxnet/issues/17923#issuecomment-628549077 @JustinhoCHN what was the CUDA version of your Jetson Nano image? This is an automated message from

[GitHub] [incubator-mxnet] bgawrych opened a new pull request #18320: Improve log_softmax op performance by using DNNL support

2020-05-14 Thread GitBox
bgawrych opened a new pull request #18320: URL: https://github.com/apache/incubator-mxnet/pull/18320 ## Description ## Improve log_softmax op performance by using DNNL support **Native implementation:** | shape | time (ms/iter) | |---|---| | (5, 512, 512) | 5.385858

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18320: Improve log_softmax op performance by using DNNL support

2020-05-14 Thread GitBox
mxnet-bot commented on pull request #18320: URL: https://github.com/apache/incubator-mxnet/pull/18320#issuecomment-628561712 Hey @bgawrych , 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 c

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

2020-05-14 Thread GitBox
hgt312 commented on pull request #18313: URL: https://github.com/apache/incubator-mxnet/pull/18313#issuecomment-628565299 Jax does not implement inplace op overload like `__iadd__`. It is not a real inplace operation. In my opinion, inplace operation should keep its type, while the other

[GitHub] [incubator-mxnet] bhavitvyamalik edited a comment on issue #17923: can not install mxnet on jetson nano, jetpack 4.3, mxnet 1.4

2020-05-14 Thread GitBox
bhavitvyamalik edited a comment on issue #17923: URL: https://github.com/apache/incubator-mxnet/issues/17923#issuecomment-628549077 @JustinhoCHN what was the CUDA version of your Jetson Nano image? It gives me the following error: ``` >>> import mxnet Traceback (most recent ca

[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 2674330 Bump the publis

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

2020-05-14 Thread GitBox
TaoLv commented on pull request #18310: URL: https://github.com/apache/incubator-mxnet/pull/18310#issuecomment-628614890 @mxnet-bot run ci [unix-cpu] This is an automated message from the Apache Git Service. To respond to 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-628614964 Jenkins CI successfully triggered : [unix-cpu] This is an automated message from the Apache Git Serv

[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-628621084 > @JustinhoCHN what was the CUDA version of your Jetson Nano image? It gives me the following error: > > ``` > >>> import mxnet > Traceback (most recent ca

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

2020-05-14 Thread GitBox
bhavitvyamalik commented on issue #17923: URL: https://github.com/apache/incubator-mxnet/issues/17923#issuecomment-628629929 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 :/ --

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

2020-05-14 Thread GitBox
D-Roberts commented on a change in pull request #18197: URL: https://github.com/apache/incubator-mxnet/pull/18197#discussion_r425140133 ## File path: src/operator/numpy/linalg/np_qr-inl.h ## @@ -514,15 +548,28 @@ struct qr_backward { template size_t QrBackwardWorkspaceSize(

[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-628641606 The code follows the idea in the reference [Differential Programming Tensor Networks](https://journals.aps.org/prx/pdf/10.1103/PhysRevX.9.031041). At high level, par

[GitHub] [incubator-mxnet] bhavitvyamalik edited a comment on issue #17923: can not install mxnet on jetson nano, jetpack 4.3, mxnet 1.4

2020-05-14 Thread GitBox
bhavitvyamalik edited a comment on issue #17923: URL: https://github.com/apache/incubator-mxnet/issues/17923#issuecomment-628629929 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:

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

2020-05-14 Thread GitBox
sxjscience commented on pull request #18313: URL: https://github.com/apache/incubator-mxnet/pull/18313#issuecomment-628721077 > Jax does not implement inplace op overload like `__iadd__`. It is not a real inplace operation. > In my opinion, inplace operation should keep its type, while t

[GitHub] [incubator-mxnet] alexsisu opened a new issue #18321: Problem when doing batch inference with fpn_resnest101.

2020-05-14 Thread GitBox
alexsisu opened a new issue #18321: URL: https://github.com/apache/incubator-mxnet/issues/18321 ## Description The problem is that im trying to sent a batch of images ( > 1) to the model. But it fails with the following errr ### Error Message ``` input_sym_arg_type = in_pa

[GitHub] [incubator-mxnet] leezu opened a new issue #18322: test_recordimage_dataset

2020-05-14 Thread GitBox
leezu opened a new issue #18322: URL: https://github.com/apache/incubator-mxnet/issues/18322 http://jenkins.mxnet-ci.amazon-ml.com/blue/rest/organizations/jenkins/pipelines/mxnet-validation/pipelines/windows-gpu/branches/PR-18312/runs/2/nodes/109/steps/155/log/?start=0 ``` [2020-0

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

2020-05-14 Thread GitBox
leezu commented on pull request #18318: URL: https://github.com/apache/incubator-mxnet/pull/18318#issuecomment-628745475 We may also simply fix the tests https://github.com/apache/incubator-mxnet/pull/18312 This is an auto

[GitHub] [incubator-mxnet] leezu commented on pull request #18083: Changes to mxnet.metric

2020-05-14 Thread GitBox
leezu commented on pull request #18083: URL: https://github.com/apache/incubator-mxnet/pull/18083#issuecomment-628746204 The reason it fails is that the CI check in this PR was run too long ago. I should have restarted it before merging the PR. Meantime master changed and caused some addit

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

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

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

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

[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 t

[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 message

[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 G

[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] 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] 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 m

[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 m

[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 Apa

[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 from

[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] 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 [MXNET-$JIRA_ID],

[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 T

[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 depe

[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 seg

[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 @sandeep-krishn

[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 followin

[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 publis

[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 re

[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 somethin

[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? It

[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 right

[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 something

[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 (a

[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 anym

[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] 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 f

[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. a

[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] 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 righ

[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] 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: [09ba8be](https://github.com/apache/incubator-mxnet/commit/09ba8be4be36135e2a8d6c4

[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 t

[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 xfail_when_nonstandard_decimal

[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 xfail_when_nonstandard_decimal

[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 have

[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 Apa

[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 re

[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 the

[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 re

[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 Apa

[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 Serv

[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 ab

[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 a

[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 A

[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 a

[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 ab

[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 ab

[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 This

[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 ab

[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 from

[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 Ap

[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 to

[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 co

[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 unaggreg

[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 followin

[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

  1   2   >