-
Notifications
You must be signed in to change notification settings - Fork 20
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
Any way to ignore PL in bam? #112
Comments
I think you will have to rewrite the bam headers. Luckily samtools makes that quite easy. Use https://www.htslib.org/doc/samtools-view.html |
thanks. i think an option to override this sort of check would be nice in a future release. does the given platform affect the analysis results? if not, seems like this check could be performed by the user. |
That depends on the difference being seen. If you are talking about Illumina X10 v's Illumina Novaseq then it probably won't make much difference. But if you are comparing Illumina to ONT data it would have a significant impact. In our internal system we tend to populate Platform with the vendor (Illumina, ONT, PacBio) not a specific sequencing machine model. |
yes, but brass doesn't behave differently whether i have |
We have a few paired samples that for some reason have different platforms in each bam of a given pair. As a result we got the following error:
They are actually the same platform but the labeling is different. Based on the code
Implement.pm
I don't think using the-pl
command line option will work. Is there any way to make brass run without rewriting the bams?The text was updated successfully, but these errors were encountered: