-
Notifications
You must be signed in to change notification settings - Fork 16
GeoMapper doubles the effective population in state-level prop signals #325
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
Comments
Alright, so it's likely this change which switched us over to the new population source file. What's weird is that the previous source file included state FIPS in it as well, so I'm not sure why they're not being filtered out now. Spent a bit too long testing on |
Comparing the new population source file with the old one: in all the non-Puerto Rico counties, the total of all the absolute differences is 0.0 (which is just a verification that we're using the same 2019 estimates). The only change other change is that Puerto Rico FIPS now have population counts and the FIPS code is no longer a float. |
oh noooo that was me -- I merged #301 a week ago but never propagated into main. Since it's a breaking change though, I'll hold off until we have a fix. |
More information in this Slack thread, but the TL;DR is that JHU state-level
prop
signals are half the magnitude they should be, according to USAFacts and NYT. Since GeoMapper includes population for both state and FIPS, it may be summing both of them when computing stateprop
instead of just using one or the other.Might be related to Unassigned handling.
The text was updated successfully, but these errors were encountered: