FCP-INDI / C-PAC

Configurable Pipeline for the Analysis of Connectomes
https://fcp-indi.github.io/
GNU Lesser General Public License v3.0
64 stars 41 forks source link

T1 and EPI-based reg forking doesn't work when both ANTS + FSL reg forked #1270

Open sgiavasis opened 4 years ago

sgiavasis commented 4 years ago

ANTS & FSL

T1 only, T1 & Off WORKS

ANTS & FSL

T1 & EPI

  File "/code/CPAC/pipeline/cpac_pipeline.py", line 2520, in build_workflow
    'functional_to_epi-standard': (func_to_epi, 'outputspec.func_in_epi'),
  File "/code/CPAC/utils/strategy.py", line 50, in update_resource_pool
    'replacing with %s ' % (key, value)
Exception: Key functional_to_epi-standard already exists in resource pool, replacing with (resting_preproc_1019436_1.func_to_epi_fsl_0, 'outputspec.func_in_epi')

ANTS & FSL

EPI only

  File "/code/CPAC/pipeline/cpac_pipeline.py", line 2520, in build_workflow
    'functional_to_epi-standard': (func_to_epi, 'outputspec.func_in_epi'),
  File "/code/CPAC/utils/strategy.py", line 50, in update_resource_pool
    'replacing with %s ' % (key, value)
Exception: Key functional_to_epi-standard already exists in resource pool, replacing with (resting_preproc_1019436_1.func_to_epi_fsl_0, 'outputspec.func_in_epi')

FSL only

T1 & EPI, T1 & EPI & Off

  File "/code/CPAC/pipeline/cpac_pipeline.py", line 2580, in build_workflow
    func_type='non-ica-aroma')
  File "/code/CPAC/registration/output_func_to_standard.py", line 505, in output_func_to_standard
    map_node=map_node)
  File "/code/CPAC/registration/output_func_to_standard.py", line 53, in fsl_apply_transform_func_to_mni
    func_mni_warp, 'in_file')
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/workflows.py", line 155, in connect
    self._check_nodes(newnodes)
  File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/workflows.py", line 727, in _check_nodes
    'Duplicate node name "%s" found.' % node.name)
OSError: Duplicate node name "func_mni_fsl_warp_functional_brain_mask_to_standard_0" found.

ANTS only

T1 & EPI & Off, T1 & EPI, T1 only, EPI only, T1 & Off, EPI & Off WORKS

FSL only

T1 only, EPI only, T1 & Off, EPI & Off WORKS

sgiavasis commented 3 years ago

I believe this was resolved through the v1.8 refactoring, but I want to confirm during the 1.8.1 cycle.