search

mind-inria / mri-nufft

Doing non-Cartesian MR Imaging has never been so easy.
https://mind-inria.github.io/mri-nufft/
BSD 3-Clause "New" or "Revised" License
54 stars 10 forks source link

Fix MaxRecursionDepth error, Memory leaks and move to CUDA12 #174

Closed chaithyagr closed 3 months ago

chaithyagr commented 3 months ago

We had some issues of MaxRecurssionDepth, which was fixed and on extensively testing, I saw a memory leak in gpuNUFFT (fixed in https://github.com/chaithyagr/gpuNUFFT/pull/23). Some more memory leaks from cuFFT comes from from CUDA11.8: https://forums.developer.nvidia.com/t/nvc-cufft-memory-leak/279476/3

To fix this, we moved to using CUDA12.1, which is default version which comes with torch.

Also have a bunch of other miscellaneous tasks:

1) Added support for binder button in docs. 2) Fixed a minor bug in smaps estimation which didnt allow int values for blurr_factor.