-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add QPP profiles to flexporter mapping file #52
Conversation
Coverage report
Test suite run success95 tests passing in 8 suites. Report generated by 🧪jest coverage report action from 4589520 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cleanup looks good, and the new resources are all working! Yay! A couple of comments about potentially making the data more relevant if we can do it easily enough. And a quick question: For resources that were already exported by synthea or the previous 130 mapping file, were there any fields from QPP that weren't already exported on those resources? Wondering in order to contextualize the upcoming task.
synthea/qpp_qicore.yaml
Outdated
- http://hl7.org/fhir/us/qicore/StructureDefinition/qicore-adverseevent | ||
fields: | ||
- location: AdverseEvent.event.coding | ||
value: $randomCode([http://hl7.org/fhir/ValueSet/adverse-event-type]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We run the risk of creating unrealistic data when we select a random code. For this code set especially, a lot of the codes indicate a death, which may be problematic when the patient doesn't have other data indicating death. We can probably leave it for now and see how it turns out in testing as it may not actually be a big deal. If we want to be extra proactive, we could identify some common bad outcomes that come from synthea modules and create the AdverseEvent resource based on that. (I would guess that Synthea already endeavors to create some adverse events even if they're not labeled as such, and Dylan might have more insight into this.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good! Or, I am happy to not use randomCode
here if it would make more sense to just pick a code from that list...but then we wouldn't have as much variability so I am open to either.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Off the top of my head I can't think of any adverse events in the modules, other than possibly some of the surgeries have a risk of death. I can look for some though if that's of interest.
This also makes me think there should be some way of saying "random code from these specific few codes" rather than requiring the use of a terminology server and a published ValueSet. I can probably get that implemented pretty quickly
Co-authored-by: lmd59 <[email protected]>
Co-authored-by: lmd59 <[email protected]>
To answer your question, no, I did not find any fields missing from the resources that were already exported by synthea or the previous 130 mapping file. Regarding the status codes for some of the profiles that we have arbitrarily set in the mapping file, is it ok for me to go ahead and use |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a recommendation for the .medication
fields and a note for future tasking!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Nice updates!
Summary
This PR adds profiles defined by the generated CMS Quality Payment Program (QPP) IG to the existing qicore flexporter mapping file.
New behavior
AdverseEvent
,Communication
,MedicationAdministration
,MedicationDispense
andTask
.Code changes
synthea/qpp_qicore.yaml
- renamed file and added additional QI Core profile creationsynthea/README.md
- updates to explain the changes to the mapping fileTesting guidance
src/main/resources/synthea.properties
in order to use the terminology server../run_synthea -fm <path to qpp_qicore.yaml>
/output
directory