connor-lab / ncov2019-artic-nf

A Nextflow pipeline for running the ARTIC network's fieldbioinformatics tools (https://github.com/artic-network/fieldbioinformatics), with a focus on ncov2019
GNU Affero General Public License v3.0
89 stars 89 forks source link

Remove unnecessary samtools view -bS #48

Closed ac55-sanger closed 4 years ago

ac55-sanger commented 4 years ago

Samtools sort has been able to read more than BAM for a long time, hence "samtools view -bS | samtools sort ..." is just an unnecessary CPU consuming command.

I tested both the outputs (with and without 'samtools view -bS') and results are same for both.