Replace tabix module - #329
georgiakes wants to merge 9 commits into
Conversation
|
❌ nf-test failed with latest Nextflow versionNote Tests with Nextflow's latest version failed but it will not cause a CI workflow failure.
See the full run for details. |
| // meta is shared by all test vcfs of a tool and tag, add the file name so each vcf joins with its own index | ||
| TABIX_BGZIPTABIX( | ||
| REFORMAT_HEADER.out.output | ||
| REFORMAT_HEADER.out.output.map { meta, vcf -> [meta + [vcf_name: vcf.name], vcf, [], []] }, |
There was a problem hiding this comment.
why vcf_name is required here?
There was a problem hiding this comment.
I thought is was failing because the meta id would be the same for multiple samples.
I removed it and added groupBy since groupTuple is a legacy operator.
There was a problem hiding this comment.
Hmm it doesn't run with version 25.10.4 though.
There was a problem hiding this comment.
meta.id wont be the same as it is always unique (schema controls it)
Ohhh. Yeah really sorry for that. It will not happen again. |
Closes #323
PR checklist
nf-core pipelines lint).nextflow run . -profile test,docker --outdir <OUTDIR>).nextflow run . -profile debug,test,docker --outdir <OUTDIR>).docs/usage.mdis updated.docs/output.mdis updated.CHANGELOG.mdis updated.README.mdis updated (including new tool citations and authors/contributors).