-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Create Health Connect to FHIR converter module #15
Conversation
...es/healthconnectonfhir/src/main/java/edu/stanford/healthconnectonfhir/HealthConnectToFHIR.kt
Outdated
Show resolved
Hide resolved
...es/healthconnectonfhir/src/main/java/edu/stanford/healthconnectonfhir/HealthConnectToFHIR.kt
Outdated
Show resolved
Hide resolved
...healthconnectonfhir/src/test/java/edu/stanford/healthconnectonfhir/CreateObservationTests.kt
Outdated
Show resolved
Hide resolved
...healthconnectonfhir/src/test/java/edu/stanford/healthconnectonfhir/CreateObservationTests.kt
Outdated
Show resolved
Hide resolved
...healthconnectonfhir/src/test/java/edu/stanford/healthconnectonfhir/CreateObservationTests.kt
Outdated
Show resolved
Hide resolved
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.
Thank you for adding this in the @StanfordSpezi ecosystem @vishnuravi!
I added my thoughts in the discussions below, thank you @eldcn & @Basler182 for the first reviews!
...es/healthconnectonfhir/src/main/java/edu/stanford/healthconnectonfhir/HealthConnectToFHIR.kt
Outdated
Show resolved
Hide resolved
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #15 +/- ##
============================================
+ Coverage 19.38% 26.40% +7.02%
- Complexity 146 193 +47
============================================
Files 99 101 +2
Lines 2611 2860 +249
Branches 392 404 +12
============================================
+ Hits 506 755 +249
Misses 2063 2063
Partials 42 42
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report in Codecov by Sentry.
|
Thanks for the updates here @vishnuravi 🚀 |
We can merge it if everyone is OK with the updated method of implementing the mapper - can discuss at the meeting today. |
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.
LGTM 🚀
...onnectonfhir/src/main/java/edu/stanford/healthconnectonfhir/RecordToObservationMapperImpl.kt
Outdated
Show resolved
Hide resolved
...lthconnectonfhir/src/main/java/edu/stanford/healthconnectonfhir/RecordToObservationModule.kt
Outdated
Show resolved
Hide resolved
...nnectonfhir/src/test/java/edu/stanford/healthconnectonfhir/RecordToObservationMapperTests.kt
Outdated
Show resolved
Hide resolved
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.
Thank you @vishnuravi!
…e of record are equal
Create Health Connect to FHIR converter module
The HealthConnectOnFHIR library provides a mapper that converts supported Android Health Connect Records to corresponding HL7® FHIR® R4 Observations with standardized codes (e.g. LOINC).
For more information, please refer to the API documentation.
Mapping Table
Installation
HealthConnectOnFHIR can be installed into your Android Studio project via Jitpack.
Usage
Example
First, you will need to configure your application to use Android Health Connect. For more information, please see the official documentation.
📝 Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines: