You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2018/08/01 17:07:37 UTC

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

apeforest commented on issue #10951: [MXNET-545] Fix broken cython build
URL: https://github.com/apache/incubator-mxnet/pull/10951#issuecomment-409649058
 
 
   @asitstands Since you initially made a mistake to set MXNET_ENABLE_CYTHON instead of MXNET_ENFORCE_CYTHON, I also find these env to be error prone. Personally, I prefer less env-based switches in the code. Do you think it makes sense to consolidate these two boolean env variables into a single int env variable such as:
   ```
   MXNET_USE_CYTHON   0 | 1 | 2
   ```
   0: do not use cython
   1: use cython, but can fallback to ctypes
   2: enforce cython
    

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services