Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Map CSTE Member OID to member-oid extension during convert-step #14511

Open
3 tasks
victor-chaparro opened this issue May 22, 2024 · 11 comments
Open
3 tasks

Map CSTE Member OID to member-oid extension during convert-step #14511

victor-chaparro opened this issue May 22, 2024 · 11 comments
Assignees
Labels
platform Platform Team Spillover-Underestimated Effort Work turned out to be more complex or time-consuming than initially estimated.

Comments

@victor-chaparro
Copy link
Collaborator

victor-chaparro commented May 22, 2024

User Story

As a STLT receiving ELR,
I want to only receive test results that are antigen tests for a given condition,
so that my programs have the correct type of tests we are interested in analyzing.

Description/Use Case

Currently the covid pipeline uses the Covid LIVD table to determine if reports are for antigen tests. This is needed because some receivers only want to receive antigen tests. We need the ability to do this in the UP, but can't rely on the LIVD table since that only supports Covid results and this needs to work for any condition.

For the UP, we need to implement a way to route items based on whether they are antigen or another type. This information is reflected by the Member OID the test performed belongs to

Risks/Impacts/Considerations

Dev Notes

Implementation checklist:

1.Add new extension to map test code member oid (Member OID column in RS condition mapping table): https://reportstream.cdc.gov/fhir/StructureDefinition/test-performed-member-oid

  1. Modify convert step to stamp ELR messages with test-performed-member-oid
  2. Ensure this data is NOT in Azure Events

With the above implemented, the following routing filter expression should be functional:

%resource.code.coding.extension('https://reportstream.cdc.gov/fhir/StructureDefinition/condition-code').value.where(code  in ('840539006')).exists() and %resource.interpretation.coding.code = 'A' and %resource.code.coding.extension('https://reportstream.cdc.gov/fhir/StructureDefinition/test-performed-member-oid').value.where(code  in ('2.16.840.1.113762.1.4.1146.1142')).exists()

Docs on our condition to code mapping dataset can be found here: prime-router/docs/design/proposals/0023-condition-to-code-mapping/0023-condition-to-code-mapping.md

CSV version of the code to condition mapping table in our repo that is used to seed a dev DB: prime-router/metadata/tables/local/observation-mapping.csv

Official dataset our condition mapping table is derived from: https://docs.google.com/spreadsheets/d/1rO_p145xXO1AD76vx8vBqXgoQUnLqfc8/edit?gid=739612351#gid=739612351

Acceptance Criteria

  • test-performed-member-oid stamped on Observation resources in ELR messages
  • Results can be filtered out based on test type (member oid) as determined by CSTE value sets
  • The process to determine results are antigen is documented
@victor-chaparro victor-chaparro added the onboarding-ops Work related to onboarding with a partner. Addressed by the Onboarding & Operations team in RS. label May 22, 2024
@victor-chaparro victor-chaparro added the engineering Work to be completed by an engineer label May 22, 2024
@victor-chaparro victor-chaparro added the needs-refinement Tickets that need refinement from the team label May 23, 2024
@victor-chaparro victor-chaparro removed the needs-refinement Tickets that need refinement from the team label May 30, 2024
@chris-kuryak
Copy link
Collaborator

Will require Platform input for longer term solution

@chris-kuryak
Copy link
Collaborator

Projected to take more than 1 sprint

@GilmoreA6 GilmoreA6 changed the title Map antigen results in the Universal Pipeline Map CSTE value set to condition-code extension during convert-step Jul 16, 2024
@chris-kuryak
Copy link
Collaborator

@victor-chaparro
Per James, can we update this ticket and potentially hand off to Platform since it seems like it is more in their wheelhouse?

@chris-kuryak chris-kuryak added the platform Platform Team label Sep 27, 2024
@arnejduranovic arnejduranovic added the blocked Issue State label to flag PRs and issues to show they are blocked label Oct 4, 2024
@arnejduranovic
Copy link
Collaborator

Need to meet with @victor-chaparro and @chris-kuryak to better understand what values we are expecting to be stamped and how we can use the OOID to get the values we want.

@arnejduranovic
Copy link
Collaborator

Unblocked. Spoke with Victor and got clarification on what exactly needs to be stamped. Will update ticket.

@arnejduranovic arnejduranovic added ready-for-refinement Ticket is a point where we can productively discuss it and removed onboarding-ops Work related to onboarding with a partner. Addressed by the Onboarding & Operations team in RS. blocked Issue State label to flag PRs and issues to show they are blocked engineering Work to be completed by an engineer labels Oct 7, 2024
@MichaelEsuruoso
Copy link
Collaborator

@MichaelEsuruoso MichaelEsuruoso removed the ready-for-refinement Ticket is a point where we can productively discuss it label Oct 7, 2024
@arnejduranovic arnejduranovic changed the title Map CSTE value set to condition-code extension during convert-step Map CSTE Member OID to member-oid extension during convert-step Oct 7, 2024
@kant777 kant777 self-assigned this Oct 18, 2024
@kant777
Copy link
Collaborator

kant777 commented Oct 29, 2024

@chris-kuryak
Copy link
Collaborator

@kant777 Where is that link located on the website?

@victor-chaparro @jillian-hammer FYI

@kant777
Copy link
Collaborator

kant777 commented Oct 30, 2024

Here is the first version of the PR https://github.com/CDCgov/prime-reportstream/pull/16397/files this is not full ready yet. I plan getting a review from @arnejduranovic or @mkalish to make sure I am in the right direction

@kant777
Copy link
Collaborator

kant777 commented Nov 6, 2024

Almost ready, fixing some tests. Slight delay due to onboarding.

@MichaelEsuruoso MichaelEsuruoso added Spillover-Dependencies/Blockers(Internal) Task delayed due to dependencies on other teams or tasks Spillover-Underestimated Effort Work turned out to be more complex or time-consuming than initially estimated. and removed Spillover-Dependencies/Blockers(Internal) Task delayed due to dependencies on other teams or tasks labels Nov 6, 2024
@kant777
Copy link
Collaborator

kant777 commented Nov 7, 2024

All tests pass. #16397

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform Platform Team Spillover-Underestimated Effort Work turned out to be more complex or time-consuming than initially estimated.
Projects
Status: New items
Development

No branches or pull requests

6 participants