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.

Sign up for GitHub

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

Jump to bottom

Add NAN code support to doctors visits #990

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

Add NAN code support to doctors visits #990

wants to merge 1 commit into from

Conversation

dshemetov
Copy link
Contributor

@dshemetov dshemetov commented Apr 7, 2021 •
edited

Description

Work as part #838.

Changelog

  • 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

Fixes

@dshemetov dshemetov mentioned this pull request Apr 7, 2021
Open
17 tasks
@dshemetov dshemetov force-pushed the nans_dv branch 2 times, most recently from a4f3dcc to 4b8d2a2 Compare April 27, 2021 19:28
@dshemetov
NANS doctor visits:
f9bdf28 
* write missing columns
* update test files
@dshemetov dshemetov requested review from a team and vicdunmin and removed request for a team November 10, 2021 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vicdunmin vicdunmin Awaiting requested review from vicdunmin vicdunmin was automatically assigned from cmu-delphi/covidcast-engineering

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@dshemetov