Skip to content

chore: compatibility with NumPy2#207

Merged
pbeaucage merged 6 commits intomainfrom
numpy2_compat
Jun 25, 2025
Merged

chore: compatibility with NumPy2#207
pbeaucage merged 6 commits intomainfrom
numpy2_compat

Conversation

@pbeaucage
Copy link
Copy Markdown
Collaborator

@pbeaucage pbeaucage commented May 7, 2025

#208

This branch is an attempt to understand what the pain point(s) are with numpy2.

@pbeaucage
Copy link
Copy Markdown
Collaborator Author

Adding Win/Py3.13 to CI matrix to see if we pass there now.

@pbeaucage
Copy link
Copy Markdown
Collaborator Author

This seems to work, but it is likely that numpy 2 will break outer user workflows. It would be prudent to test well before merging.

@pbeaucage pbeaucage marked this pull request as ready for review May 7, 2025 19:51
@PriyankaKetkarBNL
Copy link
Copy Markdown
Collaborator

PriyankaKetkarBNL commented Jun 12, 2025

Thanks for working on this! I just tried it out, and it works great on my end. I’m happy to provide an approval, but just wanted to check first if it would be good for me to test anything specific?

I tested this on a Windows computer using Python 3.13.5. I’ve attached a notebook with the workflow I used. The workflow is pretty basic, so I’d be happy to test additional workflow items as well.
TestNotebook.zip

@pbeaucage
Copy link
Copy Markdown
Collaborator Author

The major thing is that I believe downstream packages/usage might break, but we can't insulate against that. Particularly users that frequently cast to np.array() will find that there are problems with the new array primitives, but that will hopefully help reduce that - it's bad practice anyway.

@pbeaucage
Copy link
Copy Markdown
Collaborator Author

The CI testing covers the usage, the main question is "does the change make sense", at least that's how I look at it.

@PriyankaKetkarBNL
Copy link
Copy Markdown
Collaborator

Ah, okay. The tracked changes seem fine to me, but I don't think I'm very knowledgeable on what all the implications might be.

@pbeaucage pbeaucage merged commit 351619e into main Jun 25, 2025
11 checks passed
@pbeaucage pbeaucage deleted the numpy2_compat branch June 25, 2025 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants