Skip to content

Add option to turn off bowtie analyses#149

Open
percyfal wants to merge 2 commits into
mainfrom
bowtie-false
Open

Add option to turn off bowtie analyses#149
percyfal wants to merge 2 commits into
mainfrom
bowtie-false

Conversation

@percyfal
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Contributor

@LeandroRitter LeandroRitter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@ZoePochon
Copy link
Copy Markdown
Collaborator

I think this PR would be worth reviving, because this is essentially what issue #140 was aiming for: being able to skip Bowtie2 when we only need the screening/classification outputs. It is also something that I would like to be able to do on a regular basis. This would save a lot of unnecessary compute, especially since Bowtie2 mapping to the full microbial database can be heavy. Since MapDamage depends on the Bowtie2 BAMs, it makes sense that bowtie: false also skips MapDamage, but this behaviour should probably be made explicit in the config/docs. Maybe we could rebase this PR against current main and test both bowtie: true and bowtie: false?

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

Successfully merging this pull request may close these issues.

3 participants