Skip to content

Commit 93b0f76

Browse files
committed
Check if dataset has zero rows to avoid warning message while filtering
1 parent 640a80e commit 93b0f76

1 file changed

Lines changed: 13 additions & 13 deletions

File tree

R/helper_functions.R

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,9 @@ prepare_initial_data <- function(
6767
)
6868
)
6969
})
70-
70+
if (nrow(sel_dataset_list[[1]]) == 0) {
71+
return(NULL)
72+
}
7173
dataset <- Reduce(dplyr::full_join, sel_dataset_list) %>%
7274
dplyr::filter(.data[[lb_test_var]] %in% lb_test_choices) %>%
7375
dplyr::group_by(.data[[subjectid_var]], .data[[arm_var]], .data[[lb_test_var]], .data[[visit_var]]) %>%
@@ -114,12 +116,13 @@ prepare_initial_data <- function(
114116
#'
115117
#' @keywords internal
116118
filter_data <- function(dataset, arm_var, sel_arm, lb_test_var, sel_lb_test) {
117-
dataset <- dataset %>%
118-
dplyr::filter(
119-
.data[[lb_test_var]] %in% sel_lb_test,
120-
.data[[arm_var]] %in% sel_arm
121-
)
122-
119+
if (!is.null(dataset)) {
120+
dataset <- dataset %>%
121+
dplyr::filter(
122+
.data[[lb_test_var]] %in% sel_lb_test,
123+
.data[[arm_var]] %in% sel_arm
124+
)
125+
}
123126
return(dataset)
124127
}
125128

@@ -172,11 +175,8 @@ derive_req_vars <- function(
172175
ref_range_upper_lim_var,
173176
sel_x,
174177
sel_y) {
175-
if (nrow(dataset) == 0) {
176-
return(NULL)
177-
}
178-
179-
# Get the data frame in required structure (Pivot wider grouped by certain variables)
178+
if (!is.null(dataset)) {
179+
# Get the data frame in required structure (Pivot wider grouped by certain variables)
180180
dataset <- dataset %>%
181181
dplyr::filter(.data[[lb_test_var]] %in% c(sel_x, sel_y)) %>%
182182
dplyr::mutate(
@@ -194,7 +194,7 @@ derive_req_vars <- function(
194194
"r_Baseline_{{sel_x}}" = as.numeric(.data[[paste0("r_Baseline_", sel_x)]]),
195195
"r_Baseline_{{sel_y}}" = as.numeric(.data[[paste0("r_Baseline_", sel_y)]])
196196
)
197-
197+
}
198198
return(dataset)
199199
}
200200

0 commit comments

Comments
 (0)