Skip to content
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

Consider examining "text" in race/ethnicity extensions #222

Open
mikix opened this issue Apr 24, 2024 · 1 comment
Open

Consider examining "text" in race/ethnicity extensions #222

mikix opened this issue Apr 24, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@mikix
Copy link
Contributor

mikix commented Apr 24, 2024

The race and ethnicity extensions offer several levels of detail: ombCategory codings (10,000 foot view), detailed codings (nitty gritty), and text string (:shrug:).

We currently only examine ombCategory and detailed codings (preferring ombCategory values). And only actually use the display values found there.

Should we also examine text.valueString? text is actually the only required field and we only use display values anyway. But... who knows what the data quality is like there.

A part of me doesn't even like looking at display values, instead preferring to stick to the ombCategory values, looking at the code/system, and providing the string ourselves. That way, we don't rely on any weird EHR presentations of the strings. But maybe that's too paranoid. (Note: I did get that paranoid though for the data-metrics study)

@mikix mikix added the enhancement New feature or request label Apr 24, 2024
@dogversioning
Copy link
Contributor

we talked about this off thread, and the preferred approach to this, and more generally for extensions that are well defined, is to create a column for each possible field - this allows for maximal downstream flexibility.

so in this case we would have race_ombSystem_code, race_ombStytem_display (and perhaps a race_ombSystem_version and race_ombSystem_userSelected) from the ValueCoding, a similar set for race_detailed_*, and a race_text from the ValueString.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants