correctly draw graph when a workflow level input & output share the same name

[mr-c]

https://view.commonwl.org/workflows/github.com/roryk/salmon-cwl/tree/master///salmon-workflow.cwl

See index in the linked workflow

[MarkRobbo]

@mr-c Is this valid as a workflow under the rules of salad-schema?

I had a similar issue #56 earlier and closed it due to a discussion that this is technically possible in cwltool but is actually considered not valid.

[mr-c]

Yes, this is valid and reasonable (a benefit of the scoping rules) Pe 3 apr. 2017 18:17, "Mark Robinson" <notifications@github.com> a scris: @mr-c <https://github.com/mr-c> Is this valid as a workflow under the rules of salad-schema? I had a similar issue #56 <#56> earlier and closed it due to a discussion that this is technically possible in cwltool but is actually considered not valid <common-workflow-language/schema_salad#56>. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#99 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABROCGk1LiD9lycmsHjHl_oAdgtn5mj4ks5rsQ2hgaJpZM4Mwhr1> .

[MarkRobbo]

@mr-c I would like some clarification on this issue. In the current version of schema salad/cwltool the following error is thrown due to the "index" ID being reused

object id `https://cdn.rawgit.com/roryk/salmon-cwl/9772782d041c4d0a180fc89d55c83c31b24bb6af/salmon-workflow.cwl#index` previously defined

Is this a bug in cwltool or is this no longer considered valid?

[mr-c]

Looks like the spec says "unique over the entire document", so an input and an output cannot share the same ID -- so I'll close this issue.