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.

Sign up for GitHub

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

Jump to bottom

Ambiguous reads should be filtered by default #4

Closed
ressy opened this issue Apr 23, 2018 · 0 comments
Closed

Ambiguous reads should be filtered by default #4

ressy opened this issue Apr 23, 2018 · 0 comments
Milestone
Version 0.2.0

Comments

@ressy
Copy link
Member

ressy commented Apr 23, 2018

Currently sequences including N for base calls (or anything outside of [ACTG]) are not handled specially, so they may appear in final output including in purported alleles. By default these sequences should marked in the output of analyze_sample and filtered in the output of summarize_sample to avoid this.
@ressy ressy added this to the Version 0.2.0 milestone Apr 23, 2018
ressy added a commit that referenced this issue Apr 24, 2018
@ressy
Fix #4: track and filter ambiguous sequences
936bbef 
Add columns named "Ambiguous" to sample and sample summary data frames
to track sequences with non-ACTG characters and track filtering of these
sequences, respectively.
@ressy ressy closed this as completed Apr 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Version 0.2.0
Development

No branches or pull requests
1 participant
@ressy