Clinical-Genomics / scout

VCF visualization interface
https://clinical-genomics.github.io/scout
BSD 3-Clause "New" or "Revised" License
152 stars 46 forks source link

Show SNV/indels in SMN genes via button redirect from the SMN CN page #4822

Closed Jakob37 closed 2 months ago

Jakob37 commented 2 months ago

Is your feature request related to a problem in the current program to new available techology or software? Please describe and add links/citations if appropriate.

Our local geneticists are looking into using the SMN CN module in their clinical routine, and raised some points for discussion. I'll open issues for their discussion points to discuss.

Describe the solution you'd like

They asked whether it would be possible to add a panel showing what SNV/indels are residing inside the SMN genes similarly to how it is on the SV panel page.

I.e. this panel:

overlapping_snvs

dnil commented 2 months ago

Right, this could be done with the current setup. The most salient reason not to is that at least our short read pipelines are not guaranteed to give (and with that I mean essentially guaranteed to not give 😉) very reliable results on pseudogene/genefamily genes. The calls on the SMN page are instead reasonably reliable, up to the performance reported for the SMNCopyNumberCaller. The developer felt the CLGs interpreting might not appreciate the difference. 😊

Jakob37 commented 2 months ago

OK I see, that makes sense! I'll forward that thought, let's see what they say.

Jakob37 commented 2 months ago

I asked our geneticists about this. They decided to check in our data for themselves whether this would be of value, and screened variants from our ~8000 wgs cases for this region.

They came back with some points:

In short, in their method document they want to have a safety-check for one copy SMN1 plus exon 8 SNV/indel in the other copy. Having this available in Scout together with where they check the SMN calls would make that check much easier to do.

Sounds to me that they are making some good points 🤔 What do you say @dnil ?

dnil commented 2 months ago

Yes, there are a few parts of the gene that are decent, while not large, it is of course unnecessary to miss something. Sounds like they would go search for them anyway - it’s not like we are hiding the calls on the SNV view We could always add a warning to the variants table - or to save a lot of code just add a button to the snv and sv view prefilled with a SMN1 & SMN2 query…

ehre commented 2 months ago

My two cents on this - with current srWGS the SMNCopyNumberCaller is sufficient for screening purposes. I would rather that we put our energy into a high quality SMN1 (and SMN2) calling, phasing and visualization for soon to come lrWGS data.

Jakob37 commented 2 months ago

OK I see! I discussed it briefly with the other geneticists here as well. In conclusion it seems like adding a button with a prefilled search into SNV/SV view would be fully sufficient for what they want to achieve, and would be easy to add. So maybe that is the way to go then 👍

dnil commented 2 months ago

If you feel for this one @Jakob37, perphas look at the SNV/SV buttons on the WTS page, and make the search for SMN1 and SMN2 instead of the gene with an RNA outlier!

Jakob37 commented 2 months ago

If you feel for this one @Jakob37, perphas look at the SNV/SV buttons on the WTS page, and make the search for SMN1 and SMN2 instead of the gene with an RNA outlier!

Sure, I'll look into it!