[GitHub] [incubator-mxnet] larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'.
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 that separately. 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 to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [incubator-mxnet] larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'.
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 automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [incubator-mxnet] larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'.
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 bit support as the target arch doesn't change. We are compiling for 32 bit in arm7 right? 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 to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [incubator-mxnet] larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'.
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. 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 to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [incubator-mxnet] larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'.
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 in the test, this initial multiplication creates an introduction of fp error. I had a look at this and tried to address, but I then had failures in other tests and run ouf of time allocated for this. https://github.com/apache/incubator-mxnet/pull/15770 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 to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] [incubator-mxnet] larroy commented on issue #15882: Move Windows CI build to a 64-bit toolchain to fix 'out of heap space'.
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 CI and development takes priority. 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 to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
