-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Loading a MRD converted from Bruker in python, missing encoding header fields #33
Comments
Hi @rajramasawmy, thank you for the bug report! I fixed the missing reconSpace on master so that you can test this out. I also added a dummy entry for parallelImaging. encodingLimits are still missing. Not sure if the python software requires that. If yes: could you say what entries within encodingLimits are actually used? |
I also added the encoding limits (at least partially). |
Hey @tknopp sorry for the delay! Here is a printout of the encoding structure (for spiral, showing userParameter options too):
|
The encoding-limits are the overall experiment dimensions, which can be extracted from the data>acquisition header as well (which was my workaround). |
|
https://github.com/MagneticResonanceImaging/MRIReco.jl/blob/master/src/IO/Brukerfile.jl#L330 Here you can see how I extract the encoding limit data from the headers of the profiles. |
Thanks, I'll test it out! |
@rajramasawmy Did you manage to make it works ? |
I was testing out this example https://magneticresonanceimaging.github.io/MRIReco.jl/latest/filehandling/#Conversion
to convert the example Bruker data at http://media.tuhh.de/ibi/mrireco/brukerfileCart.zip to MRD/ISMRMRD
However, I cannot open the converted data using python's ismrmrd package,
I can open the data in matlab, but a few fields are missing in the header structure - though I can reconstruct the dataset with some workarounds.
MRD header encoding sub-structure:
The text was updated successfully, but these errors were encountered: