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

Add API support for SA tags #2115

Closed
jamesemery opened this issue Aug 25, 2016 · 0 comments · Fixed by #2133
Closed

Add API support for SA tags #2115

jamesemery opened this issue Aug 25, 2016 · 0 comments · Fixed by #2133
Assignees

Comments

@jamesemery
Copy link
Collaborator

The SA tag field for reads is becoming more relevant with the introduction of alternate contigs into references which make use of chimeric alignments to denote reads aligned to an alternate contig. The SA tag, which is necessary to distinguish secondary alignments, is defined in the SAMSpec as follows:

Other canonical alignments in a chimeric alignment, formatted as a semicolon-delimited list: (rname,pos,strand,CIGAR,mapQ,NM ;)+. Each element in the list represents a part of the chimeric alignment. Conventionally, at a supplementary line, the first element points to the primary line.

These are complicated enough to get right that there needs to be code that deals with them as more than just strings. Some of the desired utility is added by htsjdk but this only allows for reading the SA tag, not altering it.

@jamesemery jamesemery self-assigned this Aug 25, 2016
jamesemery added a commit that referenced this issue Aug 31, 2016
lucidtronix pushed a commit that referenced this issue Nov 1, 2016
lucidtronix pushed a commit that referenced this issue Dec 9, 2016
lucidtronix pushed a commit that referenced this issue Dec 9, 2016
… tags from reads

resolves #2115

updated splitNCigarReads integration tests to be consistent with new NM tag
lucidtronix pushed a commit that referenced this issue Dec 9, 2016
… tags from reads (#2133)

* Added an SATagBuilder class that allows for adding and deletion of SA tags from reads

resolves #2115

updated splitNCigarReads integration tests to be consistent with new NM tag

* Rebased and removed expected exception
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant