Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change input binding to arguments in example files #332

Open
wants to merge 24 commits into
base: main
Choose a base branch
from

Conversation

Fienne
Copy link
Contributor

@Fienne Fienne commented Oct 23, 2022

fixes #34

@swzCuroverse
Copy link
Contributor

@Fienne -- so, this isn't how arguments work. You can't just unfortunately change input finding to arguments and how it work the same way. Unfortunately there isn't a really good section on arguments. Let me look and find some resources.-

@swzCuroverse
Copy link
Contributor

You can see an explain here (some code that a co-worker wrote):
https://github.com/arvados/l7g-ml/blob/main/PhasingImputation/Phasing/eagle.cwl

@swzCuroverse
Copy link
Contributor

@tetron thoughts on this ticket
do you have information about when to up inputBinding vs arguments and some resources? I will admit I always use inputBinding.

@Fienne
Copy link
Contributor Author

Fienne commented Oct 24, 2022

You can see an explain here (some code that a co-worker wrote): https://github.com/arvados/l7g-ml/blob/main/PhasingImputation/Phasing/eagle.cwl

Thank you @swzCuroverse . I'll need to take sometime on this one 😅

@tetron
Copy link
Member

tetron commented Oct 24, 2022

The interaction between inputBinding and arguments is described here:

https://www.commonwl.org/v1.2/CommandLineTool.html#Input_binding

The arguments field is separate from inputs, but has the same inputBinding type.

arguments is usually simpler, it is a plain list of command line arguments and values more like you would write in other programming languages. However it separates the definition of the input parameter from the definition of command line parameter it corresponds to, so if you have a large number of possible options (some bioinformatics tools have dozens of flags) can be more maintainable to define them together.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

switch from inputBinding to arguments: based syntax
3 participants