-
Notifications
You must be signed in to change notification settings - Fork 120
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
Improve installation instructions to account for unuspported Python versions #445
Comments
Thank you for submitting your first issue to p5py |
Hi, thank you for the feedback. Feel free to make a PR, we will try to extend support for other Python versions. However, p5py should work fine for versions <= 3.10, as we have a CI suite running for all these versions. |
TL;DR -- would you welcome a PR that sets a floor for the Thanks for setting expectations there. Looks like my issues on 3.10 and 3.11 were transient environmental things.
Worth noting that your tests don't test against Python 3.11 or .12. My running on Apple Silicon also introduces some drift from your CI environment, which could matter while Cythonizing dependencies. I can consistently reproduce the installation failure for 3.12. Looks like numpy installation failure
(I'll note that this is consistent with my professional experience with Cython. Disappointing to see that errors haven't gotten clearer in the past 5 years. But I guess that's C/Cython/cpython for ya.) By manually specifying a successfull installation
So from my perspective, the correct PR here would be one that restricts |
Hi @mambocab , thank you for the detailed research. Please feel free to make a PR for this and we can have a release with updated docs. |
I also failed installing P5.py on my mac silicon M2; I have Python 3.11 and 3.10, failed to install with both versions. I now created an environment for python 3.7, but I hope you guys are aware that 3.7 is older than silicon, so it is really troublesome to find all libraries matching 3.7. |
Can you share the errors you faced while installing? |
Describe the bug
p5
doesn't support newer versions of Python yet, but this isn't reflected in the installation docs, which say "p5 requires Python 3.6 or above".I'm seeing maintainers recommend 3.8 in bug reports, which is ultimately what worked for me.
If there's some way to pin a
Cython
ornumpy
version and get this working on newer Pythons, that could also be a nice improvement. I couldn't find one but based my attempts off the recommendations here.To Reproduce
I tried a few ways to install on newer Pythons but ran into bugs that are already reported elsewhere. Pinning a
Cython
version ultimately didn't work.System information:
Thanks for your work! I'm up and running now, but it would've been a smoother experience with this small improvement to installation instructions.
The text was updated successfully, but these errors were encountered: