Skip to content

Commit e88a26f

Browse files
authored
Merge pull request #129 from cmu-delphi/90-dont-filter-by-time_value-in-as_of
Removed time filter in relation to version in `as_of`
2 parents 8b0d699 + 41a6a5c commit e88a26f

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

R/archive.R

+2-4
Original file line numberDiff line numberDiff line change
@@ -213,10 +213,8 @@ epi_archive =
213213
# Filter by version and return
214214
return(
215215
# Make sure to use data.table ways of filtering and selecting
216-
self$DT[between(time_value,
217-
min_time_value,
218-
max_version) &
219-
version <= max_version, ] %>%
216+
self$DT[time_value >= min_time_value &
217+
version <= max_version, ] %>%
220218
unique(by = c("geo_value", "time_value", other_keys),
221219
fromLast = TRUE) %>%
222220
tibble::as_tibble() %>%

0 commit comments

Comments
 (0)