search

cesium-ml / cesium

Machine Learning Time-Series Platform
Other
671 stars 101 forks source link

Added cython to installation process #59

Closed bnaul closed 9 years ago

acrellin commented 9 years ago

:+1:

stefanv commented 9 years ago

The requirements.txt file lists runtime requirements, so I don't think we should put Cython in there.

acrellin commented 9 years ago

Maybe we can just mention it in the installation instructions

bnaul commented 9 years ago

Should I add it to the conda create ... command?

stefanv commented 9 years ago

Again, this should only be in the developer instructions. Normal users would not need to compile anything if they install using pip. But wherever we put the "Installation from source" instructions should contain this.

bnaul commented 9 years ago

Got it, thanks!