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

envValue does not accept int #4

Open
bwbioinfo opened this issue Oct 21, 2022 · 1 comment
Open

envValue does not accept int #4

bwbioinfo opened this issue Oct 21, 2022 · 1 comment

Comments

@bwbioinfo
Copy link

Trying to inject an integer into the environment from an input value yields the following:

'envValue expression must evaluate to a str. Got '6' for expression '$(inputs.threads)'.

any minimal .cwl that has includes the following would produce the same error:

requirements:
- class: EnvVarRequirement
  envDef: 
    THREADS: $(inputs.threads)

inputs:
  threads:
    doc: |-
      The number of parallel iterations.
    type: string
    default: 4

As it is "threads" it should not accept anything other than an integer. I can think of a JS workaround converting it to a string but why exactly can't this be other flat atomic values?
I confirmed that it is indeed only accepting string inputs here:
https://www.commonwl.org/v1.0/CommandLineTool.html#EnvVarRequirement

cwltool version: 3.1.20211107152837

@mr-c mr-c transferred this issue from common-workflow-language/cwl-v1.2 Oct 24, 2022
@mr-c
Copy link
Member

mr-c commented Oct 24, 2022

I think accepting integers here is an easy thing to do for CWL v1.3

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

No branches or pull requests

2 participants