ccsm-cds-tools / ccsm-cds-with-tests

This repository contains clinical decision support (CDS) which provides recommendations for cervical cancer screening and management (CCSM).
Apache License 2.0
7 stars 3 forks source link

Add rare cytology to CytologyInterpretation, fix LastKnownAisBiopsy #134

Closed mrnosal closed 4 months ago

mrnosal commented 7 months ago

Add rare cytology for AGC favor neoplasia and Endocervical Cells Favor Neoplasia for these results do not appear as UNK. Also fix name of MostRecentAisBiopsy to LastKnownAisBiopsy since it is not only at the most recent histology result.

mrnosal commented 6 months ago

Added additional rare cytology results Do I need to add the rare cytology SNOMED-CT codes to fsh-tank/input/fsh/CervicalCytologyResult.fsh ?

mrnosal commented 6 months ago

Added additional rare cytology results Do I need to add the rare cytology SNOMED-CT codes to fsh-tank/input/fsh/CervicalCytologyResult.fsh ?

I've added the same rare cytology codes to CervicalCytologyResult.fsh as in CytologyInterpretation. Newly added rare cytologies are being correctly displayed in Dashboard.

mickohanlon23 commented 6 months ago

Added additional rare cytology results Do I need to add the rare cytology SNOMED-CT codes to fsh-tank/input/fsh/CervicalCytologyResult.fsh ?

I've added the same rare cytology codes to CervicalCytologyResult.fsh as in CytologyInterpretation. Newly added rare cytologies are being correctly displayed in Dashboard.

We talked about your original question in-person; I agree with adding the rare cytology codes to CervicalCytologyResult.fsh. This will allow the new codes to be used by our translate code in CQL-Services, and by our questionnaire input in CCSM Dashboard, if we go through with adding that feature.

Is this ready for re-review?

mrnosal commented 6 months ago

Is this ready for re-review? Not yet - there are some questions around AIS/Endocervical AIS that need to be resolved with clinician feedback.

mickohanlon23 commented 6 months ago

Is this ready for re-review? Not yet - there are some questions around AIS/Endocervical AIS that need to be resolved with clinician feedback.

Is there are short-term version of this that we can merge for this Friday's deadline, maybe with at least the UI updates needed for all the cytology test results that can be entered via the pap tracker to not show up as an "unknown" missing result? And then we can address the additional rare cytology results as well as logic for tests that have more than one result with one of them being a rare cytology result in a follow-on PR?

mrnosal commented 6 months ago

Is there are short-term version of this that we can merge for this Friday's deadline, maybe with at least the UI updates needed for all the cytology test results that can be entered via the pap tracker to not show up as an "unknown" missing result? And then we can address the additional rare cytology results as well as logic for tests that have more than one result with one of them being a rare cytology result in a follow-on PR?

I've bumped Endocervical AIS out of the 'main' interpretations in CytologyInterpretation. It will only be treated as a rare cytology until we get clinician clarification. This is sufficient to give names to rare cytologies and handle them when they are single conclusions for diagnostic reports.