Skip to content

Commit 2f5b2aa

Browse files
committed
fixed white-space and extra indentations
1 parent 24f7540 commit 2f5b2aa

File tree

2 files changed

+54
-44
lines changed

2 files changed

+54
-44
lines changed

CPAC/nuisance/nuisance.py

Lines changed: 34 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -1714,7 +1714,7 @@ def ICA_AROMA_FSLreg(wf, cfg, strat_pool, pipe_num, opt=None):
17141714
"inputs": [["desc-preproc_bold", "bold"],
17151715
"from-bold_to-T1w_mode-image_desc-linear_xfm",
17161716
"from-T1w_to-template_mode-image_xfm"],
1717-
"outputs": ["desc-preproc_bold",
1717+
"outputs": ["desc-preproc_bold",
17181718
"desc-cleaned_bold"]}
17191719
'''
17201720

@@ -1864,8 +1864,8 @@ def ICA_AROMA_FSLEPIreg(wf, cfg, strat_pool, pipe_num, opt=None):
18641864
}
18651865

18661866
return (wf, outputs)
1867-
1868-
1867+
1868+
18691869
def ICA_AROMA_ANTsEPIreg(wf, cfg, strat_pool, pipe_num, opt=None):
18701870
'''
18711871
Node Block:
@@ -1891,7 +1891,7 @@ def ICA_AROMA_ANTsEPIreg(wf, cfg, strat_pool, pipe_num, opt=None):
18911891

18921892
if reg_tool != 'ants':
18931893
return (wf, None)
1894-
1894+
18951895
num_cpus = cfg.pipeline_setup['system_config'][
18961896
'max_cores_per_participant']
18971897

@@ -1928,7 +1928,7 @@ def ICA_AROMA_ANTsEPIreg(wf, cfg, strat_pool, pipe_num, opt=None):
19281928

19291929
node, out = strat_pool.get_data('from-EPItemplate_to-bold_mode-image_xfm')
19301930
wf.connect(node, out, apply_xfm, 'inputspec.transform')
1931-
1931+
19321932
outputs = {
19331933
'desc-preproc_bold': (apply_xfm, 'outputspec.output_image'),
19341934
'desc-cleaned_bold': (apply_xfm, 'outputspec.output_image')
@@ -1962,7 +1962,7 @@ def erode_mask_T1w(wf, cfg, strat_pool, pipe_num, opt=None):
19621962

19631963
node, out = strat_pool.get_data('space-T1w_desc-brain_mask')
19641964
wf.connect(node, out, erode, 'inputspec.brain_mask')
1965-
1965+
19661966
node, out = strat_pool.get_data(['label-CSF_desc-preproc_mask',
19671967
'label-CSF_mask'])
19681968
wf.connect(node, out, erode, 'inputspec.mask')
@@ -2077,15 +2077,15 @@ def erode_mask_WM(wf, cfg, strat_pool, pipe_num, opt=None):
20772077
erode.inputs.inputspec.erode_prop = cfg.nuisance_corrections[
20782078
'2-nuisance_regression']['regressor_masks']['erode_wm'][
20792079
'wm_erosion_prop']
2080-
2080+
20812081
erode.inputs.inputspec.mask_erode_mm = cfg.nuisance_corrections[
20822082
'2-nuisance_regression']['regressor_masks']['erode_wm'][
20832083
'wm_mask_erosion_mm']
20842084

20852085
node, out = strat_pool.get_data(['label-WM_desc-preproc_mask',
20862086
'label-WM_mask'])
20872087
wf.connect(node, out, erode, 'inputspec.mask')
2088-
2088+
20892089
node, out = strat_pool.get_data('space-T1w_desc-brain_mask')
20902090
wf.connect(node, out, erode, 'inputspec.brain_mask')
20912091

@@ -2094,8 +2094,8 @@ def erode_mask_WM(wf, cfg, strat_pool, pipe_num, opt=None):
20942094
}
20952095

