CDCgov / prime-reportstream

ReportStream is a public intermediary tool for delivery of data between different parts of the healthcare ecosystem.
https://reportstream.cdc.gov
Creative Commons Zero v1.0 Universal
71 stars 39 forks source link

IL - ORC-12.1 & ORC-16.1 ordering facility name being placed in an ‘ID’ subfield causing messages to fail #2833

Closed Adrian-Brewster closed 2 years ago

Adrian-Brewster commented 2 years ago

ORC12.1 (OrderingProvider->ID) is for an ID and has maximum character size limit of 15, it would be best if this subfield could be left blank or the correct NPI be placed here. The same problem is happening in OBX 16.1.1 (OrderingProvider->ID), the ordering facility name is being placed here and it’s exceeding the character size limit.

Messages do not need to be resent.

image.png

Adrian-Brewster commented 2 years ago

Mike working on triaging the ticket today. Will coordinate with Herman to get eta in creating a fix

rhood23699 commented 2 years ago

@Adrian-Brewster @anshulkumar-usds @MikeC-A6 I've reviewed the Metadata and their a 66 rows with Egyptian Health Department as the Ordering Provider ID. All rows came from Sender ID: simple_report.default. I can't see the actual data sent, but from reviewing the Schema Name: pdi-covid-19, I'm almost 100% sure that we are receiving Egyptian Health Department in the Ordering Provider ID field; so this appears to be a data entry problem in Simple Report.

anshulkumar-usds commented 2 years ago

@LGFranklin can you please take a look on your side? I'll forward you the email we received from IL.

MikeC-A6 commented 2 years ago

@anshulkumar-usds - I'm thinking we need a fix for IL on our end as any changes on the sender side could take a bit. Their specific request is to only populate ORC 12.1 and OBX 16.1.1 with a value if there is a properly formatted NPI. Flagging this for a 16th minute topic for tomorrow to discuss potential solutions - not sure if we can do this in the HL7 serializer, as @mlighdc mentioned that some States use the Provider ID field in different ways- so a serializer change could have unwanted side effect for other STLTs.

LGFranklin commented 2 years ago

@anshulkumar-usds The user input the health department as the NPI. I will reach out to them to request that they update it from the text to a number and talk with our team about making sure our validation is working.

mlighdc commented 2 years ago

@rhood23699 did you say there are 66 rows with Egyptian Health Department as the Ordering Provider ID? How can that be since they have not created any facilities or submitted any tests via SimpleReport. Are they submitting their test results via Cue Health or another OTC device? Or am I misunderstanding your comment.

I am not sure how this org got to the state if they have not sent any test results.

@LGFranklin is the school with Egyptian in its facility name using Egyptian Health Department as their ordering provider?

Also, they should have an NPI number since they are a health care facility.

LGFranklin commented 2 years ago

Sorry Ming! My original sleuthing was incorrect. There is an Egyptian Health org with Ms. Byrd as the only user. They haven’t done any setup. This above is for a facility in a separate org that listed Ms. Byrd as the OP and Egyptian PH as the NPI. Should have written “scratch that” in my GH comment as opposed to the edit.

I will follow up with this user here to have them update and our team at SR to make sure our validation is fixed.

Sent from my iPhone

On Nov 16, 2021, at 11:36 PM, Ming Ligh @.***> wrote:

 @rhood23699 did you say there are 66 rows with Egyptian Health Department as the Ordering Provider ID? How can that be since they have not created any facilities or submitted any tests via SimpleReport. Are they submitting their test results via Cue Health or another OTC device?

They should have an NPI number since they are a health care facility.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

MikeC-A6 commented 2 years ago

@LGFranklin - The specific issue that is causing messages to fail in ILs integration engine is when the character length of the Provider ID field is >15.. There are 14 SimpleReport users who have sent data in the last month that have >15 characters. Some info regarding potential validation options in SimpleReport:

LGFranklin commented 2 years ago

Great ideas; thanks, Mike! I'll log this and our team will amend shortly.

TY!

On Wed, Nov 17, 2021 at 7:27 AM MikeC-A6 @.***> wrote:

@LGFranklin https://github.com/LGFranklin - The specific issue that is causing messages to fail in ILs integration engine is when the character length of the Provider ID field is >15.. There are 14 SimpleReport users who have sent data in the last month that have >15 characters. Some info regarding potential validation options in SimpleReport:

  • NPI is always a 10 digit numeric value
  • Might be useful to have this as an optional field, but if there is data populated it must be 10 numeric characters
  • this would eliminate the need for users to type in random or dummy values for NPI if they don't have one

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/CDCgov/prime-reportstream/issues/2833#issuecomment-971533264, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATM5LDZJI26DQCCQSA6NB6DUMONUJANCNFSM5G6MZFTA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

anshulkumar-usds commented 2 years ago

@LGFranklin @MikeC-A6 IL is looking for updates on this issue. Here's my proposed response:

  1. Users in SR are entering the PHD name in the NPI field causing the issue at hand. SR will contact the user and advise they use the correct numeric value
  2. SR will also work on a longer term solution to add data validation to the NPI fields, so we prevent future issue
  3. At ReportStream, we can still enable logic to blank these fields ( per state?) if the 15 char limit is broken. Any thoughts on this Mike.
LGFranklin commented 2 years ago

@anshulkumar-usds Yes, we have contacted the user to request and update, and we created a ticket to address this: https://app.zenhub.com/workspaces/simplereport-better-data-team-605b43d51ea9f700119a48ee/issues/cdcgov/prime-simplereport/3005

Once this is complete, the validation will allow only numerical characters, and not allow input longer than 10 digits.

anshulkumar-usds commented 2 years ago

Thanks @LGFranklin I will close this out on RS board.

RS will track a platform wide enhancement to blank these fields if the char limit is violated here: https://app.zenhub.com/workspaces/onboarding--operations-6166edbd409257001e09f1b2/issues/cdcgov/prime-reportstream/3137

anshulkumar-usds commented 2 years ago

SimpleReport is working on the validation process here: https://github.com/CDCgov/prime-simplereport/issues/3005

FYI - @MikeC-A6