Refactor samplesheet #228
Refactor samplesheet #228
Conversation
…from_assembly' and to accept assembler as parameter
Codecov Report
@@ Coverage Diff @@
## dev #228 +/- ##
=======================================
Coverage 27.36% 27.36%
=======================================
Files 45 45
Lines 5314 5314
=======================================
Hits 1454 1454
Misses 3860 3860
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
📝 Add caution for samplesheet about absolute filepaths
| @@ -29,7 +29,7 @@ process ALIGN_READS { | |||
| path "*.version.txt" , emit: version | |||
|
|
|||
| when: | |||
| !meta.cov_from_spades | |||
There was a problem hiding this comment.
I think these when control flows should be handled within the calling workflow/subworkflow because I don't think it's unreasonable to think these modules might be reused for other subworkflows in the future. It also will probably allow defining that flow in a single place
There was a problem hiding this comment.
I agree. Although I'm not sure the when directive can be used within a workflow definition
There was a problem hiding this comment.
…ither workflow 🎨📝 Rename install.rst to installation.rst 🎨📝 Rename running-autometa-nextflow-workflow to autometa-nextflow-workflow 🐛🎨📝🔥 Fix typos and update entrypoint parameters in step-by-step tutorial
…allation to redirect new users
…Add to its own doc
…erly set.. (What a pesky bug this was to find...
Now samplesheet takes input
cov_from_assemblyinstead ofcov_from_contig_headersso in the future, other assembler contig header parsers may be easily added.