[GitHub] azai91 commented on issue #13433: Fix deconvolution / PR 13421

2018-11-29 Thread GitBox
azai91 commented on issue #13433: Fix deconvolution / PR 13421 URL: https://github.com/apache/incubator-mxnet/pull/13433#issuecomment-442754672 what do you mean? the test changes don't catch the issue on the master branch which currently has the bug?

[GitHub] azai91 commented on issue #13433: Fix deconvolution / PR 13421

2018-11-28 Thread GitBox
azai91 commented on issue #13433: Fix deconvolution / PR 13421 URL: https://github.com/apache/incubator-mxnet/pull/13433#issuecomment-442576086 the difference is that the unit test does not use filter length that is divisible by 8. When the the input shape has a filter size that is

[GitHub] azai91 commented on issue #13433: Fix deconvolution / PR 13421

2018-11-28 Thread GitBox
azai91 commented on issue #13433: Fix deconvolution / PR 13421 URL: https://github.com/apache/incubator-mxnet/pull/13433#issuecomment-442531158 @TaoLv I think you're referring to this test?

[GitHub] azai91 commented on issue #13433: Fix deconvolution / PR 13421

2018-11-27 Thread GitBox
azai91 commented on issue #13433: Fix deconvolution / PR 13421 URL: https://github.com/apache/incubator-mxnet/pull/13433#issuecomment-442281644 @TaoLv @anirudh2290 This is an automated message from the Apache Git Service.