Skip to content

Commit

Permalink
Temp remove use of arrow tables until NAs match (as they do in dplyr)…
Browse files Browse the repository at this point in the history
…. Related to apache/arrow#14907
  • Loading branch information
annakrystalli committed Sep 27, 2023
1 parent c7a1da3 commit d1e2861
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions R/check_tbl_values_required.R
Original file line number Diff line number Diff line change
Expand Up @@ -22,14 +22,13 @@ check_tbl_values_required <- function(tbl, round_id, file_path, hub_path) {
round_id = round_id,
required_vals_only = FALSE,
all_character = TRUE,
as_arrow_table = TRUE,
as_arrow_table = FALSE,
bind_model_tasks = FALSE
)

tbl <- purrr::map(
full,
~ dplyr::inner_join(.x, tbl, by = names(tbl))[, names(tbl)] %>%
tibble::as_tibble()
~ dplyr::inner_join(.x, tbl, by = names(tbl))[, names(tbl)]
)

missing_df <- purrr::pmap(
Expand Down Expand Up @@ -183,7 +182,7 @@ missing_req_rows <- function(opt_cols, x, mask, req, full, split_req = FALSE) {
# avoids erroneously returning missing required values that are not applicable
# to a given model task or output type.
expected_req <- dplyr::inner_join(req,
tibble::as_tibble(applicaple_full[, names(req)]),
applicaple_full[, names(req)],
by = names(req)
) %>%
unique()
Expand All @@ -200,7 +199,7 @@ missing_req_rows <- function(opt_cols, x, mask, req, full, split_req = FALSE) {
unique(x[, opt_cols])
)[, names(x)]
} else {
tibble::as_tibble(full[1, names(x)])[0, ]
full[1, names(x)][0, ]
}
}

Expand Down

0 comments on commit d1e2861

Please sign in to comment.