smart-on-fhir / chart-review

Measure agreement between chart reviewers.
https://docs.smarthealthit.org/cumulus/chart-review/
Apache License 2.0
1 stars 0 forks source link

feat: add new config key 'ignore' for notes to skip #11

Closed mikix closed 6 months ago

mikix commented 6 months ago

You can now add config like the following to have matching notes excluded from any accuracy comparisons:

ignore:
  - 1023  # Label Studio note ID
  - ENC5342  # real ID
  - c3959eda-a67d-11ee-bd6e-b7eafa174aa3  # anonymized ID
  - Encounter/c3959eda-a67d-11ee-bd6e-b7eafa174aa3  # anonymized ref

Also some small tweaks:

mikix commented 6 months ago

This comment is out of scope of this PR, which is fine once I wrapped my head around it, but in calmer times we should consider revisiting the naming of things - I'm having a hard time getting oriented here.

Great feedback - hit me up on slack (or here) with some terms that gave you pause. I'm guessing what external means might need some clarity. And ranges/labels could use some nice definitions. And annotator vs truth.

At minimum, we/I should bulk up whatever "terminology" section exists in the readme. And/or consider renaming stuff.