You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hi,dear authors.elegant implementation of your codes, I'm trying to replace my ugly code with yours.But when I'm reading the code, I notice that there is a change of isign in finufft and cufinutt for autodiff in adjnufft? Why?I didn't see similar thing as in this link(https://github.com/guanhuaw/Bjork). It seems that I miss some basic knowledge. you are trying to use same code to calculate the gradient of traj for all backend? But I also notice that your other backend didn't have funtions like toggle_grad_traj( if I didn't understand it rightly). It would be grad that you give me some tips about it.
Thanks in advances
The text was updated successfully, but these errors were encountered:
Hello @mmlyj , thank you for your interest in our work. Indeed, you are right, we use a different way to compute our gradients by using the isign in finufft, cufinufft and gpuNUFFT, as compared to what was done by original authors. Sorry that I didnt update this in docs. I think thats the best way to explain rather than in issue here, so I shall use this issue to update the docs accordingly.
But I also notice that your other backend didn't have funtions like toggle_grad_traj( if I didn't understand it rightly).
Can you point to the codes? If I remember correctly we currently only support finufft for CPU and cufinufft and gpuNUFFT for GPU for autodiff capabilities. TorchkbNUFFT and tensorflow-nufft support it internally and as they are usually slower, we didnt add support for it
chaithyagr
changed the title
Adds support to isign in finufft, cufinufft,gpunufft?
Add docs on autodiff and difference with original reference, use of isign etc.
Dec 6, 2024
It's so nice of your quick replys.It makes sense for me now that "the current vsersion only support finufft for CPU and cufinufft gpuNUFFT for GPU for autodiff capabilities". As other backend don't support autodiff, there is no need to inplement the toggle_grad_traj in other backend(/operators/autodiff.py line 81,101,).Glad to see the doc about the isign.Thanks again.
Hi,dear authors.elegant implementation of your codes, I'm trying to replace my ugly code with yours.But when I'm reading the code, I notice that there is a change of isign in finufft and cufinutt for autodiff in adjnufft? Why?I didn't see similar thing as in this link(https://github.com/guanhuaw/Bjork). It seems that I miss some basic knowledge. you are trying to use same code to calculate the gradient of traj for all backend? But I also notice that your other backend didn't have funtions like toggle_grad_traj( if I didn't understand it rightly). It would be grad that you give me some tips about it.
Thanks in advances
The text was updated successfully, but these errors were encountered: