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

Fix: Use non-Hispanic data for Black, Asian, and other race estimates #146

Merged
merged 5 commits into from
Apr 4, 2024

Conversation

rossellhayes
Copy link
Contributor

This PR updates the tables pulled by census_geo_api(). It will now pull non-Hispanic tables to estimate Black, Asian, and other race populations. Previously, Hispanic respondents could be double counted because they may count towards both the Hispanic population and the Black, Asian, or other race populations of a geography.

Fixes #145.

@kosukeimai
Copy link
Owner

@etrrosenman Please let us know if this looks good to you. Thanks.

Copy link
Collaborator

@etrrosenman etrrosenman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@etrrosenman
Copy link
Collaborator

etrrosenman commented Apr 3, 2024 via email

@1beb
Copy link
Collaborator

1beb commented Apr 4, 2024

We also did some due diligence by comparing estimates at state level to TargetSmart Voter File Race and CPS 2022 targets. They look better than previous, with the black and hispanic registered voters having more reasonable proportions. I'm going to merge this branch, then submit as a bugfix to CRAN.

@1beb 1beb merged commit 439e10d into kosukeimai:main Apr 4, 2024
1 check passed
@rossellhayes rossellhayes mentioned this pull request Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Census variable selection?
4 participants