-
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
General Feedback - Temporal Data #83
Comments
Architecture Inflection Points for Transforming PHI Dates in FHIR DataTransforming data that includes Protected Health Information (PHI) dates requires careful architectural planning to support regulatory compliance, scalability, and, crucially, the ability to maintain utility for analysis. Below are key inflection points to guide this process, with a focus on enabling longitudinal studies and temporal trend analysis while retaining the original sections for completeness. 1. Support for Analytical Use Cases
2. Data Privacy and Compliance Requirements
3. Transformation Logic
4. Data Storage Considerations
5. Testing and Validation
Comparison of Effort: Shifting Datetime Fields vs. Introducing an
|
Aspect | Shifting Datetime Fields | Introducing ageAtEvent Extension |
---|---|---|
Implementation Effort | Moderate | High |
Schema Changes | None | Requires new field/extension |
Maintenance Effort | Low | Medium |
Analytical Complexity | Medium (requires handling shifts) | Low (explicit age simplifies queries) |
Interoperability Impact | Minimal | Potential challenges with existing systems |
Privacy Compliance | Moderate (risks with deterministic shifts) | High (avoids direct PHI dates) |
Temporal Relationship | Requires careful validation | Naturally preserved through derivation |
User-Friendliness | Moderate | High |
Recommendation
-
Use Shifting Datetime Fields if:
- Minimal schema changes are preferred.
- The dataset requires interoperability with systems expecting datetime fields.
- Analysts are comfortable handling shifted dates during queries.
-
Use
ageAtEvent
Extension if:- Simplifying analytical workflows is a priority.
- Privacy concerns about datetime fields are significant.
- There is a focus on longitudinal studies where age is a primary analytical metric.
Each approach has trade-offs, but introducing an ageAtEvent
extension provides stronger support for user-friendly and privacy-compliant analysis, albeit at a higher initial implementation cost.
@JamedFV You were auto-assigned. Sorry, but I can't seem to remove the assignment? @teslajoy Can you review and comment? @RobertJCarroll FYI: Still a draft but wanted to follow up from our last call. |
Hi @bwalsh - Thanks for submitting this! Sorry about that, I've got the issues generated through a template auto-assigned to myself so they don't get missed. Would you like this issue assigned to yourself or to someone else for review? |
Thanks @JamedFV - if you can assign it back to me. Thanks again |
LGTM 👍 - thank you @bwalsh! As an analyst : measuring N categories as a function of time where the time interval is on a standard unit and shift is important/crucial. As a data engineer : In FHIR R5 version, looks like relativeTime is the closest FHIR datatype to relativeDateTime. relativeTime doesn't seem to allow user to point to a fhir resource directly. |
This looks great Brian, thanks. I think my straw proposal would be: What we if required both? It's a bit extreme, but considering the value for our users it might be worth it. Plus we could potentially build some tooling together to help the lift on the repository side. |
I can't find the R5 relative-time extension that's referenced in the deprecated relativeDateTime, but the R6 version you linked does appear to use |
I see - "context used as a base point" can be interpreted more broadly. Thank you. |
@RobertJCarroll Thanks. Building tooling together sounds like a great idea. I'm thinking through |
What were you reviewing?
Temporal data:
datetime
andageAtEvent
fields across fhir resourcesReview Date:
December 2024
Relevant Link:
For example see birthDate and deceasedDateTime in https://nih-ncpi.github.io/ncpi-fhir-ig-2/StructureDefinition-ncpi-participant.html#profile
Feedback:
See below
The text was updated successfully, but these errors were encountered: