ClinGen / gene-and-variant-curation-tools

ClinGen's gene and variant curation interfaces (GCI & VCI). Developed by Stanford ClinGen team.
https://curation.clinicalgenome.org/
MIT License
3 stars 1 forks source link

Protein Interactions Feature Requests #366

Open bmpbowen opened 2 months ago

bmpbowen commented 2 months ago

We've received a request to update the field related to "Has this gene or genes been implicated in the above disease?" in the Protein Interactions section of the GCI. Example: https://curation-test.clinicalgenome.org/curation-central/7dbe9416-aa81-40c1-9aec-b2cfad3a501a/annotation/d61cbe3b-3cc3-4be3-9092-3f15682c5b1e/experimental-curation/c906268a-7646-4d47-9a73-d4f3ff36c0ff/edit/

A user observed that evidence related to this field is added to the overall [score] (https://curation-test.clinicalgenome.org/curation-central/7dbe9416-aa81-40c1-9aec-b2cfad3a501a/gene-disease-evidence-summary/?preview=yes) even when the box was is not checked. Going forward, users would like this field to directly relate to scoring in the following ways:

  1. Make “Has this gene or genes been implicated in the above disease?" a required field.
  2. Change from one checkbox to two radio buttons: 1 Yes, 2 No
  3. Edit Warning Box text: Old text: “Warning: not checking the above box indicates this criteria has not been met for this evidence; this should be taken into account during its evaluation.” New text: “Warning: Selecting “No” for the above required question indicates this criteria has not been met for this evidence; therefore this protein interaction will not be scored.”

image

  1. Disallow "Score" as a dropdown Status option when the "Has this gene or genes been implicated in the above disease?" box is unanswered; show an error instead of Saving: “Missing field required”.
  2. Disallow “Score” as a dropdown Status option when the "Has this gene or genes been implicated in the above disease?" box is marked No; show an error instead of Saving: "Only genes implicated in the above disease are eligible to be scored for protein interactions" If 4 and 5 can't be implemented, an alternative would be to set default score to 0 when the question "Has this gene or genes been implicated in the above disease?" is answered "No"

In the future, we could try to automate the answer to this question by cross-referencing other systems, but that would require consistent disease names across data sources or a way to check parent/sibling disease terms (GCI, GenCC) which does not yet exist, so tabling that suggestion for now.

wrightmw commented 1 month ago

@bmpbowen This ticket requires further internal discussion with our dev team. There are impacts on the logic within the UI which may mean this needs to be re-specified.

bmpbowen commented 1 month ago

@wrightmw certainly! Iplanned to discuss at our next Stanford Clingen meeting. Will add to that agenda.

bmpbowen commented 1 month ago

Proposed alternative to the above fix is: Instead of yes/no, have the check box auto-select once a gene is entered in the first field. This way no additional changes are required.