Unexpected behavior with optional plugs

[servoz]

Concrete rather than long speeches:

• In Mia, drag and drop a process with optional parameter, ex. mia_processes.bricks.preprocess.fsl.Segment. Segments and output_type are optional.
• Export all unconnected plugs.
• Pipeline > Save pipeline as: give a name ex. test10
• We observe the new Test10 brick in the pipeline Packages.
• Drag and drop this new Test10 brick in the pipeline editor.
• Because segments and output_type are optional, only do Export unconnected mandatory plugs.
• Run the pipeline.
• We observe Pipeline was not initialised successfully.

Error during initialisation of the "NoName" pipeline ...! Traceback: File "/data/Git_Projects/populse_mia/python/populse_mia/user_interface/pipeline_manager/pipeline_manager_tab.py", line 1464, in initialize self.test_init = self.init_pipeline() File "/data/Git_Projects/populse_mia/python/populse_mia/user_interface/pipeline_manager/pipeline_manager_tab.py", line 1544, in init_pipeline self.workflow = workflow_from_pipeline( File "/data/Git_Projects/capsul/capsul/pipeline/pipeline_workflow.py", line 1594, in workflow_from_pipeline = workflow_from_pipeline_structure( File "/data/Git_Projects/capsul/capsul/pipeline/pipeline_workflow.py", line 1454, in workflow_from_pipeline_structure sjob = jobs[process] KeyError: <User_processes.test10.Test10 object at 0x7f052ff8cef0>

I haven't had time to look at the capsul.pipeline.pipeline_workflow module yet to figure out what's going on and I don't know if this is the way it's supposed to work in caspul, but I think it's not desirable. In my opinion an optional plug should not necessarily be connected.

EDIT: mia_processes.bricks.preprocess.fsl.Segment is only available in the mriqcPlusReport branch of mia_processes

[servoz]

The last commit in capsul by @denisri fix this issue. I close this ticket !