common-workflow-language / user_guide

The CWL v1.0 - v1.2 user guide
http://www.commonwl.org/user_guide/
Other
42 stars 67 forks source link

"Custom Types" & "Specifying Software Requirements" should use a smaller tool #63

Open mr-c opened 6 years ago

mr-c commented 6 years ago

These are lessons 19 & 20

A smaller tool can have a small Docker container and would be more easily used & tested

For "Custom Types", we could use https://github.com/EBI-Metagenomics/ebi-metagenomics-cwl/blob/master/tools/biom-convert.cwl

For "Specifying Software Requirements" we could use https://github.com/EBI-Metagenomics/ebi-metagenomics-cwl/blob/master/tools/nhmmer.cwl

Thanks to @tom-tan for reporting this: https://github.com/common-workflow-language/user_guide/pull/61#issue-283426364

manabuishii commented 6 years ago

A smaller tool can have a small Docker container and would be more easily used & tested

Yes , it is good.

19 is more easy for the readers. I think to innstall interproscan is not easy .

tom-tan commented 5 years ago

Update: Now the section 18 (reordered from section 19: custom types) uses containerized tool. Once the section 19 (reordered from section 20: software requirements) uses smaller tool, we can close this issue.