[GitHub] [incubator-mxnet] ciyongch edited a comment on pull request #18632: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes

2020-07-01 Thread GitBox
ciyongch edited a comment on pull request #18632: URL: https://github.com/apache/incubator-mxnet/pull/18632#issuecomment-652349101 > * this is part of other branches as well? Yes, this is a common issue for all the current branches, I will do the backport to other branches as well.

[GitHub] [incubator-mxnet] ciyongch commented on pull request #18632: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes

2020-07-01 Thread GitBox
ciyongch commented on pull request #18632: URL: https://github.com/apache/incubator-mxnet/pull/18632#issuecomment-652349101 > * this is part of other branches as well? Yes, this is a common issue for all the current branches, I will do the backport to other branches as well. > *

[GitHub] [incubator-mxnet] ciyongch commented on pull request #18648: [v1.7.x] Backport some numpy features + fixes

2020-07-01 Thread GitBox
ciyongch commented on pull request #18648: URL: https://github.com/apache/incubator-mxnet/pull/18648#issuecomment-652355926 As numpy operator feature is in a quite active development in current master branch and mainly targeting on 2.0 release. It'll be fine if the backport patch is small

[GitHub] [incubator-mxnet] wkcn commented on a change in pull request #18644: Fix BatchNorm backward synchronization

2020-07-01 Thread GitBox
wkcn commented on a change in pull request #18644: URL: https://github.com/apache/incubator-mxnet/pull/18644#discussion_r448320053 ## File path: tests/python/unittest/test_gluon.py ## @@ -665,6 +665,34 @@ def transpose(shape): assert (layer(x).shape==ceil_out_shape)

[GitHub] [incubator-mxnet] nicolaerosia opened a new issue #18650: nightly builds for Linux

2020-07-01 Thread GitBox
nicolaerosia opened a new issue #18650: URL: https://github.com/apache/incubator-mxnet/issues/18650 I see that nightly builds for Linux are missing and adding them would help Linux users test before the actual release. https://pypi.org/project/mxnet-mkl/1.7.0b20200315/#files

[GitHub] [incubator-mxnet] ciyongch opened a new pull request #18651: [v1.x]Add constraint to numpy version to <1.19.0 in CI

2020-07-01 Thread GitBox
ciyongch opened a new pull request #18651: URL: https://github.com/apache/incubator-mxnet/pull/18651 ## Description ## As reported in https://github.com/apache/incubator-mxnet/issues/18600, there're some failures related to numpy operators with numpy 1.19.0 (which was released on Jun

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18651: [v1.x]Add constraint to numpy version to <1.19.0 in CI

2020-07-01 Thread GitBox
mxnet-bot commented on pull request #18651: URL: https://github.com/apache/incubator-mxnet/pull/18651#issuecomment-652390653 Hey @ciyongch , 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] ciyongch commented on pull request #18634: [v1.x]Enhance license checker to cover multiple license header and md files

2020-07-01 Thread GitBox
ciyongch commented on pull request #18634: URL: https://github.com/apache/incubator-mxnet/pull/18634#issuecomment-652392677 https://github.com/apache/incubator-mxnet/pull/18651 was created as described in option 1. This is

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18632: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes

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

[GitHub] [incubator-mxnet] ciyongch commented on pull request #18632: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes

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

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18644: Fix BatchNorm backward synchronization

2020-07-01 Thread GitBox
mxnet-bot commented on pull request #18644: URL: https://github.com/apache/incubator-mxnet/pull/18644#issuecomment-652372006 Undefined action detected. Permissible actions are : run ci [all], run ci [job1, job2] Example : @mxnet-bot run ci [all] Example : @mxnet-bot run ci

[GitHub] [incubator-mxnet] anko-intel commented on pull request #18644: Fix BatchNorm backward synchronization

2020-07-01 Thread GitBox
anko-intel commented on pull request #18644: URL: https://github.com/apache/incubator-mxnet/pull/18644#issuecomment-652371982 @mxnet-bot [codecov/project] This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-mxnet] BenjaminCHEN2016 opened a new pull request #18649: backport mixed type to 1.7

