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

Why didn't we find this variant supported? #97

Closed
ihodes opened this issue Dec 8, 2016 · 11 comments
Closed

Why didn't we find this variant supported? #97

ihodes opened this issue Dec 8, 2016 · 11 comments
Assignees
Labels

Comments

@ihodes
Copy link
Contributor

ihodes commented Dec 8, 2016

screen shot 2016-12-08 at 12 11 05 pm

This variant didn't make it into the report for some reason (this is RNA support being shown), all well-aligned good data.

@ihodes ihodes added the bug label Dec 8, 2016
@ihodes
Copy link
Contributor Author

ihodes commented Dec 8, 2016

Also one like this
screen shot 2016-12-08 at 12 53 57 pm

@iskandr
Copy link
Contributor

iskandr commented Dec 10, 2016

These are all by exon boundaries (I think -- can't see the annotation track in these screen shots). After dropping all the multi-exon spliced reads we're left with a bunch of reads that contain intronic sequences (blame RiboZero). Since the intronic sequences disagree with all annotated transcripts then we never end up discovering a reading frame for these variants.

@ihodes
Copy link
Contributor Author

ihodes commented Dec 11, 2016

Would it be reasonable and feasible to stop reading cDNA at exon boundaries, so when we inevitably get intronic RNA in some reads, they aren't effectively thrown out?

@iskandr
Copy link
Contributor

iskandr commented Dec 11, 2016 via email

@ihodes
Copy link
Contributor Author

ihodes commented Dec 12, 2016 via email

@ihodes
Copy link
Contributor Author

ihodes commented Jan 19, 2017

@julia326 @iskandr can we close this/have we solved this?

@julia326
Copy link
Contributor

I don't think we detect altered splicing (@iskandr please chime in if I'm wrong about this) - and the recent change to count intronic reads as reference mismatches after the exon boundary will actually make us throw out more reads. Coupled with these Ribozero side effects, that could be problematic. Is this a variant we would've wanted to keep, or should we double down on working with poly-A capture in the future?

@iskandr
Copy link
Contributor

iskandr commented Jan 19, 2017

@julia326 Define "detect" :-) -- we were certainly detecting it in this case, just incorrectly.

@iskandr
Copy link
Contributor

iskandr commented Jan 19, 2017

@ihodes How do these variants look after our changes to indel realignment?

@iskandr
Copy link
Contributor

iskandr commented Jan 19, 2017

@ihodes @julia326 When samples are degraded and/or prepared with ribosomal depletion we should count mismatches on both sides of the variant, which will penalize variant sequences that look like abnormal splicing.

@iskandr
Copy link
Contributor

iskandr commented Mar 3, 2017

Closing, since this is really an isovar issue: openvax/isovar#55 (which will be fixed by openvax/isovar#80)

@iskandr iskandr closed this as completed Mar 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants