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

Updating pileup conversion code to convert sequences that use the X and = (EQ) CIGAR operators #199

Merged
merged 1 commit into from
Apr 2, 2014

Conversation

fnothaft
Copy link
Member

@fnothaft fnothaft commented Apr 2, 2014

This update patches the pileup converter to allow for the conversion of reads that use the X and = CIGAR operators instead of the M operator for alignment matches. The X and = CIGAR operators provide additional information (specifically, = is base match, X is base mismatch) for alignment blocks that are an alignment match.

@AmplabJenkins
Copy link

All automated tests passed.
Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/ADAM-prb/258/

massie added a commit that referenced this pull request Apr 2, 2014
Updating pileup conversion code to convert sequences that use the X and = (EQ) CIGAR operators
@massie massie merged commit 93ed54f into bigdatagenomics:master Apr 2, 2014
@massie
Copy link
Member

massie commented Apr 2, 2014

Thanks, Frank!

}

case CigarOperator.X =>

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whitespace is a bit funky in this block (below this comment)

@fnothaft fnothaft deleted the pileup-xeq branch July 10, 2014 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants