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

Numpy and Torch Dependencies #104

Closed
gabuzi opened this issue Dec 8, 2024 · 3 comments
Closed

Numpy and Torch Dependencies #104

gabuzi opened this issue Dec 8, 2024 · 3 comments

Comments

@gabuzi
Copy link
Contributor

gabuzi commented Dec 8, 2024

Hi,

Thanks for the new releases!

I saw that the >= v1.5.0 releases require torch >= 2.0. Has there been any change related to #98?

Similarly, I saw the >= 1.5.2 require numpy >= 2.0. I just quickly checked the diffs of the new releases, but couldn't spot any change that makes the package incompatible with the 1.x numpy (I might have missed it).

@mmuckley
Copy link
Owner

mmuckley commented Dec 8, 2024

Hello @gabuzi, I ran some tests offline and found that older versions of numpy caused some errors, so I decided to bump up requirements to the latest just to be safe, as well as torch 2.0.

Users can always ignore dependencies on installation if they want to try to use an older numpy, or if there are errors, they can also select 1.4 of torchkbnufft (which is almost exactly the same as 1.5).

@gabuzi
Copy link
Contributor Author

gabuzi commented Dec 9, 2024

I see, thanks for clarifying!
I was looking at this from the conda side, preparing the new release and was considering potential impact and wasn't quite sure. As you say, 1.4 of course remains available, which the solver will automatically fall back to in case of some other dependency prohibiting torch 2.0 or numpy 2.0.

@mmuckley
Copy link
Owner

Closing as this does not document a bug.

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

No branches or pull requests

2 participants