Skip to content
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

Issue with NumPy version 2.0 #120

Closed
chryswoehler opened this issue Oct 2, 2024 · 6 comments
Closed

Issue with NumPy version 2.0 #120

chryswoehler opened this issue Oct 2, 2024 · 6 comments
Labels
bug Something isn't working compatibility

Comments

@chryswoehler
Copy link

chryswoehler commented Oct 2, 2024

Hi,

When I was trying to set up the repo to work on issue #115, found issue with version listed for the numpy library in requirements.txt. File currently lists version as numpy>=1.1

Downgrading to a version lower then 2.0 resolved the building error when trying to run the compile command:

"If using np.array(obj, copy=False) replace it with np.asarray(obj) to allow a copy when needed (no behavior change in NumPy 1.x).
For more details, see https://numpy.org/devdocs/numpy_2_0_migration_guide.html#adapting-to-changes-in-the-copy-keyword."

So it seems that the library has had a change to the copy keyword in it's 2.0 version.

Changing the version restriction for numpy to numpy>=1.1,<=2.0 should solve the issue? Or would it be better to work towards updating for 2.0 keyword support rather then restricting to pre 2.0 library versions?

@chryswoehler
Copy link
Author

This is my first time working on an open source project, sorry if this is not an actual issue :)

@jimy-byerley jimy-byerley added bug Something isn't working compatibility labels Oct 2, 2024
@jimy-byerley
Copy link
Owner

Hello @chryswoehler, this totally seems to be an issue. The concept of issue is not very specific anyway. They serve as bug reports as well as feature request anyway ;)

changing the version requirement would be an easyfix of course but it is far more interesting to extend support to numpy 2 as you said. Since it doesn't seem much more than changing few numpy calls we should go for it.

@chryswoehler
Copy link
Author

I'll go ahead and just work towards numpy 2 support then. Thanks!

chryswoehler added a commit to chryswoehler/pymadcad that referenced this issue Oct 2, 2024
@Oakchris1955
Copy link

Oakchris1955 commented Nov 10, 2024

Since 0.17 is now broken, shouldn't it be yanked, or does this issue cover older versions too?

@jimy-byerley
Copy link
Owner

Yes old oversions a well, in fact numpy 2 was not existing while the previous madcad versions were released

@jimy-byerley
Copy link
Owner

fixed in the new release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working compatibility
Projects
None yet
Development

No branches or pull requests

3 participants