Skip to content

Commit ba78d43

Browse files
authored
Merge pull request #156 from cmu-delphi/ndefries/na-time-printing
Ignore `NA`s when printing an archive's min/max `time_value`
2 parents 7c1e6da + faec0c3 commit ba78d43

File tree

2 files changed

+13
-7
lines changed

2 files changed

+13
-7
lines changed

R/archive.R

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -188,10 +188,16 @@ epi_archive =
188188
})
189189
}
190190
cat("----------\n")
191+
if (length(self$DT$time_value) == 0 || all(is.na(self$DT$time_value))) {
192+
min_time = max_time = NA
193+
} else {
194+
min_time = Min(self$DT$time_value)
195+
max_time = Max(self$DT$time_value)
196+
}
191197
cat(sprintf("* %-14s = %s\n", "min time value",
192-
min(self$DT$time_value)))
198+
min_time))
193199
cat(sprintf("* %-14s = %s\n", "max time value",
194-
max(self$DT$time_value)))
200+
max_time))
195201
cat(sprintf("* %-14s = %s\n", "min version",
196202
min(self$DT$version)))
197203
cat(sprintf("* %-14s = %s\n", "max version",

man/as_epi_df.Rd

Lines changed: 5 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)