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

Support chng-fips county group reporting in chng for privacy #1883

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

nmdefries
Copy link
Contributor

Description

Report signals for chng-fips county groups instead of counties. See the geomapper support PR for more context.

Turn off sirCAL alerting for county versions of all signals using this format.

Big Note

From local testing, it looks like the new county groups are never actually reported... This doesn't seem to be a definitional issue, that is, theoretically county groups could be reported if they had enough total claims reported. But because the county groups are low-population areas in general, they tend not to have enough total claims reported to be included in the output, based on our filtering logic.

Do we want to do more investigation here? I started in on population comparisons (do counties of similar population to county groups also get filtered out? Some megacounties do, at least) but didn't do anything super formal or thorough.

Non-adjusted county and chng-fips values match exactly (although county files have an extra entry for a county called "\N"??); adjusted county and chng-fips values are very similar but not exactly the same.

Changelog

  • update_sensor.py
  • Local and production template params for chng and sirCAL.

@nmdefries
Copy link
Contributor Author

nmdefries commented Aug 4, 2023

@jingjtang The code is ready, but wanted to get your thoughts on the data note above. Seems like we're going to a lot of work to report data at this new county group level but it very very rarely actually gets reported (might get reported during an upswing in cases?).

@nmdefries
Copy link
Contributor Author

Need to incorporate county groups format.

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.

None yet

1 participant