STY: Use f-strings where possible

As suggested by `pyupgrade --py38-plus`:
https://github.com/asottile/pyupgrade#f-strings

Author: DimitriPapadopoulos

niworkflows/interfaces/bids.py

@@ -269,14 +269,12 @@ def _run_interface(self, runtime):
 
         if self._require_t1w and not bids_dict['t1w']:
             raise FileNotFoundError(
-                "No T1w images found for subject sub-{}".format(self.inputs.subject_id)
+                f"No T1w images found for subject sub-{self.inputs.subject_id}"
             )
 
         if self._require_funcs and not bids_dict["bold"]:
             raise FileNotFoundError(
-                "No functional images found for subject sub-{}".format(
-                    self.inputs.subject_id
-                )
+                f"No functional images found for subject sub-{self.inputs.subject_id}"
             )
 
         for imtype in ["bold", "t2w", "flair", "fmap", "sbref", "roi", "pet", "asl"]:

niworkflows/interfaces/cifti.py

@@ -400,6 +400,6 @@ def _create_cifti_image(
     img.set_data_dtype(bold_img.get_data_dtype())
     img.nifti_header.set_intent("NIFTI_INTENT_CONNECTIVITY_DENSE_SERIES")
 
-    out_file = "{}.dtseries.nii".format(split_filename(bold_file)[1])
+    out_file = f"{split_filename(bold_file)[1]}.dtseries.nii"
     ci.save(img, out_file)
     return Path.cwd() / out_file

niworkflows/interfaces/confounds.py

@@ -318,7 +318,7 @@ def spike_regressors(
         spikes = np.zeros((max(indices) + 1, len(mask)))
         for i, m in enumerate(sorted(mask)):
             spikes[m, i] = 1
-        header = ["{:s}{:02d}".format(header_prefix, vol) for vol in range(len(mask))]
+        header = [f"{header_prefix}{vol:02d}" for vol in range(len(mask))]
         spikes = pd.DataFrame(data=spikes, columns=header)
     if concatenate:
         return pd.concat((data, spikes), axis=1)
@@ -359,7 +359,7 @@ def temporal_derivatives(order, variables, data):
         variables_deriv[0] = variables
         order = set(order) - set([0])
     for o in order:
-        variables_deriv[o] = ["{}_derivative{}".format(v, o) for v in variables]
+        variables_deriv[o] = [f"{v}_derivative{o}" for v in variables]
         data_deriv[o] = np.tile(np.nan, data[variables].shape)
         data_deriv[o][o:, :] = np.diff(data[variables], n=o, axis=0)
     variables_deriv = reduce((lambda x, y: x + y), variables_deriv.values())
@@ -402,7 +402,7 @@ def exponential_terms(order, variables, data):
         variables_exp[1] = variables
         order = set(order) - set([1])
     for o in order:
-        variables_exp[o] = ["{}_power{}".format(v, o) for v in variables]
+        variables_exp[o] = [f"{v}_power{o}" for v in variables]
         data_exp[o] = data[variables] ** o
     variables_exp = reduce((lambda x, y: x + y), variables_exp.values())
     data_exp = pd.DataFrame(

niworkflows/interfaces/fixes.py

@@ -68,7 +68,7 @@ def _run_interface(self, runtime, correct_return_codes=(0,)):
         _copyxform(
             self.inputs.reference_image,
             os.path.abspath(self._gen_filename("output_image")),
-            message="%s (niworkflows v%s)" % (self.__class__.__name__, __version__),
+            message=f"{self.__class__.__name__} (niworkflows v{__version__})",
         )
         return runtime
 
@@ -110,7 +110,7 @@ def _run_interface(self, runtime, correct_return_codes=(0,)):
             _copyxform(
                 self.inputs.fixed_image[0],
                 os.path.abspath(out_file),
-                message="%s (niworkflows v%s)" % (self.__class__.__name__, __version__),
+                message=f"{self.__class__.__name__} (niworkflows v{__version__})",
             )
 
         # Inverse transform
@@ -119,7 +119,7 @@ def _run_interface(self, runtime, correct_return_codes=(0,)):
             _copyxform(
                 self.inputs.moving_image[0],
                 os.path.abspath(out_file),
-                message="%s (niworkflows v%s)" % (self.__class__.__name__, __version__),
+                message=f"{self.__class__.__name__} (niworkflows v{__version__})",
             )
 
         return runtime

niworkflows/interfaces/freesurfer.py

@@ -552,7 +552,7 @@ def medial_wall_to_nan(in_file, subjects_dir, den=None, newpath=None):
     if target_subject.startswith("fsaverage"):
         cortex = nb.freesurfer.read_label(
             os.path.join(
-                subjects_dir, target_subject, "label", "{}.cortex.label".format(fn[:2])
+                subjects_dir, target_subject, "label", f"{fn[:2]}.cortex.label"
             )
         )
         medial = np.delete(np.arange(len(func.darrays[0].data)), cortex)
@@ -578,7 +578,7 @@ def mri_info(fname, argument):
     import subprocess as sp
     import numpy as np
 
-    cmd_info = "mri_info --%s %s" % (argument, fname)
+    cmd_info = f"mri_info --{argument} {fname}"
     proc = sp.Popen(cmd_info, stdout=sp.PIPE, shell=True)
     data = bytearray(proc.stdout.read())
     mstring = np.fromstring(data.decode("utf-8"), sep="\n")

niworkflows/interfaces/header.py

@@ -315,7 +315,7 @@ def _run_interface(self, runtime):
     Analyses of this dataset MAY BE INVALID.
 </p>
 """
-        snippet = '<h3 class="elem-title">%s</h3>\n%s\n' % (warning_txt, description)
+        snippet = f'<h3 class="elem-title">{warning_txt}</h3>\n{description}\n'
         # Store new file and report
         img.to_filename(out_fname)
         with open(out_report, "w") as fobj:

niworkflows/interfaces/nibabel.py

@@ -432,7 +432,7 @@ def _run_interface(self, runtime):
             self.inputs.moving_image,
             fov_mask=self.inputs.fov_mask,
             force_xform_code=self.inputs.xform_code,
-            message="%s (niworkflows v%s)" % (self.__class__.__name__, __version__),
+            message=f"{self.__class__.__name__} (niworkflows v{__version__})",
             newpath=runtime.cwd,
         )
         return runtime

niworkflows/interfaces/tests/test_bids.py

@@ -652,7 +652,7 @@ def test_ReadSidecarJSON_connection(testdata_dir, field):
     "derivatives, subjects_dir",
     [
         (os.getenv("FREESURFER_HOME"), "subjects"),
-        ("/tmp", "%s/%s" % (os.getenv("FREESURFER_HOME"), "subjects")),
+        ("/tmp", "{}/{}".format((os.getenv("FREESURFER_HOME"), "subjects"))),
     ],
 )
 def test_fsdir_noaction(derivatives, subjects_dir):

niworkflows/interfaces/utility.py

@@ -201,13 +201,13 @@ def _check_len(self, name, new):
         if name in self._fields:
             if isinstance(new, str) or len(new) < 1:
                 raise ValueError(
-                    'Trying to set an invalid value (%s) for input "%s"' % (new, name)
+                    f'Trying to set an invalid value ({new}) for input "{name}"'
                 )
 
             if len(new) != len(self.inputs.keys):
                 raise ValueError(
-                    'Length of value (%s) for input field "%s" does not match '
-                    "the length of the indexing list." % (new, name)
+                    f'Length of value ({new}) for input field "{name}" '
+                    "does not match the length of the indexing list."
                 )
 
     def _run_interface(self, runtime):

niworkflows/reports/core.py

@@ -303,7 +303,7 @@ def _load_config(self, config):
             self.out_dir = self.out_dir / self.packagename
 
         if self.subject_id is not None:
-            self.root = self.root / "sub-{}".format(self.subject_id)
+            self.root = self.root / f"sub-{self.subject_id}"
 
         if "template_path" in settings:
             self.template_path = config.parent / settings["template_path"]
@@ -371,7 +371,7 @@ def index(self, config):
 
         # Populate errors section
         error_dir = (
-            self.out_dir / "sub-{}".format(self.subject_id) / "log" / self.run_uuid
+            self.out_dir / f"sub-{self.subject_id}" / "log" / self.run_uuid
         )
         if error_dir.is_dir():
             from ..utils.misc import read_crashfile