Problem with merge_bams about analysis-workflows HOT 4 OPEN

The short answer is that the error you're seeing is due to quoting around the default value here:

The longer answer is that the merge_bams.cwl tool as it currently stands only works with Cromwell and not with cwltool. There are some escapes to the bash script that will also be required to make it work with cwltool (but will break it in Cromwell)--everywhere ${...} appears in the script will need to be replaced by \${...} instead.

I'm inclined to replace the bash script here with a perl script to get around this problem so the same CWL works with both runners, but it'll take a bit of time to get that done.

from analysis-workflows.

Great - thanks a lot for explanation! We would like to try to run pipelines using cwl-airflow which is based on cwl-tool. So for now I will try to escape problematic expressions in bash. Do you know if I should expect a lot of issues with other tools/pipelines when running them with cwl-runner instead of Cromwell?

from analysis-workflows.

Thanks again. I wanted to confirm that after escaping "${...}" in few other lines the script is now working with cwl-runner.

from analysis-workflows.

Most tools should be fine. If you run into other issues they're likely to be similar to this--where cromwell's behavior differs from cwltool in small ways. It's our goal to have these workflows also work with cwltool (though we don't currently use it routinely ourselves), so if you do run into more issues like this feel free to let us know!

I've opened #1046 to update this tool to address the boolean value as well as the ${...} in a way that should work with both runners.

from analysis-workflows.