20962096
return (wf, outputs)
2097-
2098-
2097+
2098+
20992099
def nuisance_regressors_generation(wf, cfg, strat_pool, pipe_num, opt=None):
21002100
'''
21012101
Node Block:
@@ -2130,9 +2130,11 @@ def nuisance_regressors_generation(wf, cfg, strat_pool, pipe_num, opt=None):
21302130
'from-template_to-T1w_mode-image_desc-linear_xfm')
21312131
reg_tool = check_prov_for_regtool(xfm_prov)
21322132
use_ants = reg_tool == 'ants'
2133-
2133+
21342134
ventricle = strat_pool.check_rpool('lateral-ventricles-mask')
2135-
csf_mask = strat_pool.check_rpool(["label-CSF_desc-eroded_mask", "label-CSF_desc-preproc_mask", "label-CSF_mask"])
2135+
csf_mask = strat_pool.check_rpool(["label-CSF_desc-eroded_mask",
2136+
"label-CSF_desc-preproc_mask",
2137+
"label-CSF_mask"])
21362138

21372139
regressors = create_regressor_workflow(opt, use_ants,
21382140
ventricle_mask_exist=ventricle,
@@ -2161,7 +2163,7 @@ def nuisance_regressors_generation(wf, cfg, strat_pool, pipe_num, opt=None):
21612163
"label-CSF_desc-preproc_mask",
21622164
"label-CSF_mask"]):
21632165
node, out = strat_pool.get_data(["label-CSF_desc-eroded_mask",
2164-
"label-CSF_desc-preproc_mask",
2166+
"label-CSF_desc-preproc_mask",
21652167
"label-CSF_mask"])
21662168
wf.connect(node, out, regressors, 'inputspec.csf_mask_file_path')
21672169

@@ -2174,10 +2176,10 @@ def nuisance_regressors_generation(wf, cfg, strat_pool, pipe_num, opt=None):
21742176
wf.connect(node, out, regressors, 'inputspec.wm_mask_file_path')
21752177

21762178
if strat_pool.check_rpool(["label-GM_desc-eroded_mask",
2177-
"label-GM_desc-preproc_mask",
2179+
"label-GM_desc-preproc_mask",
21782180
"label-GM_mask"]):
21792181
node, out = strat_pool.get_data(["label-GM_desc-eroded_mask",
2180-
"label-GM_desc-preproc_mask",
2182+
"label-GM_desc-preproc_mask",
21812183
"label-GM_mask"])
21822184
wf.connect(node, out, regressors, 'inputspec.gm_mask_file_path')
21832185

@@ -2253,10 +2255,10 @@ def nuisance_regression(wf, cfg, strat_pool, pipe_num, opt=None):
22532255
"desc-cleaned_bold",
22542256
"regressors"]}
22552257
'''
2256-
2258+
22572259
regressor_prov = strat_pool.get_cpac_provenance('regressors')
22582260
regressor_strat_name = regressor_prov[-1].split('_')[-1]
2259-
2261+
22602262
for regressor_dct in cfg['nuisance_corrections']['2-nuisance_regression'][
22612263
'Regressors']:
22622264
if regressor_dct['Name'] == regressor_strat_name:
@@ -2316,7 +2318,7 @@ def nuisance_regression(wf, cfg, strat_pool, pipe_num, opt=None):
23162318

23172319
elif cfg.nuisance_corrections['2-nuisance_regression'][
23182320
'bandpass_filtering_order'] == 'Before':
2319-
2321+
23202322
node, out = strat_pool.get_data("desc-preproc_bold")
23212323
wf.connect(node, out, filt, 'inputspec.functional_file_path')
23222324

@@ -2332,7 +2334,7 @@ def nuisance_regression(wf, cfg, strat_pool, pipe_num, opt=None):
23322334
else:
23332335
node, out = strat_pool.get_data("desc-preproc_bold")
23342336
wf.connect(node, out, nuis, 'inputspec.functional_file_path')
2335-
2337+
23362338
outputs = {
23372339
'desc-preproc_bold': (nuis, 'outputspec.residual_file_path'),
23382340
'desc-cleaned_bold': (nuis, 'outputspec.residual_file_path')
@@ -2366,7 +2368,7 @@ def erode_mask_bold(wf, cfg, strat_pool, pipe_num, opt=None):
23662368

23672369
node, out = strat_pool.get_data('space-bold_desc-brain_mask')
23682370
wf.connect(node, out, erode, 'inputspec.brain_mask')
2369-
2371+
23702372
node, out = strat_pool.get_data(['space-bold_label-CSF_desc-preproc_mask',
23712373
'space-bold_label-CSF_mask'])
23722374
wf.connect(node, out, erode, 'inputspec.mask')
@@ -2376,8 +2378,8 @@ def erode_mask_bold(wf, cfg, strat_pool, pipe_num, opt=None):
23762378
}
23772379

