Skip to content
This repository has been archived by the owner on Jun 9, 2023. It is now read-only.

Update code to work with latest cwltool #24

Open
jsunny23 opened this issue Jun 29, 2018 · 1 comment
Open

Update code to work with latest cwltool #24

jsunny23 opened this issue Jun 29, 2018 · 1 comment

Comments

@jsunny23
Copy link

Cwltool made some updates to their code which makes it incompatible with dx-cwl. For instance, here where they changed the function name from defaultMakeTool to default_make_tool.

There are other places where the code is now broken, due to the cwltool changes.

@skchronicles
Copy link

skchronicles commented Nov 29, 2018

Work-around

I am also obsveing the same error as well. If youpip install cwltool==1.0.20180225105849, an older release of cwltools from Feburary 28th 2018, it works!

I will also point out that is not the only problem. The README.mdtestfiles described here do not exist.
I ended up using a test cwl file in this location:
/path/to/git/repo/dx-cwl/tests/md5sum/md5sum.cwl

That being said, this command seemed to do the trick:

python dx-cwl compile-workflow tests/md5sum/md5sum.cwl  --token yourAUTHTOKEN --project yourPROJECTID

That being said, it may be time to update the install portion of the README.md.

Here are the steps I took for installation:

  1. virtualenv dxenv
  2. source dxenv/bin/activate
  3. pip install --upgrade pip
  4. pip install cwltool==1.0.20180225105849
  5. pip install PyYAML
  6. pip install dxpy

I am not sure why you are running that get-cwltool.sh script since you are already pip installing cwltool (seems unnecessary).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants