asitstands commented on issue #10951: [MXNET-545] Fix broken cython build
URL: https://github.com/apache/incubator-mxnet/pull/10951#issuecomment-397852430
 
 
   I added the cython build to Python3 CentOS 7 CPU and GPU environments. I 
have made an intentional bug in the cython module and confirmed that it 
actually causes [the failures of the unit 
tests](http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-10951/16/pipeline/739).
 I think that now this PR is completed. Please review the code. Here is the 
summary of the changes.
   
   1. A bug in the `setup.py` which causes problems with the recent cython 0.28 
(#10295, #10738) is fixed.
   1. The cython module for the ndarray has caused segmentation faults since 
the introduction of the sparse array. This is fixed.
   1. Now the compiled cython module has `rpath` to find `libmxnet.so`, so the 
deployment is easier.
   1. A new makefile variable `PYTHON` to set the python executable is added 
(`make/config.mk`). The cython build target needs to run python but some 
environments have nonstandard executable name such as `python3.6`.
   1. Documentation for the environment variables `MXNET_ENABLE_CYTHON` and 
`MXNET_ENFORCE_CYTHON` is added.
   1. Documentation for the optional cython build is added to the install guide.
   1. Python3 CentOS 7 CPU and Python3 CentOS 7 GPU environments in the CI now 
use the cython modules.
   
   `make cython` generates two `.so` files in `mxnet/_cy3` (or `_cy2` with 
python 2). The exact names of the files depend on the environment. They don't 
need a special care for installation. It is enough just to be there.

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to