Pending updates to columns in db_signals.csv #1442
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR is derived from #1434; i removed all of the new columns but this should include all of the changes to the existing columns (except
Available Geography
, more on that in a bit).Please let me know if we need to fix any of these -- the summary of differences appears to me to be:
day
"' removed fromTime Type
column, replaced with empty string: (this seems like it was accidental)dsew-cpr
:confirmed_admissions_covid_1d_7dav
"n/a
" removed fromPathogen/Disease Area
column, replaced with empty string: (these seem intentional)nchs-mortality
:deaths_allcause_incidence_num
nchs-mortality
:deaths_allcause_incidence_prop
nchs-mortality
:deaths_percent_of_expected
safegraph-daily
:completely_home_prop
safegraph-daily
:completely_home_prop_7dav
safegraph-daily
:full_time_work_prop
safegraph-daily
:full_time_work_prop_7dav
safegraph-daily
:median_home_dwell_time
safegraph-daily
:median_home_dwell_time_7dav
safegraph-daily
:part_time_work_prop
safegraph-daily
:part_time_work_prop_7dav
safegraph-weekly
:bars_visit_num
safegraph-weekly
:bars_visit_prop
safegraph-weekly
:restaurants_visit_num
safegraph-weekly
:restaurants_visit_prop
The
Available Geography
column has some sweeping changes applied to it... In one example fromchng
, the text was modified fromcounty,hhs,hrr,msa,nation,state
tocounty, hrr (by Delphi), msa (by Delphi), state (by Delphi), hhs (by Delphi), nation (by Delphi)
. I believe this signifies that only county data came from the source, and we computed the various other higher levels of geo aggregation. This is valuable information, but i would suggest we keep the column the way it was and create a new column called something likeGeographies aggregated by Delphi
orPost-aggregated geographies
that lists the geography types that were extrapolated by us. There are a few reasons for doing it this way, including that (i believe) the Signal Documentation app expects the structured comma-separated text without the extra annotations as it was before, and that representing the same information in its own column should save some space. If you agree with this, let me know as i think i should be able to apply those changes pretty easily. Also, some entries (likequidel
for instance) have "(by Delphi)
" attached to every geography in the list; that suggests to me that we did aggregations to produce county-level data from finer-grained locations, but i didn't think that was the case.