-
Notifications
You must be signed in to change notification settings - Fork 354
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
QC metrics #35
Comments
Paul; More generally, can you remind me what is holding up the move to variant2 from your perspective? I am thinking about deprecating and removing the older pipeline, so want to understand what features it has that variant2 lacks. |
Hi Brad, no problem. I should have looked into the issue as well, but a bit occupied I'm switching over to variant2 pipeline as well. But I kind of want to I am stuck at the GEMINI QC metrics right now, but I also tried to use GATK Thanks again for all of your help. Paul On Mon, Jul 8, 2013 at 8:52 AM, Brad Chapman notifications@github.comwrote:
|
and i thought you have special reason to remove the duplicates from the bam. Your fix is the way I used to by-pass the issue. |
(sorry, I am still using variant, instead of varinat2 pipeline)
duplicates metrics is not correctly reported since it's trying to get this information from the recalibrated realigned bam (which has already had duplicates removed). I would actually prefer 'mark duplicates' to 'duplicates removal' in the earlier step
The text was updated successfully, but these errors were encountered: