Skip to content

allow multicoil dataset coil map estim + fastmri multicoil + cmrxrecon singlecoil#53

Open
Andrewwango wants to merge 6 commits into
MatthiasLen:mainfrom
Andrewwango:multicoil
Open

allow multicoil dataset coil map estim + fastmri multicoil + cmrxrecon singlecoil#53
Andrewwango wants to merge 6 commits into
MatthiasLen:mainfrom
Andrewwango:multicoil

Conversation

@Andrewwango
Copy link
Copy Markdown
Collaborator

@Andrewwango Andrewwango commented May 22, 2026

closes #23

  • support multicoil fastmri
  • estimate automatically coil maps
  • save to fastmri_multicoil dir
image
  • add cmrxrecon (supports both singlecoil and multicoil)
image

@Andrewwango Andrewwango changed the title allow multicoil dataset coil map estim + save to fastmri_multicoil allow multicoil dataset coil map estim + fastmri multicoil + cmrxrecon singlecoil May 22, 2026
@Andrewwango Andrewwango linked an issue May 22, 2026 that may be closed by this pull request
@melanie-dohmen-bayer
Copy link
Copy Markdown
Collaborator

Hi @Andrewwango @MatthiasLen,

I noticed, that the multi-coil data was not compatible with the oasis model, I tried to add the missing adaptions. The results look ok, if no additional distortion is applied:

ALGO_unet-oasis-acceleration4_no distortion_sample_0

for variable undersampling, the results are also fine, I would say:
ALGO_unet-oasis-acceleration4_Cartesian undersampling (variable density)_sample_0

but for the equidistant undersampling, the model fails:

ALGO_unet-oasis-acceleration4_Cartesian undersampling (equispaced, zero ACS)_sample_0

Is this possible/expected?

@Andrewwango could you review my changes on your branch?

Best regards,
Melanie

@melanie-dohmen-bayer melanie-dohmen-bayer self-assigned this May 22, 2026
Copy link
Copy Markdown
Collaborator

@melanie-dohmen-bayer melanie-dohmen-bayer left a comment

Choose a reason for hiding this comment

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

@Andrewwango:
Please check my adaption for using the oasis fft path, apart from that, thanks a lot for your contribution!

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.

data: add CMRxReconSliceDataset cardiac MRI Add multicoil MRI

2 participants