[GitHub] larroy commented on issue #10951: [MXNET-545] Fix broken cython build

2018-12-07 Thread GitBox
larroy commented on issue #10951: [MXNET-545] Fix broken cython build URL: https://github.com/apache/incubator-mxnet/pull/10951#issuecomment-445208876 Do we have a bug tracking the segfault with openmp? The PR looks ok to me, even though I'm not familiar with cython. I think if you

[GitHub] larroy commented on issue #10951: [MXNET-545] Fix broken cython build

2019-02-06 Thread GitBox
larroy commented on issue #10951: [MXNET-545] Fix broken cython build URL: https://github.com/apache/incubator-mxnet/pull/10951#issuecomment-461105625 I'm not an expert in cython but the changes look sane. I think we can merge this at least tentatively and observe. Since the author did some