Skip to content

Conversation

dshemetov and others added 30 commits May 7, 2021 15:43
* update export utility to export, validate, and test the missing cols
* handle deleted rows: replaced with nan values
* handle deleted files: replace with an empty CSV file
* handle comparisons between CSVs with/without missing cols
Un-retire schooling indicators from sirCAL
Update setup.py files to "Python :: 3.8" annotation
Correctly ignore all receiving/*.csv files
And add log statements for data export variants
chinandrew and others added 21 commits September 21, 2021 21:42
Only install `remotes` if not in cache
Switch indicators from logging to delphi_utils structured logger
Refactor NCHS mortality to use delphi export util
Co-authored-by: Katie Mazaitis <[email protected]>
Co-authored-by: Katie Mazaitis <[email protected]>
Update archiver and export utils for nancodes and deletion-handling
Copy link
Contributor

@krivard krivard left a comment

Choose a reason for hiding this comment

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

👍

@krivard krivard marked this pull request as draft September 29, 2021 14:28
@krivard krivard marked this pull request as ready for review September 29, 2021 14:28
@krivard
Copy link
Contributor

krivard commented Sep 29, 2021

That's weird, checks didn't run until I converted to draft and back 😕

@krivard krivard merged commit 3ce0f48 into prod Sep 29, 2021
@krivard krivard deleted the release/indicators_v0.1.17_utils_v0.1.13 branch September 29, 2021 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants