nipy / heudiconv

Flexible DICOM conversion into structured directory layouts
https://heudiconv.readthedocs.io
Other
234 stars 125 forks source link

Issue a warning if there fieldmaps but they were not applied to any sequence #689

Open yarikoptic opened 1 year ago

yarikoptic commented 1 year ago

It is unlikely to have a study where fieldmaps are collected just for the sake of collecting them. I think it would be valuable to inform user that some fieldmaps left unused after assignined IntendedFor.

WDYT @pvelasco and other @nipy/team-heudiconv ?

tsalo commented 1 year ago

I think a warning is reasonable, but FWIW I've seen a lot of datasets with unused field maps (e.g., multi-echo field maps where only the first echo's data are used, or acquiring both SE and GRE field maps).