Add basic subworkflow for unzipping and indexing fasta - #12663
Add basic subworkflow for unzipping and indexing fasta#12663DLBPointon wants to merge 4 commits into
Conversation
| include { GAWK as GAWK_UPPER_SEQUENCE } from '../../../modules/nf-core/gawk/main' | ||
| include { SAMTOOLS_FAIDX } from "../../../modules/nf-core/samtools/faidx/main" | ||
|
|
||
| workflow GUNZIP_FASTA_INDEX { |
There was a problem hiding this comment.
| workflow GUNZIP_FASTA_INDEX { | |
| workflow FASTA_UNZIP_UPPER_GAWK { |
Following the naming guidelines: https://nf-co.re/docs/specifications/components/subworkflows/naming-conventions#name-format-of-subworkflow-files
Anyone else have a better suggestion? I'm not used to the subworkflow naming conventions
There was a problem hiding this comment.
There is a #subworkflows channel on the slack, good to ask there.
I think maybe something more general like FASTA_PREPARE
|
Just spoke with Matthieu Adding a header cleaner is likely a good idea, we already have a tool to do this in ASCC. Could also include some basic information about the fasta too (max_seq_length, total_length, n_seqs) in json format so that we don't pollute the meta. |
|
Maybe we could generate the dict file as well, for the tools that require that. |
A pretty simple pattern i've seen in alot of places is unzipping fasta, cleaning and then index/sizes generation.
Thought I'd make it a subworkflow seeing as it's so common.
Linked to: sanger-tol/nf-core-modules#274
Further dev:
EDIT: Just realised that it's set up for Sanger-tol data. I'll get that sorted.