Skip to content

Extend mrd for reconstruction#147

Merged
J-Endres merged 6 commits intoMRsources:mainfrom
ckolbPTB:extend_mrd
Nov 26, 2025
Merged

Extend mrd for reconstruction#147
J-Endres merged 6 commits intoMRsources:mainfrom
ckolbPTB:extend_mrd

Conversation

@ckolbPTB
Copy link
Copy Markdown
Contributor

@ckolbPTB ckolbPTB commented Nov 7, 2025

Here are a few changes which are needed to ensure reconstruction is possible with MRpro. We are using this here to validate our Pulseq sequences:
Code: https://github.com/PTB-MR/mrseq/blob/main/examples/t2_multi_echo_se_single_line.ipynb
Rendered output: https://ptb-mr.github.io/mrseq/t2_multi_echo_se_single_line.html

Main changes:

  • add resonance frequency which is a required field for ISMRMRD
  • ensure values from Pulseq (in s) are translated to ms for ISMRMRD
  • each acquisition should have orientation information, although we do not know it we give it something reasonable
  • NOISE samples are detected - here it would be great if noise could be added as simulated in MRzero, not sure if this information is available at this stage

Further things which would be nice (but I don't know how to do it):

  • get the FOV and matrix sizes correctly from MRzero
  • get a timestamp for each acquisition from MRzero (done in 8942945)

Any suggestions or further improvements very welcome!

Copy link
Copy Markdown
Collaborator

@J-Endres J-Endres left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the additions!
There is a possibility of an crash if FOV is not set, otherwise I'm happy to merge the changes.

Comment thread python/MRzeroCore/simulation/sig_to_mrd.py Outdated
Comment thread python/MRzeroCore/simulation/sig_to_mrd.py
@J-Endres J-Endres dismissed their stale review November 26, 2025 10:55

resolved

@J-Endres J-Endres merged commit f04529e into MRsources:main Nov 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants