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

Add NAN code support to claims_hosp #902

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dshemetov
Copy link
Contributor

@dshemetov dshemetov commented Mar 10, 2021

Description

Work as part #838.

Changelog

  • Now instead of the include flag not writing a CSV file, a CSV file is still written but with all the columns NAN
  • Adds missingness code columns; writes a privacy code for se and sample_size by default, writes a data insufficient code if the value doesn't have the include flag
  • Should naturally handle divide by zero cases in the daily rate calculation by marking them with insufficient data.
  • Source nans were already zero-filled on data load (not changed in this PR). A different imputation may be worth trying if we have issues in the future.

Fixes

@dshemetov dshemetov mentioned this pull request Mar 10, 2021
17 tasks
@dshemetov dshemetov force-pushed the nans_claims_hosp branch 3 times, most recently from d10a58f to e3aa0bf Compare April 27, 2021 01:30
@dshemetov dshemetov requested review from a team and alexcoda and removed request for a team November 9, 2021 21:01
* add missing columns, allow nan values
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