Skip to content

Commit 802dae9

Browse files
committed
fixup: ruff
1 parent 5e34541 commit 802dae9

File tree

2 files changed

+32
-13
lines changed

2 files changed

+32
-13
lines changed

babs/bootstrap.py

Lines changed: 31 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -230,13 +230,14 @@ def babs_bootstrap(
230230
# Discover container image path from the container dataset:
231231
containers_path = op.join(self.analysis_path, 'containers')
232232
result = dlapi.containers_list(dataset=containers_path, result_renderer='disabled')
233-
container_info = [r for r in result
234-
if r['action'] == 'containers' and r['name'] == container_name]
233+
container_info = [
234+
r for r in result if r['action'] == 'containers' and r['name'] == container_name
235+
]
235236
if not container_info:
236237
available = [r['name'] for r in result if r['action'] == 'containers']
237238
raise ValueError(
238239
f"Container '{container_name}' not found in container dataset. "
239-
f"Available: {available}"
240+
f'Available: {available}'
240241
)
241242
image_path_in_ds = op.relpath(container_info[0]['path'], containers_path)
242243
container_image_path = op.join('containers', image_path_in_ds)
@@ -265,8 +266,12 @@ def babs_bootstrap(
265266
path=op.join(self.analysis_path, container_image_path),
266267
)
267268

268-
container = Container(container_ds, container_name, container_config,
269-
container_image_path=container_image_path)
269+
container = Container(
270+
container_ds,
271+
container_name,
272+
container_config,
273+
container_image_path=container_image_path,
274+
)
270275
container.sanity_check(self.analysis_path)
271276

272277
# ==============================================================
@@ -290,11 +295,18 @@ def babs_bootstrap(
290295
container = containers[0]
291296
else:
292297
self._bootstrap_single_app_scripts(
293-
container_ds, container_name, container_config, system,
298+
container_ds,
299+
container_name,
300+
container_config,
301+
system,
302+
container_image_path=container_image_path,
303+
)
304+
container = Container(
305+
container_ds,
306+
container_name,
307+
container_config,
294308
container_image_path=container_image_path,
295309
)
296-
container = Container(container_ds, container_name, container_config,
297-
container_image_path=container_image_path)
298310

299311
# Copy in any other files needed:
300312
self._init_import_files(container.config.get('imported_files', []))
@@ -432,12 +444,20 @@ def babs_bootstrap(
432444
print('`babs init` was successful!')
433445

434446
def _bootstrap_single_app_scripts(
435-
self, container_ds, container_name, container_config, system,
447+
self,
448+
container_ds,
449+
container_name,
450+
container_config,
451+
system,
436452
container_image_path=None,
437453
):
438454
"""Bootstrap scripts for single BIDS app configuration."""
439-
container = Container(container_ds, container_name, container_config,
440-
container_image_path=container_image_path)
455+
container = Container(
456+
container_ds,
457+
container_name,
458+
container_config,
459+
container_image_path=container_image_path,
460+
)
441461

442462
# Generate `<containerName>_zip.sh`: ----------------------------------
443463
# Zip-only script (container execution is now handled by containers-run

babs/container.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,7 @@
1818
class Container:
1919
"""This class is for the BIDS App Container"""
2020

21-
def __init__(self, container_ds, container_name, config_yaml_file,
22-
container_image_path=None):
21+
def __init__(self, container_ds, container_name, config_yaml_file, container_image_path=None):
2322
"""
2423
This is to initialize Container class.
2524

0 commit comments

Comments
 (0)