Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Feb 9, 2023

dshemetov and others added 30 commits January 19, 2023 14:02
* rename RowValues to CsvRowValue
* make CsvRowValue into a dataclass
* remove argument in load_csv only used for test mocks; correctly mock test
* update tests
[Refactor] Clarify a few `csv_importer` types and names
* add type hints
* add consistent spacings
* simplify and unify CovidcastRow creation from file
* convert CSV file details tuple to NamedTuple
Fix mixed quotes

Co-authored-by: Katie Mazaitis <[email protected]>
* update covidcast_nowcast
* update tests
[Refactor] Further cleans of `csv_importer`
Document missingness of each signal
* no default values
* helper functions for creating rows
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 merged commit 15da9b4 into main Feb 9, 2023
@krivard krivard deleted the release/delphi-epidata-0.4.6 branch February 9, 2023 18:53
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.

5 participants