Pipeline will crash if pipeline name is set to an integer in the YAML config

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

Pipeline will crash if pipeline name is set to an integer in the YAML config #503

Open jpellman opened 9 years ago

jpellman commented 9 years ago

If you set the pipeline name to just a number with no other characters, it looks like the YAML parser will automatically parse this as an int. Then at line 5879 in cpac_pipeline.py (prep_workflow) you will get this error when you try to concatenate the pipeline name to a fork label:

pipeline_id = c.pipelineName + forkPointsDict[strat] TypeError: unsupported operand type(s) for +: 'int' and 'str'

nx10 commented 1 year ago

An unquoted numeric literal is a number in YAML. However we could consider converting numbers to strings in this case.