rf: Deprecate unused t1w_inversion argument

Author: effigies

sdcflows/workflows/fit/syn.py

@@ -326,7 +326,7 @@ def init_syn_preprocessing_wf(
     name="syn_preprocessing_wf",
     omp_nthreads=1,
     auto_bold_nss=False,
-    t1w_inversion=False,
+    t1w_inversion=None,
     sd_prior=True,
 ):
     """
@@ -353,6 +353,7 @@ def init_syn_preprocessing_wf(
         of BOLD images.
     t1w_inversion : :obj:`bool`
         Run T1w intensity inversion so that it looks more like a T2 contrast.
+        (DEPRECATED. Does nothing.)
     sd_prior : :obj:`bool`
         Enable using a prior map to regularize the SyN cost function.
 
@@ -401,6 +402,10 @@ def init_syn_preprocessing_wf(
     from ...interfaces.utils import Deoblique, DenoiseImage
     from ...interfaces.brainmask import BrainExtraction, BinaryDilation
 
+    if t1w_inversion is not None:
+        import warnings
+        warnings.warn("The `t1w_inversion` argument is deprecated and does nothing.", DeprecationWarning)
+
     workflow = Workflow(name=name)
 
     inputnode = pe.Node(

sdcflows/workflows/fit/tests/test_syn.py

@@ -42,7 +42,6 @@ def test_syn_wf(tmpdir, datadir, workdir, outdir, sloppy_mode, sd_prior):
         omp_nthreads=4,
         debug=sloppy_mode,
         auto_bold_nss=True,
-        t1w_inversion=True,
         sd_prior=sd_prior,
     )
     prep_wf.inputs.inputnode.in_epis = [