CuBIDS icon indicating copy to clipboard operation
CuBIDS copied to clipboard

Add command to replace IntendedFor fields with B0FieldSource/B0FieldIdentifier

Open tsalo opened this issue 1 year ago • 6 comments

SDCFlows doesn't seem to work on phase-difference field maps with the IntendedFor field, but it does work on B0FieldSource/B0FieldIdentifier fields. EPI distortion correction seems to work with either one.

I'm not sure how to correctly divvy up field maps if multiple ones are intended for the same distorted files.

Per Matt, check for shim settings.

tsalo avatar Dec 04 '23 14:12 tsalo

Just to double-check, which of the phasediff files had the IntendedFor in its sidecar? both the phasediff and the magnitude images?

mattcieslak avatar Dec 04 '23 15:12 mattcieslak

I just checked and it's actually a separate-phase maps setup, but the IntendedFor is present in all four files.

tsalo avatar Dec 04 '23 15:12 tsalo

This might be an sdcflows bug then

mattcieslak avatar Dec 04 '23 15:12 mattcieslak

@mattcieslak you're right. It looks like SDCFlows identifies B0Field* fmaps first, and it ignores the session entity when it does it, so having even one B0Field*-type fmap in the BIDS dataset will mean that IntendedFor ones won't be identified.

tsalo avatar Dec 04 '23 18:12 tsalo

Still, given that there's this weird limitation in SDCFlows, it would definitely be useful to have a command that ensures that we use consistent field map labeling in a dataset.

tsalo avatar Dec 04 '23 19:12 tsalo

Plus it would generally be a good idea to include tools to basically update BIDS datasets to use newer features. Here are a few examples:

  • IntendedFor --> B0Field*
  • Relative paths --> BIDS URIs
  • rec-magnitude/phase from old versions of Heudiconv --> part-mag/phase
  • Deprecated suffixes phase, FLASH, PD, and T2star

tsalo avatar Jan 31 '24 17:01 tsalo