Skip to content

Commit edafff2

Browse files
authored
Merge pull request #104 from cmu-delphi/dev-explicit-cols
Explicitly pick out used columns
2 parents 32adef0 + eff19fe commit edafff2

File tree

1 file changed

+12
-3
lines changed

1 file changed

+12
-3
lines changed

dashboard/app.R

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -208,9 +208,18 @@ server <- function(input, output, session) {
208208
dfStateDeaths <- getData("score_cards_state_deaths.rds")
209209
dfNationCases = getData("score_cards_nation_cases.rds")
210210
dfNationDeaths = getData("score_cards_nation_deaths.rds")
211-
# TODO handle this kind of error better
212-
dfNationDeaths = subset(dfNationDeaths, select = -c(full_location_name))
213-
dfNationCases = subset(dfNationCases, select = -c(full_location_name))
211+
212+
# Pick out expected columns only
213+
covCols = paste0("cov_", COVERAGE_INTERVALS)
214+
expectedCols = c("ahead", "geo_value", "forecaster", "forecast_date",
215+
"data_source", "signal", "target_end_date", "incidence_period",
216+
"actual", "wis", "ae",
217+
covCols)
218+
dfStateCases = dfStateCases %>% select(all_of(expectedCols))
219+
dfStateDeaths = dfStateDeaths %>% select(all_of(expectedCols))
220+
dfNationCases = dfNationCases %>% select(all_of(expectedCols))
221+
dfNationDeaths = dfNationDeaths %>% select(all_of(expectedCols))
222+
214223
df <- rbind(dfStateCases, dfStateDeaths, dfNationCases, dfNationDeaths)
215224
df <- df %>% rename("10" = cov_10, "20" = cov_20, "30" = cov_30, "40" = cov_40, "50" = cov_50, "60" = cov_60, "70" = cov_70, "80" = cov_80, "90" = cov_90, "95" = cov_95, "98" = cov_98)
216225

0 commit comments

Comments
 (0)