RF/BF(?): do not mark as motion corrected based on seriesdescription

Since it is not necessarily the case.  Decision should be made solely based
on the image_type

Author: yarikoptic

bin/heudiconv

@@ -527,8 +527,7 @@ def group_dicoms_into_seqinfos(
             refphys = '-'
 
         image_type = tuple(dcminfo.ImageType)
-        motion_corrected = 'MoCo' in dcminfo.SeriesDescription \
-                           or 'MOCO' in image_type
+        motion_corrected = 'MOCO' in image_type
 
         if dcminfo.get([0x18,0x24], None):
             # GE and Philips scanners

heuristics/dbic_bids.py

@@ -309,8 +309,9 @@ def infotodict(seqinfo):
     image_data_type = None
     for s in seqinfo:
         # XXX: skip derived sequences, we don't store them to avoid polluting
-        # the directory
-        if s.is_derived:
+        # the directory, unless it is the motion corrected ones
+        # (will get _rec-moco suffix)
+        if s.is_derived and not s.is_motion_corrected:
             skipped.append(s.series_id)
             lgr.debug("Ignoring derived data %s", s.series_id)
             continue
@@ -435,6 +436,9 @@ def infotodict(seqinfo):
             # if there is no _run -- no run label addded
             run_label = None
 
+        if s.is_motion_corrected:
+            assert s.is_derived, "Motion corrected images must be 'derived'"
+
         if s.is_motion_corrected and 'rec-' in regd.get('bids', ''):
             raise NotImplementedError("want to add _acq-moco but there is _acq- already")