Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
  • Loading branch information
effigies authored Mar 22, 2024
1 parent f60c0f1 commit ba4611b
Showing 1 changed file with 7 additions and 9 deletions.
16 changes: 7 additions & 9 deletions smriprep/workflows/surfaces.py
Original file line number Diff line number Diff line change
Expand Up @@ -248,7 +248,6 @@ def init_surface_recon_wf(
)

if not fs_no_resume:
# fmt:off
workflow.connect([
# Configuration
(inputnode, recon_config, [('t1w', 't1w_list'),
Expand Down Expand Up @@ -282,15 +281,16 @@ def init_surface_recon_wf(
('outputnode.subjects_dir', 'base_directory'),
('outputnode.subject_id', 'container'),
]),
])
# fmt:on
]) # fmt:skip
else:
# Pretend to be the autorecon1 node so fsnative2t1w_xfm gets run ASAP
fs_base_inputs = autorecon1 = pe.Node(nio.FreeSurferSource(), name='fs_base_inputs')

Check warning on line 287 in smriprep/workflows/surfaces.py

View check run for this annotation

Codecov / codecov/patch

smriprep/workflows/surfaces.py#L287

Added line #L287 was not covered by tests

# fmt:off
workflow.connect([

Check warning on line 289 in smriprep/workflows/surfaces.py

View check run for this annotation

Codecov / codecov/patch

smriprep/workflows/surfaces.py#L289

Added line #L289 was not covered by tests
(inputnode, fs_base_inputs, [('subjects_dir', 'subjects_dir'),
('subject_id', 'subject_id')]),
(inputnode, fs_base_inputs, [
('subjects_dir', 'subjects_dir'),
('subject_id', 'subject_id'),
]),
# Generate mid-thickness surfaces
(inputnode, get_surfaces, [
('subjects_dir', 'subjects_dir'),
Expand All @@ -300,10 +300,8 @@ def init_surface_recon_wf(
('subjects_dir', 'base_directory'),
('subject_id', 'container'),
]),
])
# fmt:on
]) # fmt:skip

# fmt:off
workflow.connect([
(get_surfaces, midthickness, [
('white', 'in_file'),
Expand Down

0 comments on commit ba4611b

Please sign in to comment.