2020-07-01 Thread GitBox
BenjaminCHEN2016 opened a new pull request #18649: URL: https://github.com/apache/incubator-mxnet/pull/18649 ## Description ## Backport mixed type binary ops to v1.7.x branch #18641 ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18649: backport mixed type to 1.7

2020-07-01 Thread GitBox
mxnet-bot commented on pull request #18649: URL: https://github.com/apache/incubator-mxnet/pull/18649#issuecomment-652313463 Hey @BenjaminCHEN2016 , 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] anko-intel edited a comment on pull request #18644: Fix BatchNorm backward synchronization

2020-07-01 Thread GitBox
anko-intel edited a comment on pull request #18644: URL: https://github.com/apache/incubator-mxnet/pull/18644#issuecomment-652371982 @mxnet-bot run ci [codecov/project] This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18644: Fix BatchNorm backward synchronization

2020-07-01 Thread GitBox
mxnet-bot commented on pull request #18644: URL: https://github.com/apache/incubator-mxnet/pull/18644#issuecomment-652375755 None of the jobs entered are supported. Jobs entered by user: [codecov/project] CI supported Jobs: [windows-gpu, unix-gpu, edge, sanity, miscellaneous,

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

2020-07-01 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 a9bddc7 Bump the

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

2020-07-01 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 82ec169 Publish

[GitHub] [incubator-mxnet] wkcn commented on issue #18643: ndarray.contrib.boolean_mask can not be hybridize

2020-07-01 Thread GitBox
wkcn commented on issue #18643: URL: https://github.com/apache/incubator-mxnet/issues/18643#issuecomment-652434636 I reproduce the bug too on MXNet 2.0. This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-mxnet] TaoLv commented on issue #18610: BatchNorm running variables are not properly synchronized on CPU

2020-07-01 Thread GitBox
TaoLv commented on issue #18610: URL: https://github.com/apache/incubator-mxnet/issues/18610#issuecomment-652462209 Fixed by #18064 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-mxnet] TaoLv closed issue #18610: BatchNorm running variables are not properly synchronized on CPU

2020-07-01 Thread GitBox
TaoLv closed issue #18610: URL: https://github.com/apache/incubator-mxnet/issues/18610 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] sandeep-krishnamurthy commented on pull request #18648: [v1.7.x] Backport some numpy features + fixes

2020-07-01 Thread GitBox
sandeep-krishnamurthy commented on pull request #18648: URL: https://github.com/apache/incubator-mxnet/pull/18648#issuecomment-652480776 First of all thank you very much @sxjscience for this tremendous effort in backporting PRs. I am really concerned with addition of new

[GitHub] [incubator-mxnet] ptrendx commented on pull request #18622: [WIP] Use RTC for elementwise and broadcast ops

2020-07-01 Thread GitBox
ptrendx commented on pull request #18622: URL: https://github.com/apache/incubator-mxnet/pull/18622#issuecomment-652530647 @mxnet-bot run ci [windows-gpu, unix-gpu, miscellaneous, website, centos-gpu, sanity, clang] This is

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18622: [WIP] Use RTC for elementwise and broadcast ops

2020-07-01 Thread GitBox
mxnet-bot commented on pull request #18622: URL: https://github.com/apache/incubator-mxnet/pull/18622#issuecomment-652530754 Jenkins CI successfully triggered : [unix-gpu, website, miscellaneous, windows-gpu, clang, sanity, centos-gpu]

[GitHub] [incubator-mxnet] TaoLv merged pull request #18644: Fix BatchNorm backward synchronization

2020-07-01 Thread GitBox
TaoLv merged pull request #18644: URL: https://github.com/apache/incubator-mxnet/pull/18644 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

