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

Womtool misses syntax that leads to Cromwell runtime error. #7579

Open
williamrowell opened this issue Nov 15, 2024 · 0 comments
Open

Womtool misses syntax that leads to Cromwell runtime error. #7579

williamrowell opened this issue Nov 15, 2024 · 0 comments

Comments

@williamrowell
Copy link

Tested/linted on:

  • womtool validate v86 and v87
  • cromwell run v86 and v87 with default configuration
  • miniwdl check/run v1.12.1 with minimal configuration
  • wdlTools check v0.17.17
version 1.0

workflow wf {
  input {
    String? name
  }
  output {
    String workflow_name = "hello~{" " + name}"
  }
}

This workflow lints successfully with all tools listed above.

It runs successfully on miniwdl/DNAnexus (after dxCompiler) with or without providing name as an input. On Cromwell, if name is provided, it completes successfully, but without name, it returns the error below.

[2024-11-15 09:46:04,37] [info] WorkflowManagerActor: Workflow b6319060-faec-4206-9fe0-4322f6eb8986 failed (during ExecutingWorkflowState): java.lang.RuntimeException: Failed to evaluate 'wf.workflow_name' (reason 1 of 1): Evaluating "hello~{(" " + name)}" failed: Sorry! Operation + is not supported on empty optional values. You might resolve this using select_first([optional, default]) to guarantee that you have a filled value.

If this syntax is not supported by Cromwell, womtool validate should object.

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

1 participant