Skip to content
This repository has been archived by the owner on Feb 7, 2025. It is now read-only.

CA: UCSD Results Staging EHR Filing #1229

Closed
4 tasks done
JohnNKing opened this issue Aug 6, 2024 · 15 comments
Closed
4 tasks done

CA: UCSD Results Staging EHR Filing #1229

JohnNKing opened this issue Aug 6, 2024 · 15 comments

Comments

@JohnNKing
Copy link
Contributor

JohnNKing commented Aug 6, 2024

Story

As UCSD, so that I can review and provide patient care based on NBS findings, I need sample result messages sent by ReportStream staging to appear to clinicians in my EHR test environment.

Pre-conditions

  • Out of scope: Production connectivity

Acceptance Criteria

Tasks

Notes

  • Optional: Any reference material or thoughts we may need for later reference
@basiliskus
Copy link
Contributor

We could add the correct encoding of µ as part of the tasks or AC

@JohnNKing
Copy link
Contributor Author

We could add the correct encoding of µ as part of the tasks or AC

I know we also #1222 -- I could make resolving that bug a precondition for this story if it would help.

@jorg3lopez
Copy link
Contributor

Update:
There will be a live testing session on Wednesday, Oct 2nd. In this live session, we plan to bring up the issue with the mu character. We will ask how the value shows up in UCSD's system. This will let us know if there are any issues or if everything is working as expected.

@jorg3lopez
Copy link
Contributor

Update:
During the live test, we noticed that the mu character was not showing properly. We will need to do more research and see what encoding (ISO, UTF) UCSD Epic is using.

@basiliskus basiliskus self-assigned this Oct 7, 2024
@basiliskus
Copy link
Contributor

I'll do a round of testing with RS to make sure the message is sent with the mu character showing correctly

@basiliskus
Copy link
Contributor

After some tests I found a couple of things. While testing locally, I found the files are correctly encoded throughout the flow. While testing in staging, I found that somewhere between RS receiving the initial HL7 message and sending the FHIR message to TI, the encoding gets garbled. So it's confirmed RS is sending the message to UCSD with the 'µ' character missing

@basiliskus
Copy link
Contributor

Created a bug report in RS for the encoding issue here: CDCgov/prime-reportstream#16196

@jbiskie
Copy link
Contributor

jbiskie commented Oct 11, 2024

We changed the locale to UTF-8 in the dockerfile configuration. No impact.

@jbiskie
Copy link
Contributor

jbiskie commented Oct 11, 2024

Removing -Dfile.encoding=UTF-8 in the JAVA_OPTS in local.settings.json breaks local deployments.

We're attempting to set this parameter for Azure deployments.

@GilmoreA6
Copy link
Contributor

Adding the "JAVA_OPTS = -Dfile.encoding=UTF-8" environment variable to the pdhdemo1 environment solved the issue. We will raise a PR to add this env variable to RS under RS issue #16196 linked above. This env variable need to be configured in the RS terraform files.

@JohnNKing JohnNKing assigned GilmoreA6 and unassigned basiliskus and jorg3lopez Oct 16, 2024
@basiliskus
Copy link
Contributor

It may be worth reopening #1222 because it seems there's an actual bug in Daily Data regarding the encoding of the 'µ' character, that is separate from the encoding issue currently being fixed in the deployed environments

@basiliskus
Copy link
Contributor

#1222 reopened

@basiliskus
Copy link
Contributor

basiliskus commented Oct 21, 2024

This open PR in the reportstream-sftp-ingestion repository fixes the encoding issue when sending the ORU using the SFTP service. Once the PR is merged, we need to test uploading new files to be sent in staging and checking the EHR Filing with UCSD

@basiliskus
Copy link
Contributor

The encoding fix in the reportstream-sftp-ingestion service has been merged and tested in staging. We should be ready to test this issue in the staging EHR Filing

@scleary1cs
Copy link
Contributor

Closing in favor of #1442 . Moving AC over.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants