complete.cases() -> vec_detect_complete()

[EmilHvitfeldt]

found during sparsity work. n_complete_cases() calls complete.cases() which is kinda slow compared to vctrs::vec_detect_complete().

hotel_data <- sparse_hotel_rates()
hotel_data <- sparsevctrs::coerce_to_sparse_tibble(hotel_data)

bench::mark(
  complete.cases(hotel_data),
  vctrs::vec_detect_complete(hotel_data)
)

#> # A tibble: 2 × 6
#>   expression                             min median `itr/sec` mem_alloc `gc/sec`
#>   <bch:expr>                          <bch:> <bch:>     <dbl> <bch:byt>    <dbl>
#> 1 complete.cases(hotel_data)           1.23s  1.23s     0.812      60MB        0
#> 2 vctrs::vec_detect_complete(hotel_d… 2.77ms 2.87ms   348.       66.5KB        0

[github-actions]

This pull request has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex https://reprex.tidyverse.org) and link to this issue.