chore: Rerun ruff with python >=3.10

Author: effigies

pyproject.toml

@@ -193,6 +193,7 @@ ignore = [
   "B019",
   "SIM108",
   "C901",
+  "UP038",
 ]
 
 [tool.ruff.lint.flake8-quotes]

sdcflows/interfaces/epi.py

@@ -120,14 +120,15 @@ def _run_interface(self, runtime):
                 blips,
                 self.inputs.readout_times,
                 self.inputs.in_data,
+                strict=False,
             )
         )
 
         (
             self._results['pe_dirs_fsl'],
             self._results['readout_times'],
             self._results['out_data'],
-        ) = zip(*sorted_inputs)
+        ) = zip(*sorted_inputs, strict=False)
 
         # Put sign back last
         self._results['pe_dirs_fsl'] = [

sdcflows/interfaces/fmap.py

@@ -289,7 +289,7 @@ def _run_interface(self, runtime):
         fmap_imgs = [nb.load(fname) for fname in fmap_files]
 
         # Baseline check: paired magnitude/phase maps are basically the same
-        for mag, fmap in zip(mag_imgs, fmap_imgs):
+        for mag, fmap in zip(mag_imgs, fmap_imgs, strict=False):
             msg = _check_gross_geometry(mag, fmap)
             if msg is not None:
                 LOGGER.critical(msg)

sdcflows/interfaces/utils.py

@@ -81,13 +81,15 @@ class Flatten(SimpleInterface):
 
     def _run_interface(self, runtime):
         self._results['out_list'] = _flatten(
-            zip(self.inputs.in_data, self.inputs.in_meta),
+            zip(self.inputs.in_data, self.inputs.in_meta, strict=False),
             max_trs=self.inputs.max_trs,
             out_dir=runtime.cwd,
         )
 
         # Unzip out_data, out_meta outputs.
-        self._results['out_data'], self._results['out_meta'] = zip(*self._results['out_list'])
+        self._results['out_data'], self._results['out_meta'] = zip(
+            *self._results['out_list'], strict=False
+        )
         return runtime
 
 
@@ -449,7 +451,11 @@ def _deoblique(in_file, in_affine=None, newpath=None):
     if in_affine is None:
         orientation = nb.aff2axcodes(nii.affine)
         directions = (
-            np.array([int(l1 == l2) for l1, l2 in zip(orientation, 'RAS')], dtype='float32') * 2
+            np.array(
+                [int(l1 == l2) for l1, l2 in zip(orientation, 'RAS', strict=False)],
+                dtype='float32',
+            )
+            * 2
             - 1
         )
         newaff = np.eye(4)

sdcflows/tests/test_transform.py

@@ -48,7 +48,7 @@ def generate_oracle(
     data[19:22, ...] = 0
     data = np.pad(data + nd.binary_erosion(data, ball(3)), 8)
 
-    zooms = [z if not f else -z for z, f in zip(zooms, flip)]
+    zooms = [z if not f else -z for z, f in zip(zooms, flip, strict=False)]
     affine = np.diag(zooms + [1])
     affine[:3, 3] = -affine[:3, :3] @ ((np.array(data.shape) - 1) * 0.5)
 
@@ -123,7 +123,9 @@ def test_displacements_field(tmpdir, testdata_dir, outdir, pe_dir, rotation, fli
     assert np.all((np.sqrt(((ours - theirs) ** 2).sum()) / ours.size) < 1e-1)
 
     if outdir:
-        orientation = ''.join([ax[bool(f)] for ax, f in zip(('RL', 'AP', 'SI'), flip)])
+        orientation = ''.join(
+            [ax[bool(f)] for ax, f in zip(('RL', 'AP', 'SI'), flip, strict=False)]
+        )
 
         SimpleBeforeAfter(
             after_label='Theirs (ANTs)',

sdcflows/transform.py

@@ -49,10 +49,9 @@
 
 import asyncio
 import os
-from collections.abc import Sequence
+from collections.abc import Callable, Sequence
 from functools import partial
 from pathlib import Path
-from typing import Callable
 from warnings import warn
 
 import attr
@@ -492,7 +491,7 @@ def apply(
             ro_time *= n_volumes
 
         pe_info = []
-        for vol_pe_dir, vol_ro_time in zip(pe_dir, ro_time):
+        for vol_pe_dir, vol_ro_time in zip(pe_dir, ro_time, strict=False):
             pe_axis = 'ijk'.index(vol_pe_dir[0])
             # Displacements are reversed if either is true (after ensuring positive cosines)
             flip = (axcodes[pe_axis] in 'LPI') ^ vol_pe_dir.endswith('-')

sdcflows/utils/wrangler.py

@@ -488,7 +488,7 @@ def find_estimators(
                 targets = all_targets
                 intent_map = [[target] for target in all_targets]
 
-            for target, intent in zip(targets, intent_map):
+            for target, intent in zip(targets, intent_map, strict=False):
                 logger.debug('Found single PE target %s', target.relpath)
                 # The new estimator is IntendedFor the individual targets,
                 # even if the EPI file is IntendedFor multiple

sdcflows/workflows/fit/tests/test_syn.py

@@ -304,7 +304,10 @@ def test_mm2vox(tmp_path, fixed_ornt, moving_ornt, ijk, index):
 
     vox_params = _mm2vox(str(moving_path), str(fixed_path), ijk, config)
     vox_values = [level[2] for level in vox_params]
-    assert [mm_level[:2] == vox_level[:2] for mm_level, vox_level in zip(params, vox_params)]
+    assert [
+        mm_level[:2] == vox_level[:2]
+        for mm_level, vox_level in zip(params, vox_params, strict=False)
+    ]
     assert np.array_equal(vox_values, mm_values / [2, 3, 4][index])