[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-652796279 @TaoLv helped to merge that PR, please help to check if the current code base works expected or not :)

[incubator-mxnet] branch v1.7.x updated: [v1.7.x] Backport PRs of numpy features (#18653)

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

[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-652696894 @austinmw it's the correct syntax for the experimental numpy support in MXNet 1.x. Sorry for the unclear documentation. With the upcoming MXNet 2 release, users can

[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_r448670196 ## File path: python/mxnet/gluon/block.py ## @@ -1073,8 +1060,35 @@ def _build_cache(self, *args): for name in

[GitHub] [incubator-mxnet] cloudhan commented on issue #18509: fail to mxnet to onnx

2020-07-01 Thread GitBox
cloudhan commented on issue #18509: URL: https://github.com/apache/incubator-mxnet/issues/18509#issuecomment-652771931 You can monkey patch the conversion script with ```python from mxnet.contrib import onnx as onnx_mxnet from mxnet.contrib.onnx.mx2onnx.export_onnx import

[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-652777589 Hi @leeze, @szha please help to review and merge. This is an automated message from the Apache Git

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

2020-07-01 Thread GitBox
Kh4L commented on a change in pull request #18405: URL: https://github.com/apache/incubator-mxnet/pull/18405#discussion_r448752115 ## 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] TaoLv merged pull request #18632: [v1.6] Fix the monitor_callback invalid issue during calibration with variable input shapes

2020-07-01 Thread GitBox
TaoLv merged pull request #18632: URL: https://github.com/apache/incubator-mxnet/pull/18632 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 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] 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-652791542 Hi @TaoLv @pengzhao-intel , please help to merge, thanks. This is an automated message from the

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

2020-07-01 Thread GitBox
sxjscience closed pull request #18648: URL: https://github.com/apache/incubator-mxnet/pull/18648 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] 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-652784885 Close this PR now This is an automated message from the Apache Git Service. To respond to the

[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-652784828 @ciyongch I'm going to test v1.7 after this backporting PR (https://github.com/apache/incubator-mxnet/pull/18653) is merged. WOuld you merge that one?

[GitHub] [incubator-mxnet] TaoLv merged pull request #18653: [v1.7.x] Backport PRs

2020-07-01 Thread GitBox
TaoLv merged 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 URL above

[incubator-mxnet] branch v1.7.x updated: [v1.7.x] Backport PRs of numpy features (#18653)

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

[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-652267739 @ciyongch I find that there are lots of numpy stuffs here. This is the initial attempt for backporting some commits. One issue is #17645, in which I'm not sure

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

2020-07-01 Thread GitBox
sxjscience edited a comment 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] sxjscience commented on issue #18641: Backporting recent mx.np changes to 1.7 branch

2020-07-01 Thread GitBox
sxjscience commented on issue #18641: URL: https://github.com/apache/incubator-mxnet/issues/18641#issuecomment-652270700 Attempted to backport some changes in https://github.com/apache/incubator-mxnet/pull/18648. However, there are lots of changes in numpy. Get Outlook for

[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-652272413 Thanks @sxjscience for the effort. It seems that this backport introduce **huge code changes** which will be a big concern to the current stable code base

[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-652295990 In addition, we do have the ability to test whether the numpy APIs are stable or not by checking the tests in GluonNLP:

<    1   2