Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow BWA-MEM as an alternative mapper #106

Closed
marcelm opened this issue Sep 26, 2019 · 1 comment
Closed

Allow BWA-MEM as an alternative mapper #106

marcelm opened this issue Sep 26, 2019 · 1 comment
Assignees

Comments

@marcelm
Copy link
Collaborator

marcelm commented Sep 26, 2019

I’ve benchmarked mapping 20 million reads against GRCh38 both with Bowtie2 and BWA-MEM.

These are the wall-clock runtimes (each mapper was run three times):

  • BWA-MEM 0.7.17 (from Bioconda): 33m 30s, 33m 23s, 33m 31s
  • Bowtie2 2.3.5 (from Bioconda): 43m 33s, 43m 22s, 43m 28s

CPU time was 492 minutes vs. 687 minutes.

I’d like to add BWA-MEM as an alternative mapper to the Snakefile. We can make it configurable, but given the numbers, BWA-MEM should probably be the default.

@marcelm marcelm self-assigned this Sep 26, 2019
@pontushojer
Copy link
Collaborator

Yes BWA-mem is nice. I runned some small test with it and it also looked nice. I have previously also done some testing with the Emerald aligner. This is also uses on BWA-mem but is made for aligning barcoded short read such as 10x Genomics. It has some nice correction features that could be interesting to compare to both BWA and Bowtie2.

marcelm added a commit that referenced this issue Sep 27, 2019
This introduces a new configuration setting named read_mapper, and
also renames the bowtie2_reference setting to genome_reference.

Close #106
marcelm added a commit that referenced this issue Sep 27, 2019
This introduces a new configuration setting named read_mapper, and
also renames the bowtie2_reference setting to genome_reference.

Close #106
marcelm added a commit that referenced this issue Sep 27, 2019
This introduces a new configuration setting named read_mapper, and
also renames the bowtie2_reference setting to genome_reference.

Close #106
marcelm added a commit that referenced this issue Sep 27, 2019
This introduces a new configuration setting named read_mapper, and
also renames the bowtie2_reference setting to genome_reference.

Close #106
pontushojer pushed a commit that referenced this issue Oct 10, 2019
This introduces a new configuration setting named read_mapper, and
also renames the bowtie2_reference setting to genome_reference.

Close #106
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants