-
Notifications
You must be signed in to change notification settings - Fork 18
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
update for NumPy 2.0 #173
update for NumPy 2.0 #173
Conversation
Unit tests are failing, but it looks like they are very old versions of Python. Probably need to be updated... |
Thanks! I'll update the unit tests and try to fix the Cython error. |
Fixed in murrayrm#1 |
Fix for Cython error in PR biocircuits#173 on Bioscrape
@ayush9pandey Looks like we are still checking against Python 3.7 and getting an error. |
Yes, it's failing on Mac OS because it seems that it's defaulting to arm64 architecture for Mac OS now. It should work if we fix the architecture to x64 when testing on mac-os, with py3.7. Fixing it.. |
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.
All tests pass!
This PR updates bioscrape to work with NumPy 2.0 and seems to be mainly working on Python 3.12.
There is an error in one of the unit tests associated with Cython 3.0's handling of **. The error message says that we need to set the compiler directive 'cpow(True)', but I was not able to do this.
Posting this how so that @ayush9pandey can see if this addresses issue #172.