[GitHub] [incubator-mxnet] wkcn commented on issue #14321: moveaxis operator now accepts negative indices and sequence of ints as well.

2019-03-15 Thread GitBox
wkcn commented on issue #14321: moveaxis operator now accepts negative indices and sequence of ints as well. URL: https://github.com/apache/incubator-mxnet/pull/14321#issuecomment-473278277 Great! Glad to merge it. Thanks for your contribution! --

[GitHub] [incubator-mxnet] wkcn commented on issue #14321: moveaxis operator now accepts negative indices and sequence of ints as well.

2019-03-15 Thread GitBox
wkcn commented on issue #14321: moveaxis operator now accepts negative indices and sequence of ints as well. URL: https://github.com/apache/incubator-mxnet/pull/14321#issuecomment-473196103 Current MXNet Does not support zero size tensor, so I modified the testcase. ---

[GitHub] [incubator-mxnet] wkcn commented on issue #14321: moveaxis operator now accepts negative indices and sequence of ints as well.

2019-03-14 Thread GitBox
wkcn commented on issue #14321: moveaxis operator now accepts negative indices and sequence of ints as well. URL: https://github.com/apache/incubator-mxnet/pull/14321#issuecomment-473127615 I guess assert_exception has some problems.

[GitHub] [incubator-mxnet] wkcn commented on issue #14321: moveaxis operator now accepts negative indices and sequence of ints as well.

2019-03-14 Thread GitBox
wkcn commented on issue #14321: moveaxis operator now accepts negative indices and sequence of ints as well. URL: https://github.com/apache/incubator-mxnet/pull/14321#issuecomment-473116516 Could we disable the new testcase to track the error? --

[GitHub] [incubator-mxnet] wkcn commented on issue #14321: moveaxis operator now accepts negative indices and sequence of ints as well.

2019-03-11 Thread GitBox
wkcn commented on issue #14321: moveaxis operator now accepts negative indices and sequence of ints as well. URL: https://github.com/apache/incubator-mxnet/pull/14321#issuecomment-471440764 I do not know why this CI always fail. -

[GitHub] [incubator-mxnet] wkcn commented on issue #14321: moveaxis operator now accepts negative indices and sequence of ints as well.

2019-03-07 Thread GitBox
wkcn commented on issue #14321: moveaxis operator now accepts negative indices and sequence of ints as well. URL: https://github.com/apache/incubator-mxnet/pull/14321#issuecomment-470744294 Sorry that I wrote a typo, which should be retrigger. --