Skip to content

Release covidcast-indicators 0.1.17 #1275

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

Merged
merged 60 commits into from
Sep 29, 2021

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.

6 participants