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

wts.R creates wobble Positionen with AF >90% #6

Open
sschmutz opened this issue Jun 10, 2021 · 3 comments
Open

wts.R creates wobble Positionen with AF >90% #6

sschmutz opened this issue Jun 10, 2021 · 3 comments

Comments

@sschmutz
Copy link
Member

Strangely affects CMV amplicon of gene UL97.

sschmutz added a commit that referenced this issue Jun 10, 2021
@sschmutz
Copy link
Member Author

Commit 0edb6c2 on branch issue-6 applies a quick fix of this issue but it would be better to find the cause of it and fix that.

@sschmutz
Copy link
Member Author

This issue would by the way also fail to create a wobble-free sequence if one wants to create a "majority consensus" sequence by just setting the variant_threshold to 50% (which is in my opinion not a very robust approach anyway as a major base could always also be present at <50% if there is more than one alternative base for example).

@sschmutz
Copy link
Member Author

I found that this issue is caused by CRLF line endings in wrapped reference fasta files (instead of LF).
The reference sequence of UL97 contained these, while the reference sequences of the other two genes we are looking at do not.

Using either unwrapped fasta files or ensuring LF line endings avoids this issue.

PS: The reference sequence should also not contain any ambiguous nucleotide codes as this also leads to issues for the consensus sequence.

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

No branches or pull requests

1 participant