-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
fix: pin cython to work around bug #4237
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Okay, that should fix it for real. More painful than I was expecting Just to add some background on why this is required at all - pandas distributes binaries in "wheels" (the joke here being about cheese wheels, if you're curious), but doesn't provide them for all platforms - Windows 32bit being one of them Because no binary is distributed, we have to build from scratch, but the Cython change broke that build. Hence the pin back to a working version to fix it |
There are also a couple of |
Thanks for the fixes! Agreed that should be fixed separately. |
This should hopefully fix the build
See scipy/scipy#16718