search

suzytamang / clever-rockies

clinical event recognizer for TIU
3 stars 1 forks source link

Remove FAM from na_trigs.json #100

Closed vilijajoyce closed 3 weeks ago

vilijajoyce commented 1 month ago

Original Request

Per Esther 10/3/24, "I noticed on CLEVER-Rockies that the NA triggers for CONNECT includes FAM. This should not be. Many terms related to CONNECT have a Familiar in their context rendering many of those snippets Not Applicable. With the next update could this be changed as well?"

Action Requested

Remove FAM for CONNECT in na_trigs.json

Requestor

3ST Team

Additional Actions

dax-westerman commented 1 month ago

"CONNECT": ["FAM"], updated

image

dax-westerman commented 1 month ago

Converted the Validate and Update delver-rockies SOPs to separate issues.

vilijajoyce commented 1 month ago

"CONNECT": ["FAM"], updated

image

@dax-westerman Please remove quotes contained within the bracket - "CONNECT": [] TIA!

dax-westerman commented 1 month ago

Foremost, thank you for that catch: oversight on my part. Also, thank you for providing a great example of a validation test!