Closed alexjaffray closed 1 year ago
Thanks for the report. I will have a look. Are you sure that this worked before? I cannot directly see what has changed in that domain recently.
Probably, this is the problematic line: https://github.com/MagneticResonanceImaging/MRIReco.jl/blob/master/MRIOperators/src/EncodingOp.jl#L155
If we find that a trajectory is Cartesian, we take the fast path and make a full FFT. Git blame says that this line is 4 years old.
@migrosser: you probably know better, what would be the best fix for that? I would say we need an "isUndersampling" function, that determines, whether a Cartesian trajectory is undersampled. Then we could somehow add the corresponding subsampling op (or do the subsampling manually)
Hm, actually what I describe seems to be already there:
https://github.com/MagneticResonanceImaging/MRIReco.jl/blob/master/MRIOperators/src/EncodingOp.jl#L166
probably we don't get into the if
.
fixed by @migrosser in https://github.com/MagneticResonanceImaging/MRIReco.jl/commit/10b5d8d230e17006699610886cda45f1abb56fc2 Releases are triggered.
I wanted to generate some undersampled Cartesian data for a small teaching example (e.e to show aliasing) and it seems with the new changes to the package one can no longer generate undersampled simulated data.
The following MWE fails for me on MRIReco v0.7.0 and MRISimulation v0.1.1 with the error:
DimensionMismatch("array could not be broadcast to match destination")