[GitHub] [incubator-mxnet] leezu commented on pull request #18749: Refactor Gluon parameter serialization format

2020-07-18 Thread GitBox
leezu commented on pull request #18749: URL: https://github.com/apache/incubator-mxnet/pull/18749#issuecomment-660439938 I think we can delete `mx.npx.load` (or make it private to have a transition period) This is an automat

[GitHub] [incubator-mxnet] leezu edited a comment on pull request #18749: Refactor Gluon parameter serialization format

2020-07-18 Thread GitBox
leezu edited a comment on pull request #18749: URL: https://github.com/apache/incubator-mxnet/pull/18749#issuecomment-660439938 I think we can delete `mx.npx.load` (or make it private to have a transition period). This PR marks it as deprecated. Do you see any problems? --

[GitHub] [incubator-mxnet] Dragas commented on issue #17913: ModuleNotFoundError: No module named 'mxnet.contrib.amp'

2020-07-18 Thread GitBox
Dragas commented on issue #17913: URL: https://github.com/apache/incubator-mxnet/issues/17913#issuecomment-660444070 Seems to be an issue when running an `mxnet/python:1.4.0_cpu_py3` docker image. This is an automated messa

[GitHub] [incubator-mxnet] davidhewitt opened a new issue #18754: Python 3.8 wheel appears to be slightly corrupted

2020-07-18 Thread GitBox
davidhewitt opened a new issue #18754: URL: https://github.com/apache/incubator-mxnet/issues/18754 ## Description I'm attempting to debug a crash observed by a user of PyO3 (https://github.com/PyO3/pyo3/issues/1044) which occurs when `mxnet` is imported. Attempting to use `gdb` (

[GitHub] [incubator-mxnet] gilbertfrancois commented on issue #18751: gluon.nn.BatchNorm seems to swap updated values of moving_mean and moving_var on GPU.

2020-07-18 Thread GitBox
gilbertfrancois commented on issue #18751: URL: https://github.com/apache/incubator-mxnet/issues/18751#issuecomment-660447079 @szha I've just tested it against **mxnet-cu102 v2.0.0b20200716** and it has the same problem. See below: ``` gamma on CPU and GPU are (alm

[incubator-mxnet-site] branch asf-site updated: Publish triggered by CI

2020-07-18 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 0cb0f3a Publish trigger

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

2020-07-18 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 b252ad2 Bump the publis

[GitHub] [incubator-mxnet] ptrendx commented on issue #17913: ModuleNotFoundError: No module named 'mxnet.contrib.amp'

2020-07-18 Thread GitBox
ptrendx commented on issue #17913: URL: https://github.com/apache/incubator-mxnet/issues/17913#issuecomment-660498934 AMP was introduced in MXNet 1.5. Could you try that version (or newer)? This is an automated message from

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

2020-07-18 Thread GitBox
D-Roberts commented on pull request #18197: URL: https://github.com/apache/incubator-mxnet/pull/18197#issuecomment-660501841 @hzfan Thank you for your prompt assistance, I appreciate it. @leezu @szha @DickJC123 I will resubmit a separate PR. For my future reference - what are your re

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

2020-07-18 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 71c1a62 Bump the publis

[incubator-mxnet-site] branch asf-site updated: Publish triggered by CI

2020-07-18 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 8694042 Publish trigger

[GitHub] [incubator-mxnet] DickJC123 opened a new issue #18755: test_gluon_probability_v2.py::test_gluon_kl and test_gluon_probability_v1.py::test_gluon_kl_v1 are flaky

2020-07-18 Thread GitBox
DickJC123 opened a new issue #18755: URL: https://github.com/apache/incubator-mxnet/issues/18755 ## Description I encountered a failure in one of my CI runs, and have supplied a fix in PR https://github.com/apache/incubator-mxnet/pull/18694 in commit https://github.com/apache/incubator-

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18608: Cherry-pick #18310 #18355

