-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Build issue on Windows #16
Comments
maybe we need:
in setup.py (line 10) can you try that and let me know if it works. I don't have access to a windows machine for testing. |
Didn't help. The real issue seems to be that there just isn't any lgamma See here for a compatibility table: http://www.johndcook.com/blog/math_h/ It seems to be the only function that's causing an issue, though. It all It looks like there used to be an "lngamma" function, defined in the On Wed, May 11, 2016 at 12:54 PM, Brent Pedersen - Bioinformatics <
|
this is from 398c8ca by @superbobry . can you have a look? |
I thought |
Yes, looks like scipy has it - http://docs.scipy.org/doc/scipy/reference/generated/scipy.special.gammaln.html#scipy.special.gammaln - though not sure how hard it would be to borrow the implementation. Or if it really is worth the effort - performance-wise. |
Guys, any luck getting this work on windows? |
I'm getting an error trying to install this on Windows. At first I tried this using conda, and after it failed there, I tried using just a barebones python2.7 install.
It seems that the issue stems from the fact that the Visual C++ for Python compiler that Microsoft released does not include an lgamma in its math.h
This appears to be the only issue. If I just replace the call to lgamma (in cfisher.c) with '0', then the package builds just fine.
See output from building a wheel below.
The text was updated successfully, but these errors were encountered: