cwl-tes icon indicating copy to clipboard operation
cwl-tes copied to clipboard

Upgraded version of cwltool to 3

Open zagganas opened this issue 2 years ago • 3 comments

Upgraded the version of cwltool to support CWL 1.2 and made relevant changes.

zagganas avatar Jun 01 '22 09:06 zagganas

Looks like after the most recent merge of 0.4.0 branch, you'll need to re-do the requirements. The minio client isn't installed.

kellrott avatar Jun 01 '22 20:06 kellrott

It looks like that between versions of CWL, the order of the tests was changed. The line https://github.com/ohsu-comp-bio/cwl-tes/blob/master/tests/test_conformance.py#L24 needs to be updated to skip the test with writable outputs. We'll have to address that issue, as that requirement is not currently supported by TES ( https://github.com/ga4gh/task-execution-schemas/issues/18 )

kellrott avatar Jun 13 '22 14:06 kellrott

It looks like that between versions of CWL, the order of the tests was changed. The line https://github.com/ohsu-comp-bio/cwl-tes/blob/master/tests/test_conformance.py#L24 needs to be updated to skip the test with writable outputs. We'll have to address that issue, as that requirement is not currently supported by TES ( ga4gh/task-execution-schemas#18 )

You can now skip tests by name, instead of number: https://github.com/common-workflow-language/common-workflow-language/blob/efe30c092a29924ebe00f7a0edf3a413a9bdb1a7/run_test.sh#L17

mr-c avatar Dec 15 '22 15:12 mr-c