[GitHub] [incubator-mxnet] larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'.

2019-11-27 Thread GitBox
larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'. URL: https://github.com/apache/incubator-mxnet/pull/15882#issuecomment-559214353 If we want to monitor binary sizes, a good way to do it would be to put a safeguard on CI that checks t

[GitHub] [incubator-mxnet] larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'.

2019-11-20 Thread GitBox
larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'. URL: https://github.com/apache/incubator-mxnet/pull/15882#issuecomment-556532951 Why is this PR stuck? This is an autom

[GitHub] [incubator-mxnet] larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'.

2019-08-16 Thread GitBox
larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'. URL: https://github.com/apache/incubator-mxnet/pull/15882#issuecomment-522189734 Marco. I understand this is a bug in the vs compiler version we use. I don't think this deprecates 32 b

[GitHub] [incubator-mxnet] larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'.

2019-08-16 Thread GitBox
larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'. URL: https://github.com/apache/incubator-mxnet/pull/15882#issuecomment-522179936 Thanks for the clarification. I already approved. -

[GitHub] [incubator-mxnet] larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'.

2019-08-16 Thread GitBox
larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'. URL: https://github.com/apache/incubator-mxnet/pull/15882#issuecomment-522162803 @DickJC123 I looked at laop tests, one root cause is the multiplication by a "random projection" done i

[GitHub] [incubator-mxnet] larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'.

2019-08-15 Thread GitBox
larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'. URL: https://github.com/apache/incubator-mxnet/pull/15882#issuecomment-521759226 @marcoabreu let's unblock CI and discuss what you propose later. Blocking a critical change to unblock