flaxandteal / dewret

DEclarative Workflow REndering Tool
Apache License 2.0
1 stars 0 forks source link

Revisit CWL output of global variables (feedback 1-21) #8

Open philtweir opened 4 months ago

elleryames commented 2 months ago

Was this addressed in a specific PR, and if so which one?

elleryames commented 2 months ago

Given #27 do we need global variables at all (as in being injected inside functions, a-la https://github.com/flaxandteal/dewret/discussions/20 )?

elleryames commented 2 months ago

The original issue in which the value of the global variable was not listed in the CWL workflow input parameters has no been fixed in #27. A remaining question is whether the CWL step having two parameters listed despite the corresponding python task only having one, is intended.

Related: #23