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

Supporting reads #18

Closed
RenzoTale88 opened this issue Dec 28, 2021 · 6 comments
Closed

Supporting reads #18

RenzoTale88 opened this issue Dec 28, 2021 · 6 comments
Labels
enhancement New feature or request

Comments

@RenzoTale88
Copy link

Hello,
I was wondering if it would be possible to add an option to list the supporting reads for each SV? Even if only for the pacbio/ont reads, I think it would be useful to have as an information.
Thank you in advance,
Andrea

@kcleal kcleal added the question Further information is requested label Dec 28, 2021
@kcleal
Copy link
Owner

kcleal commented Dec 28, 2021

Hi,
There are a few metrics in the output that should be sufficient:
SU column in the INFO field is the 'support' for the event, a combination of PE, SR and WR
WR is the read count for number of 'within-read' SVs
SR is the number of split-reads
PE is the number of paired-end discordant reads

Hope that helps

@RenzoTale88
Copy link
Author

Hi @kcleal thank you for your reply. Unfortunately, I actually need something in the line of the RNAMES field generated by callers such as Sniffles and/or cuteSV. I would understand a "no, we won't implement it" if too much a bother :)

@kcleal
Copy link
Owner

kcleal commented Dec 30, 2021

I see, sorry I misunderstood, I can see how this would be useful. I think this could be incorporated as an option in a future release. Would you prefer the RNAME's to be dumped into the output vcf, or perhaps to a seperate csv file in the temporary directory?

@kcleal kcleal added enhancement New feature or request and removed question Further information is requested labels Dec 30, 2021
@RenzoTale88
Copy link
Author

RenzoTale88 commented Dec 30, 2021 via email

@kcleal
Copy link
Owner

kcleal commented Feb 7, 2022

Hi,
I've had a think about this feature and ran some quick tests, and I think it will be more difficult to implement efficiently than I first thought. The main problem is that paired-end reads end up consuming huge amounts of memory due to having to store all the query names of the reads. In the initial stages of analysis this can be quite problematic. I think I will put on hold trying to implement this for now.

@kcleal kcleal closed this as completed Feb 7, 2022
@KristinaGagalova
Copy link

I think this option is very handy, just wondering if that can easily done for LR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants