search

astropy / photutils

Astropy package for source detection and photometry. Maintainer: @larrybradley
https://photutils.readthedocs.io
BSD 3-Clause "New" or "Revised" License
243 stars 137 forks source link

Dependencies: Cython #125

Closed ibackus closed 10 years ago

ibackus commented 10 years ago

I just found this great looking package, but it appears the documentation is missing a dependency. I couldn't install photutils until I had installed Cython. It took me a while to figure that out, since it wasn't clear from the error messages I got when running a sudo ./setup.py build

larrybradley commented 10 years ago

@ibackus Cython is required to build from the git source. We have no releases yet, but when we do, they will have the C files packaged with them and thus, will not require Cython for installation. I will add a note about this to the docs. Thanks!

cdeil commented 10 years ago

@ibackus Thanks for reporting this. Please file an issue against astropy-helpers if you think the error message from python setup.py build should be improved if Cython is missing. This way the improvement will be there for Astropy itself and all affiliated packages.

ibackus commented 10 years ago

I see, that makes sense. Thanks for working on this great package!

--Isaac

On Fri, Aug 1, 2014 at 1:43 PM, Larry Bradley notifications@github.com wrote:

@ibackus https://github.com/ibackus Cython is required to build from the git source. We have no releases yet, but when we do, they will have the C files packaged with them and thus, will not require Cython for installation. I will add a note about this to the docs. Thanks!

— Reply to this email directly or view it on GitHub https://github.com/astropy/photutils/issues/125#issuecomment-50932553.