[GitHub] benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2018-02-02 Thread GitBox
benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-362581380 @yzhliu Can you check and merge if ok? This i

[GitHub] benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2018-01-31 Thread GitBox
benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-362180888 rebased again This is an automated message fr

[GitHub] benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2018-01-25 Thread GitBox
benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-360445475 checks have passed! @yzhliu can you have a look please? ---

[GitHub] benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2018-01-24 Thread GitBox
benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-360368130 Done. This is an automated message from the

[GitHub] benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2018-01-24 Thread GitBox
benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-360282441 I did rebase. I wonder if just changing to ```scala val predLabel = if (pred.shape == label.

[GitHub] benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2018-01-24 Thread GitBox
benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-360257217 Hi, Sorry for the long delay. The problem is that while this change fix my problem on segmentation tasks,

[GitHub] benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2017-11-21 Thread GitBox
benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-345952216 ok, thanks. will check later how to fix this for the classification test.

[GitHub] benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2017-11-02 Thread GitBox
benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-341370830 I tried again, still no luck. The message from the CI doesn't help to understand if it is still a CI issue

[GitHub] benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2017-10-26 Thread GitBox
benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-339870110 Rebased. Waiting for the CI to complete. This

[GitHub] benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention

2017-10-16 Thread git
benqua commented on issue #8245: Use argmax instead of argmax_channel in Accuracy to keep dimention URL: https://github.com/apache/incubator-mxnet/pull/8245#issuecomment-336993852 I don't see why the test fails but don't think this is due to my proposed change. @Ldpe2G any idea? -