NervanaSystems / neon

Intel® Nervana™ reference deep learning framework committed to best performance on all hardware
http://neon.nervanasys.com/docs/latest
Apache License 2.0
3.87k stars 811 forks source link

Makefile Cython Version Outdated #460

Open tiago1405 opened 5 years ago

tiago1405 commented 5 years ago

The Makefile on the repo is seemingly outdated causing the following error when make sysinstall was used:

Failed building wheel for cython neon failed with error code 1 in /tmp/pip-install-0vxv6117/cython/

The error occurred when the sysinstall attempted to install Cython v:0.23.1. Using the commands pip install cython or conda install cython would not resolve the issue as they were installing the newer Cython v:0.29.1.

I fixed this by correcing the cython version number at these two lines at in the Makefile:

(127)$(ACTIVATE); pip install cython==0.23. (170)@pip install cython==0.23.1

----> TO ---->

(127)$(ACTIVATE); pip install cython==0.29.1 (170)@pip install cython==0.29.1

I had some other issues installing Neon through make and make sysinstall but this allowed me to move forward with the installation.