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

implement claims-only hospitalization indicator #285

Merged
merged 6 commits into from
Oct 2, 2020
Merged

Conversation

mariajahja
Copy link
Member

@mariajahja mariajahja commented Sep 28, 2020

Hospitalization indicator based solely on claims based data. EMR data from HSP will be deprecated by Oct 1.

I have done checks in covidcast-indicators/claims_hosp/REVIEW.md, but of course will need a third-party code review. Much of the code is the same as the original delphi_emr_hosp package. In addition to logic changes to remove the EMR code, there has been some minor clean-up and naming changes (e.g. "sensor"->"indicator", "backfill"->"backwards_pad") as several terms have changed meaning over time. Pylint checks and unit tests are available.

Summary checks on differences between a claims+EMR signal and a claims-only signal:

  1. The claims' denominator stream is 4x larger than EMR (except in the most recent timepoints). Despite this, we will lose estimates in 5-7 states. Historically, those states were Alaska, Maine, Montana, North/South Dakota, Vermont, Wyoming. In counties, we will lose ~40-100 (out of 400-1000).
  2. The Spearman correlation over time (within each county individually) is comparable between a claims+EMR signal and an claims-only signal.
  3. Spearman correlation by ranked geographies for a claims-only signal is slightly lower, but still has reasonable performance.

Additional details are available in the HSP slack channel (#covid-19-hsp).

@krivard krivard requested a review from nmdefries September 28, 2020 16:58
@nmdefries nmdefries marked this pull request as ready for review September 28, 2020 17:08
Copy link
Contributor

@nmdefries nmdefries left a comment

Choose a reason for hiding this comment

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

Aside from the handful of minor errors (they don't appear to impact results), looks good. Nice documentation and use of classes.

Copy link
Contributor

@nmdefries nmdefries 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!

@krivard krivard merged commit 64eb227 into main Oct 2, 2020
@krivard krivard deleted the dev-hosp-claims branch October 29, 2020 18:12
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.

3 participants