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

filter_reads in DNAmapping pipeline doesnt do the filtering #1023

Open
LeilyR opened this issue Jul 26, 2024 · 3 comments
Open

filter_reads in DNAmapping pipeline doesnt do the filtering #1023

LeilyR opened this issue Jul 26, 2024 · 3 comments

Comments

@LeilyR
Copy link
Contributor

LeilyR commented Jul 26, 2024

Hi,
even though I set dedup and mapq and see the string in the filter_rules, filtered bams are ln -s of the bam files. I had a quick look at the rule, but I could not easily spot an obvious issue there. Did you notice this before? I am using bwa-mem2 btw

@LeilyR
Copy link
Contributor Author

LeilyR commented Jul 26, 2024

OK I found the bug, it is coming from here
Filtering is only defined for bowtie2 not others. I will fix it in my locally installed one. Shall I send you a PR too?

@LeilyR LeilyR closed this as completed Jul 26, 2024
@LeilyR LeilyR reopened this Jul 30, 2024
@LeilyR
Copy link
Contributor Author

LeilyR commented Jul 30, 2024

I reopened this, since it is actually a bug to be fixed. Noticed that I cannot set a label anymore :P so maybe one of you can take care of that.

@katsikora
Copy link
Contributor

Hey Leily,

thanks for posting that issue. I've been mostly testing on bowtie2 which is the default aligner, but I agree we should fix that.
I'm only aware that UMI deduplication only works for bowtie2 sofar, this is also to be fixed for other aligners.

Were you able to open a PR?

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

No branches or pull requests

2 participants