2020-07-18 Thread GitBox
mxnet-bot commented on pull request #18608: URL: https://github.com/apache/incubator-mxnet/pull/18608#issuecomment-660535624 Jenkins CI successfully triggered : [edge, unix-gpu] This is an automated message from the Apache Gi

[GitHub] [incubator-mxnet] MoisesHer commented on pull request #18608: Cherry-pick #18310 #18355

2020-07-18 Thread GitBox
MoisesHer commented on pull request #18608: URL: https://github.com/apache/incubator-mxnet/pull/18608#issuecomment-660535605 @mxnet-bot run ci [edge, unix-gpu] This is an automated message from the Apache Git Service. To resp

[GitHub] [incubator-mxnet] ChaiBapchya commented on pull request #18608: Cherry-pick #18310 #18355

2020-07-18 Thread GitBox
ChaiBapchya commented on pull request #18608: URL: https://github.com/apache/incubator-mxnet/pull/18608#issuecomment-660554703 @MoisesHer no point in retriggering edge pipeline. It's not a flaky issue. I saw the same issue in my cherrypick PR: https://github.com/apache/incubator-mxne

[incubator-mxnet-site] branch asf-site updated: Publish triggered by CI

2020-07-18 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 43c16e1 Publish trigger

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

2020-07-18 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 03bc4e1 Bump the publis

[GitHub] [incubator-mxnet] ChaiBapchya commented on pull request #18608: Cherry-pick #18310 #18355

2020-07-18 Thread GitBox
ChaiBapchya commented on pull request #18608: URL: https://github.com/apache/incubator-mxnet/pull/18608#issuecomment-660573558 @sandeep-krishnamurthy @leezu @ciyongch This one fixes MKLDNN missing headers. Please help review/merge. -

[GitHub] [incubator-mxnet] xidulu commented on issue #18755: test_gluon_probability_v2.py::test_gluon_kl and test_gluon_probability_v1.py::test_gluon_kl_v1 are flaky

2020-07-18 Thread GitBox
xidulu commented on issue #18755: URL: https://github.com/apache/incubator-mxnet/issues/18755#issuecomment-660574101 @DickJC123 Thanks for discovering this issue, I will take a look at your fix. This is an automated messa

[GitHub] [incubator-mxnet] szha commented on pull request #18694: Unittest tolerance handling improvements

2020-07-18 Thread GitBox
szha commented on pull request #18694: URL: https://github.com/apache/incubator-mxnet/pull/18694#issuecomment-660577521 @DickJC123 thanks for the change and for fixing the issues you found along the way. Remember to mark the issues this PR resolve in the description. -

[GitHub] [incubator-mxnet] szha commented on pull request #18749: Refactor Gluon parameter serialization format

2020-07-18 Thread GitBox
szha commented on pull request #18749: URL: https://github.com/apache/incubator-mxnet/pull/18749#issuecomment-660577794 > their use is not yet defined Frontend for JVM is planned in #17783. We will at least need to support inference with backend API which will require loading the par

[GitHub] [incubator-mxnet] DickJC123 opened a new issue #18756: pytest worker crash seen on newly introduced unittest test_profiler_gpu.py::test_aggregate_duplication

2020-07-18 Thread GitBox
DickJC123 opened a new issue #18756: URL: https://github.com/apache/incubator-mxnet/issues/18756 ## Description tests/python/gpu/test_profiler_gpu.py has recently started importing unittests/test_profiler.py, and so now runs those tests for the first time with a gpu default context. Wi

[GitHub] [incubator-mxnet] TristonC commented on issue #18751: gluon.nn.BatchNorm seems to swap updated values of moving_mean and moving_var on GPU.

2020-07-18 Thread GitBox
TristonC commented on issue #18751: URL: https://github.com/apache/incubator-mxnet/issues/18751#issuecomment-660583673 The CPU result seems wrong, while the GPU result seems more resonable. This is an automated message from

