Skip to content

Conversation

chinandrew
Copy link
Contributor

@chinandrew chinandrew commented Dec 9, 2020

Description

Merge after #610

Add HHS and nation to doctor visits

Like #436: I branched this off dv-package but had to merge in a more recent branch with main commits to get the most up to date geo utils, so the changes are unreadable. Could remedy this by merging main into dv-package. The main commit is 588f805

Changelog

Itemize code/test/documentation changes and files added/removed.

  • Update conditional statements to map to nation and hhs
  • Update documentation

Fixes

  • n/a

Jingjing Tang and others added 30 commits November 16, 2020 12:24
Aggregate SirCAL messages by source and error
Fix issues causing `google_health` tests to fail
Migrate google_health to use `delphi_utils.create_export_csv()`
* same megafips filter when converting to state as in JHU
@chinandrew chinandrew changed the title Dv hhs nation Add nation and HHS to doctor visits Dec 9, 2020
@chinandrew chinandrew marked this pull request as ready for review December 9, 2020 21:05
@chinandrew chinandrew requested review from a team and huisaddison and removed request for a team December 9, 2020 21:14
@huisaddison huisaddison requested review from a team and removed request for huisaddison December 10, 2020 06:01
@huisaddison
Copy link
Contributor

Sorry folks, I'm totally slammed so I am kicking the review request back to @cmu-delphi/covidcast-indicators

@chinandrew chinandrew requested review from a team and benjaminysmith and removed request for a team December 10, 2020 17:00
@krivard
Copy link
Contributor

krivard commented Mar 18, 2021

Replaced by #935

@krivard krivard closed this Mar 18, 2021
@krivard krivard deleted the dv-hhs-nation branch March 18, 2021 19:36
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.