-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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 fails: Cython.Compiler.Errors.InternalError: Internal compiler error: '_common.pxi' not found #684
Comments
Dangit, looks like 0.5.0 didn't include the .pxi if you want to use your own cython to make the C file. I made this patch 327a8ff and released 0.5.1, can you try again? |
0.5.1 fails like this:
py311-cython3-3.0.11 is used. |
Is that a failure? Looks like a warning. If you are using numpy 2, then the 0.5 release is for you. (latest 0.5.1) If you are using numpy 1, then the 0.4 release is for you. (latest 0.4.34) |
error: incompatible integer to pointer conversion assigning to 'PyObject *' |
I think |
https://numpy.org/devdocs/numpy_2_0_migration_guide.html#migration-c-descr
You probably want the |
py311-numpy-1.26.4_2 is used. Perhaps we shouldn't upgrade the port until numpy-2 is available. |
I was resisting upgrading to numpy2 because we use cython to generate the C files and there didn't seem a clean way to make them work for both numpy1 and numpy2, so the approach someone recommended was to have an 0.4 series that supports numpy1 and and 0.5 series that supports numpy2. I couldn't get the |
Same issue here conda-forge/ta-lib-feedstock#49 But only for OSx builds. I will pin numpy to >= 2 on this conda builds and retry. |
Are you saying the “_common.pxi” issue is happening on 0.5.1 or the numpy issue?
If numpy, you can choose to support numpy 1 with ta-lib 0.4 branch, or numpy 2 with the 0.5 branch.
|
Yep, I just merged version 0.5.1 (for conda package) with the pinned version of numpy >= 2 < 3. All builds are ok. https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=1083545&view=results |
Version: 0.5.0
Python-3.11
The text was updated successfully, but these errors were encountered: