run: ruff check --fix [ignore-rev]

Author: effigies

.maint/update_authors.py

@@ -288,7 +288,7 @@ def _aslist(value):
 
     print(
         '\n\nAffiliations:\n%s'
-        % '\n'.join(['{0: >2}. {1}'.format(i + 1, a) for i, a in enumerate(affiliations)])
+        % '\n'.join([f'{i + 1: >2}. {a}' for i, a in enumerate(affiliations)])
     )
 
 

docs/tools/apigen.py

@@ -27,7 +27,7 @@
 DEBUG = True
 
 
-class ApiDocWriter(object):
+class ApiDocWriter:
     """Class for automatic detection and parsing of API docs
     to Sphinx-parsable reST format"""
 
@@ -185,7 +185,7 @@ def _parse_module(self, uri):
             # nothing that we could handle here.
             return ([], [])
 
-        f = open(filename, 'rt')
+        f = open(filename)
         functions, classes = self._parse_lines(f)
         f.close()
         return functions, classes
@@ -439,7 +439,7 @@ def write_modules_api(self, modules, outdir):
 
             out_module = ulm + self.rst_extension
             outfile = os.path.join(outdir, out_module)
-            fileobj = open(outfile, 'wt')
+            fileobj = open(outfile, 'w')
 
             fileobj.writelines(document_head + document_body)
             fileobj.close()
@@ -497,7 +497,7 @@ def write_index(self, outdir, froot='gen', relative_to=None):
             relpath = (outdir + os.path.sep).replace(relative_to + os.path.sep, '')
         else:
             relpath = outdir
-        idx = open(path, 'wt')
+        idx = open(path, 'w')
         w = idx.write
         w('.. AUTO-GENERATED FILE -- DO NOT EDIT!\n\n')
 

docs/tools/buildmodref.py

@@ -1,8 +1,6 @@
 #!/usr/bin/env python
 """Script to auto-generate API docs."""
 
-from __future__ import division, print_function
-
 # stdlib imports
 import sys
 

sdcflows/config.py

@@ -200,7 +200,7 @@
 _memory_gb = None
 try:
     if 'linux' in sys.platform:
-        with open('/proc/meminfo', 'r') as f_in:
+        with open('/proc/meminfo') as f_in:
             _meminfo_lines = f_in.readlines()
             _mem_total_line = [line for line in _meminfo_lines if 'MemTotal' in line][0]
             _mem_total = float(_mem_total_line.split()[1])

sdcflows/tests/test_transform.py

@@ -197,7 +197,7 @@ def test_apply_transform(tmpdir, outdir, datadir, pe0, hmc, fmap):
     error_margin = 0.5
     if fmap is False:  # If no fieldmap, this is equivalent to only HMC
         realigned = LinearTransformsMapping(hmc_xfms, reference=in_file).apply(in_file)
-        error = np.sqrt(((corrected.dataobj - realigned.dataobj) ** 2))
+        error = np.sqrt((corrected.dataobj - realigned.dataobj) ** 2)
 
         if outdir:
             # Do not include the first volume in the average to enhance differences
@@ -226,7 +226,7 @@ def test_apply_transform(tmpdir, outdir, datadir, pe0, hmc, fmap):
             )
     else:
         realigned = nb.load(in_file)
-        error = np.nan_to_num(np.sqrt(((corrected.dataobj - realigned.dataobj) ** 2)), nan=0)
+        error = np.nan_to_num(np.sqrt((corrected.dataobj - realigned.dataobj) ** 2), nan=0)
         error_margin = 200  # test oracle is pretty bad here - needs revision.
 
         if outdir:
@@ -290,7 +290,7 @@ def test_apply_transform(tmpdir, outdir, datadir, pe0, hmc, fmap):
             out_report=str(outdir / f'sub-pilot_ses-15_acq-b0_dir-{pe0}_desc-hmcdiff_dwi.svg'),
         ).run()
 
-        error = np.sqrt(((corrected.dataobj - corrected_nohmc.dataobj) ** 2))
+        error = np.sqrt((corrected.dataobj - corrected_nohmc.dataobj) ** 2)
         realigned.__class__(
             error,
             realigned.affine,

sdcflows/transform.py

@@ -49,9 +49,10 @@
 
 import asyncio
 import os
+from collections.abc import Sequence
 from functools import partial
 from pathlib import Path
-from typing import Callable, Sequence, Tuple
+from typing import Callable, Tuple
 from warnings import warn
 
 import attr