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

tagreads broken? #221

Closed
necrolyte2 opened this issue Mar 25, 2016 · 4 comments
Closed

tagreads broken? #221

necrolyte2 opened this issue Mar 25, 2016 · 4 comments
Assignees
Labels
Milestone

Comments

@necrolyte2
Copy link
Member

All reads are being tagged as sanger right now it seems

Not sure why

@necrolyte2 necrolyte2 added the bug label Mar 25, 2016
@necrolyte2 necrolyte2 added this to the AFRIMS Issues milestone Mar 25, 2016
@necrolyte2
Copy link
Member Author

I think the regex that is matching miseq reads may be slightly incorrect or something as some of their samples are working as expected and some do not(they show up as sanger instead of MiSeq)

@necrolyte2
Copy link
Member Author

Example of MiSeq read being tagged as Sanger

M01119:15:AARB3:1:1111:23912:25198 163 CPC-0740_USAMRIID_10807bp 3 60 58M = 3 58 CTGTTGATCTGTGTGAATCAGACTGCGACAGTTCGAGTTTGAAGCGAAAGCTAGCAAC >AAAAFBFFFFFGGGGGGGGGGHHHHGGGGGGHHGHHHGHGFHHGGEEEGHHHHHHDF NM:i:1 AS:i:57 XS:i:0 RG:Z:Sanger

@necrolyte2 necrolyte2 self-assigned this Mar 29, 2016
@averagehat
Copy link
Contributor

While this issue is still open, did you arrive at those regular expressions by examining the ID's yourself, or some other source?
And I noticed sanger is a catch-all--can it be pretty much anything?

@necrolyte2
Copy link
Member Author

So if you Google miseq reads their documentation does not match even our
read names.
So ya, pretty much made the regex based on what I've seen in our read names.
Now, obviously found ours are slightly different than what else is
possible.... Ugg

Sanger names come from whatever the technician specifies so can be anything
ya.

On Tue, Mar 29, 2016, 9:07 PM Mike Panciera [email protected]
wrote:

While this issue is still open, did you arrive at those regular
expressions by examining the ID's yourself, or some other source?
And I noticed sanger is a catch-all--can it be pretty much anything?


You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub
#221 (comment)

averagehat added a commit that referenced this issue Mar 29, 2016
Fixes issue where MiSeq reads were not being tagged correctly(#221)
necrolyte2 added a commit that referenced this issue Apr 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants