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

Adds option to write sample names in variantCounts #8656

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

odcambc
Copy link
Contributor

@odcambc odcambc commented Jan 15, 2024

In some cases it may be useful to know what reads are giving rise to which specific variants. I have run into several cases while debugging some strange results where this would be useful to know, and also there is a QC workflow we would like to implement where this would be essential information. This is unlikely to be generally useful, however.

This PR adds a flag, --write-qnames, which will, for each variant, write the list of qnames in the bam that give rise to that variant as a comma separated list in the final column.

This PR also makes synonymous variants (with no protein-level consequence) write an empty value rather than nothing, in order to keep column order.

This seems to work with SE reads, but hasn't been tested much with PE reads.

This should also probably not parse read names by default, but only if write-qnames is set.

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.

None yet

1 participant