Clinical-Genomics / scout

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

Missing views for Cancer SVs #4341

Closed ivadym closed 9 months ago

ivadym commented 9 months ago

They may be some missing fields for the SV views:

I think it's also worth considering if this issue extends beyond these cases, as it seems that cancer_sv is not considered as a proper category to display some data. We should verify if we're expecting to display information also for SVs when the variant category is just cancer.

northwestwitch commented 9 months ago

Hi @ivadym are you working on this? Otherwise I can help!

northwestwitch commented 9 months ago

Hi again. Are you sure the cases described in checkboxes 2 and 3 will not be resolved by changing the settings relative to cust059?

You need to select the loqusdb instances you want stats from in the settings:

image

Don't forget to save the settings by clicking on "save settings" at the end of the page

ivadym commented 9 months ago

Yes, but that setting seems to only affect what's shown in the Local observations component in the variants page:

Screenshot 2024-01-17 at 10 44 41

I'm talking about this one, which misses to report archive observations for Cancer:

Screenshot 2024-01-17 at 10 45 16

I've checked a bit the code and I think we need to add something like this for the observations (update the observed_cell_general macro, which currently only shows local_obs_old):

Screenshot 2024-01-17 at 10 55 09

Thanks for looking into this! I can work on it once I get access to cg-vm1, but if you have time please go ahead. I could also help with some testing :)

northwestwitch commented 9 months ago

Ok, but then we can mark the second checkbox as done, right? The third one might have to do with the variant parsing.

ivadym commented 9 months ago

Sorry, I'm not being clear enough.

In point 2, the variant I mentioned lacks the expected Local tag in the Observed column on the cancer/variants page:

Screenshot 2024-01-17 at 11 22 19

However, on the specific variant page, you'll see we're reporting cancer variants:

Screenshot 2024-01-17 at 11 22 26

I think we need to extend this part to include local_obs_cancer_somatic_old and local_obs_cancer_germline_old:

https://github.com/Clinical-Genomics/scout/blob/835dac88c81895a15249d02532ab927da95bd7b6/scout/server/blueprints/variants/templates/variants/components.html#L184-L186

northwestwitch commented 9 months ago

Ah I see, you are on the variantS page, not the variant one. Understood!

northwestwitch commented 9 months ago

Ok, point 2 should be fixed now. We merged the fix yesterday. The latest main branch is now deployed on stage