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

Do not include default races and ethnicities in locales #99

Merged
merged 2 commits into from
Nov 19, 2023

Conversation

xispa
Copy link
Member

@xispa xispa commented Nov 15, 2023

Description of the issue/feature this PR addresses

This Pull Request removes the list of races and ethnicities set by default from locales. Note this PR only removes from locales, but these ethnicities and races are still kept as default options on install. User can add new and remove existing ones at will after install.

Races

The races defined in senaite.patient by default are those as default in HL7 (by CDC):
https://phinvads.cdc.gov/vads/ViewValueSet.action?oid=2.16.840.1.114222.4.11.6065

However, this list was "developed" in 1997 by the US Office of Management and Budget (OMB) and is still used to describe groups of individuals to which individuals belong in the eyes of the (US) community. Therefore, this list is strongly biased and does not make sense to most places outside US. In addition, anthropologists nowadays use 3 or 4 main groups of races: "Caucasoid", "Mongoloid" and "Negroid". Each of these can be subdivided to up to 30 subrace. E.g. Eskimo, American Indian, Maori, etc. for Mongoloid group).

Ethnicities

Like in reaces, the default list of ethnicities in senaite.patient is the list set by default for HL7:
https://terminology.hl7.org/ValueSet-v3-Ethnicity.html

To start with, the standards specify two minimum ethnicity categories: Hispanic or Latino, and Not Hispanic or Latino. Well, what about Asian ethnicities? Again, this list comes from the default list from US CDC (see https://www.cdc.gov/nchs/data/dvs/race_ethnicity_codeset.pdf) and therefore, is strongly biased and not useful for most places located outside of the US.

Current behavior before PR

Locales contains races and ethnicities that are meaningless outside of the US

Desired behavior after PR is merged

Locales do not contain races and ethnicities that are meaningless outside of the US

--
I confirm I have tested this PR thoroughly and coded it according to PEP8
and Plone's Python styleguide standards.

@xispa xispa requested a review from ramonski November 15, 2023 10:40
@ramonski ramonski merged commit ed0cbf9 into master Nov 19, 2023
@ramonski ramonski deleted the locales-races-ethnicities branch November 19, 2023 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants