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
48 stars 9 forks source link

Add quick utils functions to get B and C matrices from off-resonance maps #124

Open chaithyagr opened 3 months ago

chaithyagr commented 3 months ago

Currently, in

https://github.com/mind-inria/mri-nufft/blob/f5a6ba006184b96a1f73ff54be27d854797c79fe/src/mrinufft/operators/off_resonnance.py#L19-L58

we just generate the fourier_op which takes the B and C values into account.

paquiteau commented 3 months ago

I fully agree with you.

Getting the off resonance map from the k space data could be done in the extras module, just as we propose it for the smaps estimation.

We may also wish to have autodiff support around this as well ?

chaithyagr commented 3 months ago

Sure. But note that the estimation isnt trivial, it is computationally expensive (for now). @Daval-G you think these codes can be open sourced though? I wanted to confirm that.

philouc commented 3 months ago

I agree with @paquiteau : Estimation of off-resonance effects from k-space data should be carried out separately, as it depends on the phase information usually extracted from the image domain and TE value.