diff --git a/.maint/update_authors.py b/.maint/update_authors.py index d1de64874f..79990625ea 100644 --- a/.maint/update_authors.py +++ b/.maint/update_authors.py @@ -152,7 +152,6 @@ def _namelast(inlist): @click.group() def cli(): """Generate authorship boilerplates.""" - pass @cli.command() diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 907fbeb250..48f9b37e3b 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -13,7 +13,7 @@ repos: - id: fix-byte-order-marker - id: trailing-whitespace - repo: https://github.com/astral-sh/ruff-pre-commit - rev: v0.15.15 + rev: v0.16.1 hooks: - id: ruff args: [ --fix ] diff --git a/sdcflows/utils/tests/test_wrangler.py b/sdcflows/utils/tests/test_wrangler.py index d933625eca..f571e809a5 100644 --- a/sdcflows/utils/tests/test_wrangler.py +++ b/sdcflows/utils/tests/test_wrangler.py @@ -527,7 +527,7 @@ def test_fieldmapless(tmp_path): }, } me_metadata = [{'EchoTime': 0.01 * i, **bold['metadata']} for i in range(1, 4)] - sbref = {**bold, **{'suffix': 'sbref'}} + sbref = {**bold, 'suffix': 'sbref'} spec = { '01': { 'anat': [T1w], @@ -561,7 +561,7 @@ def test_fieldmapless(tmp_path): spec = { '01': { 'anat': [T1w], - 'func': [{'echo': i + 1, **bold, **{'metadata': me_metadata[i]}} for i in range(3)], + 'func': [{'echo': i + 1, **bold, 'metadata': me_metadata[i]} for i in range(3)], }, } generate_bids_skeleton(bids_dir, spec) @@ -606,8 +606,8 @@ def test_fieldmapless(tmp_path): spec = { '01': { 'anat': [T1w], - 'func': [{'echo': i + 1, **bold, **{'metadata': me_metadata[i]}} for i in range(3)] - + [{'echo': i + 1, **sbref, **{'metadata': me_metadata[i]}} for i in range(3)], + 'func': [{'echo': i + 1, **bold, 'metadata': me_metadata[i]} for i in range(3)] + + [{'echo': i + 1, **sbref, 'metadata': me_metadata[i]} for i in range(3)], }, } generate_bids_skeleton(bids_dir, spec) diff --git a/sdcflows/utils/wrangler.py b/sdcflows/utils/wrangler.py index f22725167f..a27b58d70b 100644 --- a/sdcflows/utils/wrangler.py +++ b/sdcflows/utils/wrangler.py @@ -394,7 +394,8 @@ def find_estimators( for fmap in layout.get( **{ **base_entities, - **{'suffix': ['fieldmap', 'phasediff', 'phase1'], 'session': sessions}, + 'suffix': ['fieldmap', 'phasediff', 'phase1'], + 'session': sessions, } ): try: @@ -425,7 +426,7 @@ def find_estimators( dirs = layout.get_directions(**entities) if len(dirs) > 1: by_intent = {} - for fmap in layout.get(**{**entities, **{'direction': dirs}}): + for fmap in layout.get(**{**entities, 'direction': dirs}): fmapfile = fm.FieldmapFile( fmap.path, metadata=_filter_metadata(fmap.get_metadata(), subject), @@ -451,7 +452,9 @@ def find_estimators( has_intended = layout.get( **{ **base_entities, - **{'suffix': 'epi', 'IntendedFor': Query.REQUIRED, 'session': sessions}, + 'suffix': 'epi', + 'IntendedFor': Query.REQUIRED, + 'session': sessions, } ) @@ -515,7 +518,7 @@ def find_estimators( fmapless = False # Find fieldmap-less schemes - anat_file = layout.get(**{**base_entities, **{'suffix': anat_suffix, 'session': sessions}}) + anat_file = layout.get(**{**base_entities, 'suffix': anat_suffix, 'session': sessions}) if not fmapless or not anat_file: logger.debug('Skipping fmap-less estimation') @@ -590,7 +593,9 @@ def find_anatomical_estimators( candidates = layout.get( **{ **base_entities, - **{'suffix': suffixes, 'session': ses, 'datatype': datatype}, + 'suffix': suffixes, + 'session': ses, + 'datatype': datatype, } )