populationgenomics / talos

Rare Disease variant reanalysis tool
MIT License
6 stars 4 forks source link

Remove csq filter for de novo #452

Closed MattWellie closed 6 days ago

MattWellie commented 6 days ago

Fixes

Proposed Changes

Notes

Since this change ClinvArbitration will be calling variants Pathogenic/Likely Pathogenic, instead of just Pathogenic. Looking at the code here we don't need to adjust to this change. We're testing for clinvar_significance.lower().contains('pathogenic') which will still be true.

Now that we're using ClinvArbitration we no longer have "conflicting interpretations of pathogenicity", which was a conflating status, and meant we had to test for "contains pathogenic, and doesn't contain conflicting", so that's been simplified.

Checklist