fastmri-reproducible-benchmark Move to using tensorflow-nufft

Move to using tensorflow-nufft

Open chaithyagr opened this issue 2 years ago • 3 comments

Recently, tensorflow-nufft came up, which is based out of cufiNUFFT, which is known to be computationally and memory efficient.

I will proceed with initial tests for this and get a PR to completely move away from tfkbnufft

Mar 16 '22 17:03 chaithyagr

Makes sense, after we have a test that confirms this first well informed opinion.

Mar 16 '22 22:03 zaccharieramzi

@chaithyagr can you use this issue to keep track of the progress made on the convergence using tensorflow-nufft?

May 20 '22 16:05 zaccharieramzi

I think that is fixed in tensorflow nufft.. we have minor issues still there in graph mode which I'm actively fixing. But it's tracked there.

May 20 '22 16:05 chaithyagr