-
Notifications
You must be signed in to change notification settings - Fork 1
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
Post-extraction mapper #199
Conversation
a028278
to
eab1319
Compare
The latest commit changes how a mapper is included as a post-extraction process. Now, if the user includes a
|
} | ||
} | ||
}, | ||
"dateFilter": { |
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.
Is this still relevant?
"title": "Post Extraction Processes", | ||
"type": "object", | ||
"properties": { | ||
"dateFilter": { |
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.
Is this still relevant/correct?
Summary
This PR adds post-extraction filtering by date, which can be enabled in the config file. The user does this by specifying start and end dates and which resource types to filter and the post-extraction filter will only include resources within that range for those resource types.
Currently WIP is adding all the date formats for each resource type since they vary resource to resource, in the current state you can filter Observations and Conditions by date and I can easily expand it for other resource types. In the current state, the date fields to be checked are hard-coded to be the style of date we produce with the CSV extractors, for example Observation could be any of the effectiveX dates, but I made it only check for effectiveDateTime since that is what we produce with the extractor. We could make a new task to create more robust date filtering that checks all possible date formats.
Testing guidance
Add the following to the config file:
Running the extractor with this filter should result in the first patient output having no Conditions and only one Observation left