search

smart-on-fhir / cumulus-library-data-metrics

A data metrics study for the Cumulus project
https://docs.smarthealthit.org/cumulus/
Apache License 2.0
2 stars 0 forks source link

DocumentReference attachment/url check #7

Open gotdan opened 5 months ago

gotdan commented 5 months ago

Currently, the us-core-6 attachment data/url check for DocumentReference is skipped since Cumulus strips that element. This makes sense once the data has been loaded, but it's an important check if we plan to process the notes coming from bulk data using NLP. Should we have a flag to enable this, or alternatively, add a "NLP ready" version of the metric that inherits from the US Core metric and adds this check?

mikix commented 5 months ago

We talked about this off-github and we agree there's probably utility in a docref attachment metric talking about content types and all that jazz - but it would need to run in a mode that operated on the original ndjson, not the post-ETL ndjson

mikix commented 4 months ago

Also see #23