Merge pull request #495 from nipreps/fix/sanitize-out

FIX: Use the sanitized id when naming through nipype

Author: mgxd

sdcflows/transform.py

@@ -498,7 +498,8 @@ def apply(
             pe_info.append(
                 (
                     pe_axis,
-                    # Displacements are reversed if either is true (after ensuring positive cosines)
+                    # Displacements are reversed if either is true
+                    # (after ensuring positive cosines)
                     -ro_time[volid] if (axis_flip ^ pe_flip) else ro_time[volid],
                 )
             )

sdcflows/workflows/base.py

@@ -130,7 +130,7 @@ def init_fmap_preproc_wf(
         source_files = [str(f.path) for f in estimator.sources if f.suffix not in ('T1w', 'T2w')]
 
         out_map = pe.Node(
-            niu.IdentityInterface(fields=out_fields), name=f'out_{estimator.bids_id}'
+            niu.IdentityInterface(fields=out_fields), name=f'out_{estimator.sanitized_id}'
         )
         out_map.inputs.fmap_id = estimator.bids_id
 

sdcflows/workflows/tests/test_base.py

@@ -57,7 +57,7 @@ def test_fmap_wf(tmpdir, workdir, outdir, bids_layouts, dataset, subject):
         if estimator.method != fm.EstimatorType.PEPOLAR:
             continue
 
-        inputnode = wf.get_node(f'in_{estimator.bids_id}')
+        inputnode = wf.get_node(f'in_{estimator.sanitized_id}')
         inputnode.inputs.in_data = [str(f.path) for f in estimator.sources]
         inputnode.inputs.metadata = [f.metadata for f in estimator.sources]