diskin-lab-chop / AutoGVP

17 stars 3 forks source link

merge P/LP and B/LB submissions when selecting consensus call #170

Closed rjcorb closed 1 year ago

rjcorb commented 1 year ago

Purpose/implementation Section

What feature is being added or bug is being addressed?

Closes #169. This PR updates selection of ClinVar submissions based on consensus call by merging L/LP and B/LB variants together when counting submissions, rather than considering these classifications separately as was done previously.

What was your approach?

Modified code in select-clinVar-submissions.R to create a new column ClinSig that takes values P/LP, B/LB and VUS. Submissions for each VariationID were counted based on ClinSig column to identify variants with a majority call. Then, among calls in this majority group, the classification at the most recently evaluated date is taken as final call.

What GitHub issue does your pull request address?

169

Directions for reviewers. Tell potential reviewers what kind of feedback you are soliciting.

Which areas should receive a particularly close look?

Please review updated code and run select-clinVar-submissions.R:

Rscript select-clinVar-submissions.R --variant_summary input/variant_summary.txt.gz --submission_summary input/submission_summary.txt.gz

Is there anything that you want to discuss further?

No

Documentation Checklist