fix(apply_compactify): arrange(!!!<name of chr vec>) is no-op

cmu-delphi · Jan 30, 2025 · d78ea5b · d78ea5b
1 parent d3f0a7a
commit d78ea5b
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/R/archive.R b/R/archive.R
@@ -420,7 +420,7 @@ apply_compactify <- function(updates_df, ukey_names, abs_tol = 0) {
   assert_numeric(abs_tol, len = 1, lower = 0)
 
   if (!is.data.table(updates_df) || !identical(key(updates_df), ukey_names)) {
-    updates_df <- updates_df %>% arrange(!!!ukey_names)
+    updates_df <- updates_df %>% arrange(pick(all_of(ukey_names)))
   }
   updates_df %>%
     filter(!update_is_locf(ukey_names, abs_tol))