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.

Sign up for GitHub

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

Jump to bottom

specifying float or int inputs on command line to cwltool doesn't work #342

Open
2 tasks
mr-c opened this issue Mar 22, 2017 · 1 comment
Open
2 tasks

specifying float or int inputs on command line to cwltool doesn't work #342

mr-c opened this issue Mar 22, 2017 · 1 comment

Comments

@mr-c
Copy link
Member

mr-c commented Mar 22, 2017

  • fix it
  • add a test

make-instructions.cwl

#!/usr/bin/env cwl-runner

cwlVersion: v1.0
class: CommandLineTool

inputs:
  numbers:
    type: int[]
    inputBinding:
      itemSeparator: "+"

baseCommand: [ echo ]

outputs:
 response: stdout
(env) michael@dan:~/workshop$ cwltool make-instructions.cwl  --numbers 1
/home/michael/workshop/env/bin/cwltool 1.0.20170309164828
Resolved 'make-instructions.cwl' to 'file:///home/michael/workshop/make-instructions.cwl'
invalid field `job_order`, expected one of: 'numbers'
Workflow error, try again with --debug for more information:
Invalid job input record:
the `numbers` field is not valid because
  item is invalid because
    `'1'` is not int
qiukunlong pushed a commit to qiukunlong/cwltool that referenced this issue Mar 25, 2017
@mr-c
Squashed 'cwltool/schemas/' changes from 1550753..aa320ec
65f95a3 
aa320ec Merge pull request common-workflow-language#342 from common-workflow-language/fix-mystery-package-errors
3c6eaff Update for rename of argparse2cwl to argparse2tool
b14404b remove errant jsonldPredicate
7f116da Merge pull request common-workflow-language#339 from common-workflow-language/dockerOutputDirectory
4f65d1f Add test for dockerOutputDirectory option of DockerRequirement.
cb1f928 Merge pull request common-workflow-language#335 from common-workflow-language/output-literals
d8b6c8f Add tests for file and directory literals in output as produced by expression tool.
ee3bb1c Merge pull request common-workflow-language#324 from common-workflow-language/secondaryfiles-array
5e6c2d4 Add NLeSC's Xenon
cbe1c8d Merge pull request common-workflow-language#334 from StarvingMarvin/master
8300e43 output ids must start with '#' in draft-2
f87d218 draft-3 test fixes: explicit output for args
8b10319 Fixes for draft-2 tests: adding explicit args output
83ed4b4 Merge remote-tracking branch 'rabix-cwl/master'
e0a5e7c Merge pull request common-workflow-language#328 from StarvingMarvin/master
d906c4a dir4 test fix
21bdcaf Merge remote-tracking branch 'upstream/master'
dee0f4e Disabled checksum check for listing input dir
fc4cb0f Added args output and empty.json for null jobs
a4950f4 Update tools list
8e1d67c Add Rabix Bunny CI badge
2a5b95e fixing mentions to draft3/4
1356c45 Add test for secondaryFiles with arrays
b60125b Single instead of double quotes in doc
179835b Merge pull request common-workflow-language#319 from common-workflow-language/test-cwl-out2
ed40737 Add additional test for cwl.output.json behavior.
a217bb6 Merge pull request common-workflow-language#317 from StarvingMarvin/master
8dd69f2 fixing test in v1.0 and v1.1.0-dev1
e831c20 Merge pull request common-workflow-language#314 from common-workflow-language/test-requirements-on-steps
8cad6f1 Add test for requirements/hints on workflow steps.
7679adb Merge pull request common-workflow-language#308 from common-workflow-language/embedded-subworkflow-test
1582677 Tighten up formatting
779c2d4 Add test for embedded subworkflow
e22a9c2 Merge pull request common-workflow-language#307 from alaindomissy/patch-3
125cb5f Update UserGuide.yml
99f4f9f Merge pull request common-workflow-language#303 from common-workflow-language/include-stdin-in-docs
5324dbb put stdin shortcut in correct place

git-subtree-dir: cwltool/schemas
git-subtree-split: aa320ec
tetron pushed a commit that referenced this issue Jul 23, 2017
@mr-c
Merge pull request #342 from common-workflow-language/fix-mystery-pac…
aa320ec 
…kage-errors

remove errant jsonldPredicate
@manu-chroma
Copy link
Member

Related issue: #385

@susannasiebert susannasiebert mentioned this issue Oct 30, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mr-c @manu-chroma