[GitHub] [incubator-mxnet] TristonC edited a comment on issue #18751: gluon.nn.BatchNorm seems to swap updated values of moving_mean and moving_var on GPU.

2020-07-18 Thread GitBox
TristonC edited a comment on issue #18751: URL: https://github.com/apache/incubator-mxnet/issues/18751#issuecomment-660583673 Great repro script @gilbertfrancois. The CPU result seems wrong, while the GPU result seems reasonable.

[GitHub] [incubator-mxnet] TristonC edited a comment on issue #18751: gluon.nn.BatchNorm seems to swap updated values of moving_mean and moving_var on GPU.

2020-07-18 Thread GitBox
TristonC edited a comment on issue #18751: URL: https://github.com/apache/incubator-mxnet/issues/18751#issuecomment-660583673 Great repro script @gilbertfrancois. The CPU result seems wrong, while the GPU result seems reasonable. On your plot, the GPU running mean actually goes to 1 an

[GitHub] [incubator-mxnet] TristonC edited a comment on issue #18751: gluon.nn.BatchNorm seems to swap updated values of moving_mean and moving_var on GPU.

2020-07-18 Thread GitBox
TristonC edited a comment on issue #18751: URL: https://github.com/apache/incubator-mxnet/issues/18751#issuecomment-660583673 Great repro script @gilbertfrancois. The CPU result seems wrong, while the GPU result seems reasonable. On your plot, the GPU running mean actually goes to 1 an

[GitHub] [incubator-mxnet] TristonC removed a comment on issue #18751: gluon.nn.BatchNorm seems to swap updated values of moving_mean and moving_var on GPU.

2020-07-18 Thread GitBox
TristonC removed a comment on issue #18751: URL: https://github.com/apache/incubator-mxnet/issues/18751#issuecomment-660586280 On GPU, the first running mean is 0, while the following 3 running means are 0.1, 0.19 and 0.271, which can be explained as running_mean = 0.1 * running_mean +

[GitHub] [incubator-mxnet] TristonC commented on issue #18751: gluon.nn.BatchNorm seems to swap updated values of moving_mean and moving_var on GPU.

2020-07-18 Thread GitBox
TristonC commented on issue #18751: URL: https://github.com/apache/incubator-mxnet/issues/18751#issuecomment-660586280 On GPU, the first running mean is 0, while the following 3 running means are 0.1, 0.19 and 0.271, which can be explained as running_mean = 0.1 * running_mean + 0.9 * p

[GitHub] [incubator-mxnet] TristonC edited a comment on issue #18751: gluon.nn.BatchNorm seems to swap updated values of moving_mean and moving_var on GPU.

2020-07-18 Thread GitBox
TristonC edited a comment on issue #18751: URL: https://github.com/apache/incubator-mxnet/issues/18751#issuecomment-660583673 Great repro script @gilbertfrancois. The CPU result seems wrong, while the GPU result seems reasonable. On your plot, the GPU running mean actually goes to 1 an

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

2020-07-18 Thread GitBox
szha commented on pull request #18197: URL: https://github.com/apache/incubator-mxnet/pull/18197#issuecomment-660592664 @D-Roberts we will likely need to automate it so that stale CI checks are invalidated. In the meantime, if the PR sits for a long time, feel free to ping me or any other

[GitHub] [incubator-mxnet] ciyongch commented on pull request #18608: Cherry-pick #18310 #18355

2020-07-18 Thread GitBox
ciyongch commented on pull request #18608: URL: https://github.com/apache/incubator-mxnet/pull/18608#issuecomment-660595392 Hi @ChaiBapchya @MoisesHer please check the failed job. BTW, as it's for the binary release, then I think it's not a mandatory patch for the source release (the curre

[incubator-mxnet-site] branch asf-site updated: Publish triggered by CI

2020-07-18 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 994d956 Publish trigger

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

2020-07-18 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 53d20fc Bump the publis