search

broadinstitute / cromwell

Scientific workflow engine designed for simplicity & scalability. Trivially transition between one off use cases to massive scale production environments
http://cromwell.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
995 stars 360 forks source link

CWL Workflow name #2728

Closed katevoss closed 7 years ago

katevoss commented 7 years ago

@Horneth commented on Mon Sep 11 2017

The CWL parser currently hardcode the same name for all workflows. Find out if CWL has a way to specify a workflow name. If not use a hash ? A new endpoint parameter ? @katevoss this could use PO input if it turns out CWL has no way to specify a workflow name

@danbills commented on Tue Oct 03 2017

So it turns out that CWL v1.0.2 has an id field for Workflow, and cwltool gives us one for free:

http://www.commonwl.org/v1.0/Workflow.html#Workflow.

so we should update the model to have a required id: String and this problem goes away.

@danbills commented on Tue Oct 03 2017

This also applies to CommandLineTool, cwltool pre-processing gives us one.

katevoss commented 7 years ago

Is this still to-do?

Horneth commented 7 years ago

yep

cjllanwarne commented 7 years ago

I'm picking this up as a hopefully gentle "intro to CWL stuff" ticket