23782380
return (wf, outputs)
2379-
2380-
2381+
2382+
23812383
def erode_mask_boldCSF(wf, cfg, strat_pool, pipe_num, opt=None):
23822384
'''
23832385
{"name": "erode_mask_boldCSF",
@@ -2481,15 +2483,15 @@ def erode_mask_boldWM(wf, cfg, strat_pool, pipe_num, opt=None):
24812483
erode.inputs.inputspec.erode_prop = cfg.nuisance_corrections[
24822484
'2-nuisance_regression']['regressor_masks']['erode_wm'][
24832485
'wm_erosion_prop']
2484-
2486+
24852487
erode.inputs.inputspec.mask_erode_mm = cfg.nuisance_corrections[
24862488
'2-nuisance_regression']['regressor_masks']['erode_wm'][
24872489
'wm_mask_erosion_mm']
24882490

24892491
node, out = strat_pool.get_data(['space-bold_label-WM_desc-preproc_mask',
24902492
'space-bold_label-WM_mask'])
24912493
wf.connect(node, out, erode, 'inputspec.mask')
2492-
2494+
24932495
node, out = strat_pool.get_data('space-bold_desc-brain_mask')
24942496
wf.connect(node, out, erode, 'inputspec.brain_mask')
24952497

@@ -2516,7 +2518,7 @@ def nuisance_regressors_generation_EPItemplate(wf, cfg, strat_pool, pipe_num, op
25162518
"framewise-displacement-power",
25172519
"dvars",
25182520
["space-bold_desc-eroded_mask", "space-bold_desc-brain_mask"],
2519-
["space-bold_label-CSF_desc-eroded_mask", "space-bold_label-CSF_desc-preproc_mask",
2521+
["space-bold_label-CSF_desc-eroded_mask", "space-bold_label-CSF_desc-preproc_mask",
25202522
"space-bold_label-CSF_mask"],
25212523
["space-bold_label-WM_desc-eroded_mask", "space-bold_label-WM_desc-preproc_mask",
25222524
"space-bold_label-WM_mask"],
@@ -2535,7 +2537,9 @@ def nuisance_regressors_generation_EPItemplate(wf, cfg, strat_pool, pipe_num, op
25352537

25362538
use_ants = reg_tool == 'ants'
25372539
ventricle = strat_pool.check_rpool('lateral-ventricles-mask')
2538-
csf_mask = strat_pool.check_rpool([ "space-bold_label-CSF_desc-eroded_mask", "space-bold_label-CSF_desc-preproc_mask", "space-bold_label-CSF_mask"])
2540+
csf_mask = strat_pool.check_rpool(["space-bold_label-CSF_desc-eroded_mask",
2541+
"space-bold_label-CSF_desc-preproc_mask",
2542+
"space-bold_label-CSF_mask"])
25392543

25402544
regressors = create_regressor_workflow(opt, use_ants,
25412545
ventricle_mask_exist=ventricle,
@@ -2596,15 +2600,15 @@ def nuisance_regressors_generation_EPItemplate(wf, cfg, strat_pool, pipe_num, op
25962600
wf.connect(node, out, regressors, 'inputspec.anat_to_mni_linear_xfm_file_path')
25972601
wf.connect(node, out, regressors, 'inputspec.func_to_anat_linear_xfm_file_path')
25982602

2599-
if strat_pool.check_rpool('movement-parameters'):
2603+
if strat_pool.check_rpool('movement-parameters'):
26002604
node, out = strat_pool.get_data('movement-parameters')
26012605
wf.connect(node, out, regressors, 'inputspec.motion_parameters_file_path')
26022606

2603-
if strat_pool.check_rpool('framewise-displacement-jenkinson'):
2607+
if strat_pool.check_rpool('framewise-displacement-jenkinson'):
26042608
node, out = strat_pool.get_data('framewise-displacement-jenkinson')
26052609
wf.connect(node, out, regressors, 'inputspec.fd_j_file_path')
26062610

2607-
if strat_pool.check_rpool('framewise-displacement-power'):
2611+
if strat_pool.check_rpool('framewise-displacement-power'):
26082612
node, out = strat_pool.get_data('framewise-displacement-power')
26092613
wf.connect(node, out, regressors, 'inputspec.fd_p_file_path')
26102614

@@ -2619,4 +2623,4 @@ def nuisance_regressors_generation_EPItemplate(wf, cfg, strat_pool, pipe_num, op
26192623
'regressors': (regressors, 'outputspec.regressors_file_path')
26202624
}
26212625

2622-
return (wf, outputs)
2626+
return (wf, outputs)

CPAC/nuisance/utils/__init__.py

Lines changed: 20 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -323,14 +323,13 @@ def generate_summarize_tissue_mask(nuisance_wf,
323323

324324
if all_bold:
325325
pass
326-
326+
327327
if csf_mask_exist:
328328
mask_to_epi = pe.Node(interface=fsl.FLIRT(),
329-
name='{}_flirt'
330-
.format(node_mask_key),
331-
mem_gb=3.63,
332-
mem_x=(3767129957844731 / 1208925819614629174706176,
333-
'in_file'))
329+
name='{}_flirt'.format(node_mask_key),
330+
mem_gb=3.63,
331+
mem_x=(3767129957844731 / 1208925819614629174706176,
332+
'in_file'))
334333

335334
mask_to_epi.inputs.interp = 'nearestneighbour'
336335

@@ -445,13 +444,19 @@ def generate_summarize_tissue_mask_ventricles_masking(nuisance_wf,
445444

446445
nuisance_wf.connect(*(pipeline_resource_pool['Ventricles'] + (lat_ven_mni_to_anat, 'input_image')))
447446
resolution = regressor_selector['extraction_resolution']
448-
447+
449448
if csf_mask_exist:
450-
nuisance_wf.connect(*(pipeline_resource_pool[mask_key] + (lat_ven_mni_to_anat, 'reference_image')))
449+
nuisance_wf.connect(*(
450+
pipeline_resource_pool[mask_key] +
451+
(lat_ven_mni_to_anat, 'reference_image')))
451452
elif resolution == 'Functional':
452-
nuisance_wf.connect(*(pipeline_resource_pool['Functional_mean'] + (lat_ven_mni_to_anat, 'reference_image')))
453+
nuisance_wf.connect(*(
454+
pipeline_resource_pool['Functional_mean'] +
455+
(lat_ven_mni_to_anat, 'reference_image')))
453456
else:
454-
nuisance_wf.connect(*(pipeline_resource_pool['Anatomical_{}mm'.format(resolution)] + (lat_ven_mni_to_anat, 'reference_image')))
457+
nuisance_wf.connect(*(
458+
pipeline_resource_pool['Anatomical_{}mm'.format(resolution)] +
459+
(lat_ven_mni_to_anat, 'reference_image')))
455460

456461
pipeline_resource_pool[ventricles_key] = (lat_ven_mni_to_anat, 'output_image')
457462

@@ -469,18 +474,19 @@ def generate_summarize_tissue_mask_ventricles_masking(nuisance_wf,
469474

470475
if csf_mask_exist:
471476
# reduce CSF mask to the lateral ventricles
472-
mask_csf_with_lat_ven = pe.Node(interface=afni.Calc(outputtype='NIFTI_GZ'), name='{}_Ventricles'.format(mask_key))
477+
mask_csf_with_lat_ven = pe.Node(interface=afni.Calc(outputtype='NIFTI_GZ'),
478+
name='{}_Ventricles'.format(mask_key))
473479
mask_csf_with_lat_ven.inputs.expr = 'a*b'
474480
mask_csf_with_lat_ven.inputs.out_file = 'csf_lat_ven_mask.nii.gz'
475-
481+
476482
nuisance_wf.connect(*(pipeline_resource_pool[ventricles_key] + (mask_csf_with_lat_ven, 'in_file_a')))
477483
nuisance_wf.connect(*(pipeline_resource_pool[mask_key] + (mask_csf_with_lat_ven, 'in_file_b')))
478484

479485
pipeline_resource_pool['{}_Unmasked'.format(mask_key)] = pipeline_resource_pool[mask_key]
480486
pipeline_resource_pool[mask_key] = (mask_csf_with_lat_ven, 'out_file')
481487

482488
else:
483-
pipeline_resource_pool[mask_key] = pipeline_resource_pool[ventricles_key]
489+
pipeline_resource_pool[mask_key] = pipeline_resource_pool[ventricles_key]
484490

485491
return pipeline_resource_pool
486492

@@ -694,4 +700,4 @@ def encode(selector):
694700
return "_".join(selectors_representations)
695701

696702
def __repr__(self):
697-
return NuisanceRegressor.encode(self.selector)
703+
return NuisanceRegressor.encode(self.selector)

0 commit comments

Comments
 (0)