[incubator-mxnet] branch master updated: Fix BatchNorm backward synchronization (#18644)

2020-07-01 Thread taolv
This is an automated email from the ASF dual-hosted git repository. taolv 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 37bed6e Fix BatchNorm backward

[GitHub] [incubator-mxnet] sandeep-krishnamurthy commented on pull request #18632: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes

2020-07-01 Thread GitBox
sandeep-krishnamurthy commented on pull request #18632: URL: https://github.com/apache/incubator-mxnet/pull/18632#issuecomment-652476980 @pengzhao-intel @TaoLv this will be good to go after your review and approval This is

[GitHub] [incubator-mxnet] leezu closed issue #18650: nightly builds for Linux

2020-07-01 Thread GitBox
leezu closed issue #18650: URL: https://github.com/apache/incubator-mxnet/issues/18650 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 #18650: nightly builds for Linux

2020-07-01 Thread GitBox
leezu commented on issue #18650: URL: https://github.com/apache/incubator-mxnet/issues/18650#issuecomment-652538283 Builds from PyPI were removed due to resource limitations of the PyPI project: https://github.com/pypa/pypi-support/issues/50#issuecomment-562811825 Also, tApache

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18642: [numpy] Fix less/greater bug with scalar input

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

[GitHub] [incubator-mxnet] ciyongch commented on pull request #18632: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes

2020-07-01 Thread GitBox
ciyongch commented on pull request #18632: URL: https://github.com/apache/incubator-mxnet/pull/18632#issuecomment-652446228 Codecov failures are still there...which shouldn't be the blocker I think. This is an automated

[GitHub] [incubator-mxnet] TaoLv commented on pull request #18644: Fix BatchNorm backward synchronization

2020-07-01 Thread GitBox
TaoLv commented on pull request #18644: URL: https://github.com/apache/incubator-mxnet/pull/18644#issuecomment-652460275 Thanks for the contribution, @anko-intel. Does the problem also exist on v1.x? If yes, could you please cherry pick the fix to the branch also?

[GitHub] [incubator-mxnet] TaoLv commented on pull request #18602: Fix softmax, logsoftmax failed on empty ndarray

2020-07-01 Thread GitBox
TaoLv commented on pull request #18602: URL: https://github.com/apache/incubator-mxnet/pull/18602#issuecomment-652461346 Thanks for the effort @bgawrych @stu1130. Do we need to cherry pick the fix to v1.x? This is an

[GitHub] [incubator-mxnet] TaoLv closed issue #18569: [Numpy] softmax, logsoftmax failed on empty ndarray

2020-07-01 Thread GitBox
TaoLv closed issue #18569: URL: https://github.com/apache/incubator-mxnet/issues/18569 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] TaoLv merged pull request #18602: Fix softmax, logsoftmax failed on empty ndarray

2020-07-01 Thread GitBox
TaoLv merged pull request #18602: URL: https://github.com/apache/incubator-mxnet/pull/18602 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] pengzhao-intel closed issue #17595: MKLDNN incompatibility with large tensor (dim >= 2^32) data

2020-07-01 Thread GitBox
pengzhao-intel closed issue #17595: URL: https://github.com/apache/incubator-mxnet/issues/17595 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] leezu merged pull request #18651: [v1.x]Add constraint to numpy version to <1.19.0 in CI

2020-07-01 Thread GitBox
leezu merged pull request #18651: URL: https://github.com/apache/incubator-mxnet/pull/18651 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] leezu removed a comment on pull request #18651: [v1.x]Add constraint to numpy version to <1.19.0 in CI

2020-07-01 Thread GitBox
leezu removed a comment on pull request #18651: URL: https://github.com/apache/incubator-mxnet/pull/18651#issuecomment-652535147 I'm quite concerned that there has been no further investigation of the scope and amount of work needed to provide compatibility with numpy 1.19. Constraining

[GitHub] [incubator-mxnet] leezu commented on issue #18652: Can't use mx.np.concatenate with hybridize

2020-07-01 Thread GitBox
leezu commented on issue #18652: URL: https://github.com/apache/incubator-mxnet/issues/18652#issuecomment-652534248 Please use `F.np.concatenate` This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-mxnet] austinmw opened a new issue #18652: Can't use mx.np.concatenate with hybridize

2020-07-01 Thread GitBox
austinmw opened a new issue #18652: URL: https://github.com/apache/incubator-mxnet/issues/18652 ## Description I’m trying to adapt the GoogLeNet/InceptionV1 implementation in the online book d2l.ai to be compatible with hybridization. However, I’m currently facing issues with

[GitHub] [incubator-mxnet] sxjscience commented on pull request #18648: [v1.7.x] Backport some numpy features + fixes

2020-07-01 Thread GitBox
sxjscience commented on pull request #18648: URL: https://github.com/apache/incubator-mxnet/pull/18648#issuecomment-652510821 @sandeep-krishnamurthy @ciyongch I created this PR to track how many numpy commits would potentially go into v1.7.0 (not all of them should go into 1.7.0):

[incubator-mxnet] branch v1.x updated: Add constraint to numpy version to <1.19.0 in CI (#18651)

2020-07-01 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a commit to branch v1.x in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/v1.x by this push: new 16144ff8 Add constraint to numpy version

[incubator-mxnet] branch v1.x updated: Add constraint to numpy version to <1.19.0 in CI (#18651)

2020-07-01 Thread lausen
This is an automated email from the ASF dual-hosted git repository. lausen pushed a commit to branch v1.x in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/v1.x by this push: new 16144ff8 Add constraint to numpy version

[incubator-mxnet] branch master updated: Fix BatchNorm backward synchronization (#18644)

2020-07-01 Thread taolv
This is an automated email from the ASF dual-hosted git repository. taolv 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 37bed6e Fix BatchNorm backward

[GitHub] [incubator-mxnet] sandeep-krishnamurthy commented on pull request #18648: [v1.7.x] Backport some numpy features + fixes

2020-07-01 Thread GitBox
sandeep-krishnamurthy commented on pull request #18648: URL: https://github.com/apache/incubator-mxnet/pull/18648#issuecomment-652532059 > @sandeep-krishnamurthy @ciyongch I created this PR to track how many numpy commits would potentially go into v1.7.0 (not all of them should go into

[GitHub] [incubator-mxnet] sxjscience opened a new pull request #18653: [v1.7.0] Backport PRs

2020-07-01 Thread GitBox
sxjscience opened a new pull request #18653: URL: https://github.com/apache/incubator-mxnet/pull/18653 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18653: [v1.7.0] Backport PRs

2020-07-01 Thread GitBox
mxnet-bot commented on pull request #18653: URL: https://github.com/apache/incubator-mxnet/pull/18653#issuecomment-652588712 Hey @sxjscience , 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] samskalicky commented on a change in pull request #18405: Add deleting of args aux aux to Partition API

2020-07-01 Thread GitBox
samskalicky commented on a change in pull request #18405: URL: https://github.com/apache/incubator-mxnet/pull/18405#discussion_r448576383 ## File path: python/mxnet/gluon/block.py ## @@ -1073,8 +1060,35 @@ def _build_cache(self, *args): for name in

[GitHub] [incubator-mxnet] leezu commented on pull request #18403: Gluon.probability

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

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18403: Gluon.probability

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

[GitHub] [incubator-mxnet] ma-hei removed a comment on pull request #18445: updating ubuntu_cpu base image to 20.04 to observe failing tests regarding Python 3.8

2020-07-01 Thread GitBox
ma-hei removed a comment on pull request #18445: URL: https://github.com/apache/incubator-mxnet/pull/18445#issuecomment-649099499 @Roshrini I found an issue when updating onnx from 1.5.0 to 1.7.0. The issue can be reproduced with python 3.6. The following code reproduces the issue. Do you

[GitHub] [incubator-mxnet] sxjscience commented on pull request #18653: [v1.7.0] Backport PRs

2020-07-01 Thread GitBox
sxjscience commented on pull request #18653: URL: https://github.com/apache/incubator-mxnet/pull/18653#issuecomment-652589402 @ciyongch These three commits can be safely ported. This is an automated message from the Apache

[GitHub] [incubator-mxnet] austinmw edited a comment on issue #18652: Can't use mx.np.concatenate with hybridize

2020-07-01 Thread GitBox
austinmw edited a comment on issue #18652: URL: https://github.com/apache/incubator-mxnet/issues/18652#issuecomment-652615856 @leezu Thanks, that worked! Is `F.np.` the new way to do things going forward? Can't find any documentation for this syntax.

[GitHub] [incubator-mxnet] Kh4L commented on pull request #18350: Add better partial args/aux handling in symbol optimize_for

2020-07-01 Thread GitBox
Kh4L commented on pull request #18350: URL: https://github.com/apache/incubator-mxnet/pull/18350#issuecomment-652577984 @mxnet-bot run ci [centos-cpu] This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18350: Add better partial args/aux handling in symbol optimize_for

2020-07-01 Thread GitBox
mxnet-bot commented on pull request #18350: URL: https://github.com/apache/incubator-mxnet/pull/18350#issuecomment-652578051 Jenkins CI successfully triggered : [centos-cpu] This is an automated message from the Apache Git

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18403: Gluon.probability

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

[GitHub] [incubator-mxnet] szha commented on pull request #18403: Gluon.probability

2020-07-01 Thread GitBox
szha commented on pull request #18403: URL: https://github.com/apache/incubator-mxnet/pull/18403#issuecomment-652651575 @mxnet-bot run ci [unix-gpu] This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-mxnet] ChaiBapchya commented on pull request #18632: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes

2020-07-01 Thread GitBox
ChaiBapchya commented on pull request #18632: URL: https://github.com/apache/incubator-mxnet/pull/18632#issuecomment-652587349 > I will try to add some tests later to cover it. Can we add a basic test to verify this? I guess reviewers would feel confident to approve this once they

[GitHub] [incubator-mxnet] samskalicky commented on a change in pull request #18405: Add deleting of args aux aux to Partition API

2020-07-01 Thread GitBox
samskalicky commented on a change in pull request #18405: URL: https://github.com/apache/incubator-mxnet/pull/18405#discussion_r448571094 ## File path: python/mxnet/gluon/block.py ## @@ -1073,8 +1060,35 @@ def _build_cache(self, *args): for name in

[GitHub] [incubator-mxnet] austinmw closed issue #18652: Can't use mx.np.concatenate with hybridize

2020-07-01 Thread GitBox
austinmw closed issue #18652: URL: https://github.com/apache/incubator-mxnet/issues/18652 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] austinmw edited a comment on issue #18652: Can't use mx.np.concatenate with hybridize

2020-07-01 Thread GitBox
austinmw edited a comment on issue #18652: URL: https://github.com/apache/incubator-mxnet/issues/18652#issuecomment-652615856 @leezu Thanks, that worked! Is `F.np.` the new way to do things going forward? This is an

[GitHub] [incubator-mxnet] austinmw commented on issue #18652: Can't use mx.np.concatenate with hybridize

2020-07-01 Thread GitBox
austinmw commented on issue #18652: URL: https://github.com/apache/incubator-mxnet/issues/18652#issuecomment-652615856 Thanks, that worked! Is `F.np.` the new way to do things going forward? This is an automated message

[GitHub] [incubator-mxnet] leezu commented on pull request #18445: updating ubuntu_cpu base image to 20.04 to observe failing tests regarding Python 3.8

2020-07-01 Thread GitBox
leezu commented on pull request #18445: URL: https://github.com/apache/incubator-mxnet/pull/18445#issuecomment-652702713 @ma-hei I also noticed this error in some unrelated cd job which was building mxnet for python 3.8. Just FYI, and not necessarily something you need to take into

[GitHub] [incubator-mxnet] leezu commented on a change in pull request #18405: Add deleting of args aux aux to Partition API

2020-07-01 Thread GitBox
leezu commented on a change in pull request #18405: URL: https://github.com/apache/incubator-mxnet/pull/18405#discussion_r448685873 ## File path: python/mxnet/gluon/block.py ## @@ -1335,12 +1337,21 @@ def export(self, path, epoch=0, remove_amp_cast=True): arg_names =

[GitHub] [incubator-mxnet] ChaiBapchya commented on a change in pull request #18625: [DO NOT MERGE] Enable Large Tensor Support : Stage1

2020-07-01 Thread GitBox
ChaiBapchya commented on a change in pull request #18625: URL: https://github.com/apache/incubator-mxnet/pull/18625#discussion_r448685082 ## File path: ci/docker/runtime_functions.sh ## @@ -411,8 +413,9 @@ build_ubuntu_cpu_mkl() { -DUSE_MKLDNN=OFF \

[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #18625: [DO NOT MERGE] Enable Large Tensor Support : Stage1

2020-07-01 Thread GitBox
access2rohit commented on a change in pull request #18625: URL: https://github.com/apache/incubator-mxnet/pull/18625#discussion_r448685815 ## File path: src/operator/numpy/np_polynomial_op.cc ## @@ -53,7 +53,8 @@ struct polyval_backward_p { DType igrad_p = 0; index_t

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

2020-07-01 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 cbfe543 Bump the

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18634: [v1.x]Enhance license checker to cover multiple license header and md files

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

[GitHub] [incubator-mxnet] ciyongch commented on pull request #18634: [v1.x]Enhance license checker to cover multiple license header and md files

2020-07-01 Thread GitBox
ciyongch commented on pull request #18634: URL: https://github.com/apache/incubator-mxnet/pull/18634#issuecomment-652731585 @mxnet-bot run ci [unix-gpu] This is an automated message from the Apache Git Service. To

[GitHub] [incubator-mxnet] ciyongch commented on pull request #18653: [v1.7.0] Backport PRs

2020-07-01 Thread GitBox
ciyongch commented on pull request #18653: URL: https://github.com/apache/incubator-mxnet/pull/18653#issuecomment-652737478 Thanks @sxjscience , so only two remaining PRs (#18250 and #18523) to be backported, right? BTW, please help to re-trigger the [unix-gpu] job.

[GitHub] [incubator-mxnet] leezu commented on pull request #18403: Gluon.probability

2020-07-01 Thread GitBox
leezu commented on pull request #18403: URL: https://github.com/apache/incubator-mxnet/pull/18403#issuecomment-652701478 @xidulu any idea why in your PR the `test_np_mixedType_unary_funcs[float16-3-rint-None--5.0-5.0]` test persistently (8 CI runs in a row) is found to leak ndarray

[GitHub] [incubator-mxnet] ciyongch commented on pull request #18632: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes

2020-07-01 Thread GitBox
ciyongch commented on pull request #18632: URL: https://github.com/apache/incubator-mxnet/pull/18632#issuecomment-652706565 @ChaiBapchya We've verified the fix via an offline customized cases, anyway, it's quite reasonable to add a UT to cover this case. I will try to add this today.

[GitHub] [incubator-mxnet] leezu commented on a change in pull request #18625: [DO NOT MERGE] Enable Large Tensor Support : Stage1

2020-07-01 Thread GitBox
leezu commented on a change in pull request #18625: URL: https://github.com/apache/incubator-mxnet/pull/18625#discussion_r448684679 ## File path: ci/build_windows.py ## @@ -69,6 +69,7 @@ class BuildFlavour(Enum): '-DUSE_DIST_KVSTORE=OFF '

[GitHub] [incubator-mxnet] ciyongch commented on pull request #18585: [v1.7] Cherry pick Increase staggered build timeout (#18568)

2020-07-01 Thread GitBox
ciyongch commented on pull request #18585: URL: https://github.com/apache/incubator-mxnet/pull/18585#issuecomment-652738836 Hi @leeze @ChaiBapchya @marcoabreu @aaronmarkham , can you help to take a review and merged. The corresponding one

[GitHub] [incubator-mxnet] TaoLv commented on pull request #18653: [v1.7.0] Backport PRs

2020-07-01 Thread GitBox
TaoLv commented on pull request #18653: URL: https://github.com/apache/incubator-mxnet/pull/18653#issuecomment-652739282 @mxnet-bot run ci [unix-gpu] This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-mxnet] mxnet-bot commented on pull request #18653: [v1.7.0] Backport PRs

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

[GitHub] [incubator-mxnet] ciyongch commented on pull request #18648: [v1.7.x] Backport some numpy features + fixes

2020-07-01 Thread GitBox
ciyongch commented on pull request #18648: URL: https://github.com/apache/incubator-mxnet/pull/18648#issuecomment-652736178 Thanks a lot @sxjscience for the great help to filter the necessary PRs targeting at 1.7 release and @sandeep-krishnamurthy for the valuable comments :) When

[GitHub] [incubator-mxnet] leezu commented on pull request #18632: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes

2020-07-01 Thread GitBox
leezu commented on pull request #18632: URL: https://github.com/apache/incubator-mxnet/pull/18632#issuecomment-652697966 To "fix" the codecov showing up on the 1.x branches, you can include the 3 lines from https://github.com/apache/incubator-mxnet/pull/18497/files cc

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

2020-07-01 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 7e2f5cc Publish

[GitHub] [incubator-mxnet] xidulu commented on pull request #18403: Gluon.probability

2020-07-01 Thread GitBox
xidulu commented on pull request #18403: URL: https://github.com/apache/incubator-mxnet/pull/18403#issuecomment-652753148 @leezu I am looking into this issue. This is an automated message from the Apache Git Service. To

[GitHub] [incubator-mxnet] hzfan commented on a change in pull request #18614: [numpy] FFI flip, rollaxis, stack

2020-07-01 Thread GitBox
hzfan commented on a change in pull request #18614: URL: https://github.com/apache/incubator-mxnet/pull/18614#discussion_r448133071 ## File path: src/api/operator/numpy/np_matrix_op.cc ## @@ -74,6 +74,68 @@ MXNET_REGISTER_API("_npi.expand_dims") *ret = ndoutputs[0]; });

[GitHub] [incubator-mxnet] ChaiBapchya commented on pull request #18632: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes

2020-07-01 Thread GitBox
ChaiBapchya commented on pull request #18632: URL: https://github.com/apache/incubator-mxnet/pull/18632#issuecomment-652220819 It passed all 11 why did we have to retrigger? Is codecov blocking merge? Also we should try to use mxnet-bot for re-triggering specific pipelines if any.

[GitHub] [incubator-mxnet] ChaiBapchya commented on pull request #18632: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes

2020-07-01 Thread GitBox
ChaiBapchya commented on pull request #18632: URL: https://github.com/apache/incubator-mxnet/pull/18632#issuecomment-652227203 I don't think that's the case. @szha @leezu can confirm if code-cov is a blocker. If its not a blocker, lets get this PR merged. Also I'm guessing since

[GitHub] [incubator-mxnet] anko-intel commented on a change in pull request #18644: Fix BatchNorm backward synchronization

2020-07-01 Thread GitBox
anko-intel commented on a change in pull request #18644: URL: https://github.com/apache/incubator-mxnet/pull/18644#discussion_r448157104 ## File path: tests/python/unittest/test_gluon.py ## @@ -665,6 +665,34 @@ def transpose(shape): assert

[GitHub] [incubator-mxnet] wkcn commented on pull request #18504: [Improvement] Invoke mkldnn and cudnn BatchNorm when axis != 1

2020-07-01 Thread GitBox
wkcn commented on pull request #18504: URL: https://github.com/apache/incubator-mxnet/pull/18504#issuecomment-652212670 > @wkcn, do you have any performance number? Sorry that I do not have any performance table. I have only a laptop computer with i7-7500u 2C4T. Could you help me

[GitHub] [incubator-mxnet] hzfan commented on a change in pull request #18614: [numpy] FFI flip, rollaxis, stack

2020-07-01 Thread GitBox
hzfan commented on a change in pull request #18614: URL: https://github.com/apache/incubator-mxnet/pull/18614#discussion_r448136696 ## File path: src/api/operator/numpy/np_matrix_op.cc ## @@ -74,6 +74,68 @@ MXNET_REGISTER_API("_npi.expand_dims") *ret = ndoutputs[0]; });

[GitHub] [incubator-mxnet] sxjscience commented on pull request #17645: [CI] Test CMake builds instead of Makefile builds

2020-07-01 Thread GitBox
sxjscience commented on pull request #17645: URL: https://github.com/apache/incubator-mxnet/pull/17645#issuecomment-652216359 @ciyongch Would this go to v1.7? I come up to this commit when I'm cherry-picking the related commits one by one

[GitHub] [incubator-mxnet] szha commented on pull request #18634: [v1.x]Enhance license checker to cover multiple license header and md files

2020-07-01 Thread GitBox
szha commented on pull request #18634: URL: https://github.com/apache/incubator-mxnet/pull/18634#issuecomment-652216592 +1 for option 1 since the integration indeed needs fix. Thanks for driving this. This is an automated

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

2020-07-01 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 f3f4a51 Publish

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

2020-07-01 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 27d2838 Bump the

[GitHub] [incubator-mxnet] ciyongch commented on pull request #18632: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes

2020-07-01 Thread GitBox
ciyongch commented on pull request #18632: URL: https://github.com/apache/incubator-mxnet/pull/18632#issuecomment-652226348 Hi @ChaiBapchya , I saw the codecov test cases failed and the mxnet-bot doesn't support re-trigger. Not sure if they're a merge blocker or not, I just re-trigger the

[GitHub] [incubator-mxnet] sxjscience commented on pull request #18648: [v1.7.x] [Backport]add zero grad for npi_unique (#18080)

2020-07-01 Thread GitBox
sxjscience commented on pull request #18648: URL: https://github.com/apache/incubator-mxnet/pull/18648#issuecomment-652235614 @ciyongch I traced the commits that are before 743bbcb and these are the initial list of commits. There are two remaining commits like: -

[GitHub] [incubator-mxnet] sandeep-krishnamurthy commented on pull request #18648: [v1.7.x] Backport some numpy features + fixes

2020-07-01 Thread GitBox
sandeep-krishnamurthy commented on pull request #18648: URL: https://github.com/apache/incubator-mxnet/pull/18648#issuecomment-652767947 @sxjscience when will it be possible to have a Backport PRs? @ciyongch what is the timeline you had planned for rc0?

[GitHub] [incubator-mxnet] ciyongch commented on pull request #18648: [v1.7.x] Backport some numpy features + fixes

2020-07-01 Thread GitBox
ciyongch commented on pull request #18648: URL: https://github.com/apache/incubator-mxnet/pull/18648#issuecomment-652773680 Hi @sandeep-krishnamurthy , actually I was planning to drop rc0 before numpy related issues were raised. @sxjscience has already submitted partial PRs listed as

[GitHub] [incubator-mxnet] ciyongch commented on pull request #18632: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes

2020-07-01 Thread GitBox
ciyongch commented on pull request #18632: URL: https://github.com/apache/incubator-mxnet/pull/18632#issuecomment-652777978 Hi @ChaiBapchya @leezu @pengzhao-intel @TaoLv , now all the CI passed and the UT is added as well, please help to merge, thanks.

[incubator-mxnet] branch v1.6.x updated: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes (#18632)

2020-07-01 Thread taolv
This is an automated email from the ASF dual-hosted git repository. taolv pushed a commit to branch v1.6.x in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/v1.6.x by this push: new e503704 [v1.6] Fix the

[incubator-mxnet] branch v1.6.x updated: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes (#18632)

2020-07-01 Thread taolv
This is an automated email from the ASF dual-hosted git repository. taolv pushed a commit to branch v1.6.x in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/v1.6.x by this push: new e503704 [v1.6] Fix the

[incubator-mxnet] branch v1.6.x updated: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes (#18632)

2020-07-01 Thread taolv
This is an automated email from the ASF dual-hosted git repository. taolv pushed a commit to branch v1.6.x in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git The following commit(s) were added to refs/heads/v1.6.x by this push: new e503704 [v1.6] Fix the

[GitHub] [incubator-mxnet] yzhliu commented on a change in pull request #18642: [numpy] Fix less/greater bug with scalar input

2020-07-01 Thread GitBox
yzhliu commented on a change in pull request #18642: URL: https://github.com/apache/incubator-mxnet/pull/18642#discussion_r448755291 ## File path: src/api/operator/numpy/np_elemwise_broadcast_logic_op.cc ## @@ -44,31 +44,65 @@ MXNET_REGISTER_API("_npi.not_equal")

[GitHub] [incubator-mxnet] ciyongch commented on pull request #18648: [v1.7.x] Backport some numpy features + fixes

2020-07-01 Thread GitBox
ciyongch commented on pull request #18648: URL: https://github.com/apache/incubator-mxnet/pull/18648#issuecomment-652792263 Sure, that one should be fine, how about the rest of two PRs? May I know how much effort and time you'll need to backport them? Thanks!

  1   2   >