Address CRAN review comments

- DESCRIPTION: remove single quotes around Boruta
- DESCRIPTION: add method references as authors (year) <doi:...>
- fs_bayes(): replace \dontrun{} with \donttest{}, guarded on brms and
  limited to two single-predictor fits via max_comb_size = 1
- inst/WORDLIST: add cited author surnames

Author: elkronos

DESCRIPTION

@@ -7,17 +7,24 @@ Authors@R:
 Description: Filter, wrapper, and embedded feature-selection methods behind a
     consistent set of functions that share one calling convention and one
     return type: correlation and chi-squared filters, information gain, LASSO
-    and elastic net, Bayesian model comparison, 'Boruta', recursive feature
+    and elastic net, Bayesian model comparison, Boruta, recursive feature
     elimination, random forest importance, multivariate adaptive regression
     splines, support vector machine recursive feature elimination, stepwise
     selection, and principal component / singular value decomposition
-    helpers. Heavy modeling engines are optional and only required by the
-    functions that use them.
+    helpers. The implemented methods follow
+    Tibshirani (1996) <doi:10.1111/j.2517-6161.1996.tb02080.x>,
+    Zou and Hastie (2005) <doi:10.1111/j.1467-9868.2005.00503.x>,
+    Friedman (1991) <doi:10.1214/aos/1176347963>,
+    Breiman (2001) <doi:10.1023/A:1010933404324>,
+    Guyon, Weston, Barnhill and Vapnik (2002) <doi:10.1023/A:1012487302797>,
+    Kursa and Rudnicki (2010) <doi:10.18637/jss.v036.i11>, and
+    Vehtari, Gelman and Gabry (2017) <doi:10.1007/s11222-016-9696-4>.
+    Heavy modeling engines are optional and only required by the functions
+    that use them.
 License: MIT + file LICENSE
 Encoding: UTF-8
 Language: en-US
 Roxygen: list(markdown = TRUE)
-RoxygenNote: 8.1.0
 Config/roxygen2/version: 8.1.0
 Depends:
     R (>= 4.1.0)

R/fs_bayes.R

@@ -656,24 +656,27 @@ bayes_pick_model <- function(results, idx, comparison = NULL,
 #' }
 #'
 #' @examples
-#' # Each candidate model compiles a Stan program, so this example is not run
-#' # automatically (compilation alone takes far longer than a typical example
-#' # budget). The same call is exercised by the package tests.
-#' \dontrun{
-#' x1 <- seq(-2, 2, length.out = 40)
-#' x2 <- rep(c(-1, 1), 20)
-#' d <- data.frame(
-#'   y  = 1 + 2 * x1 + sin(seq_len(40)),
-#'   x1 = x1,
-#'   x2 = x2
-#' )
-#' res <- fs_bayes(
-#'   d, target = "y", predictors = c("x1", "x2"),
-#'   brm_args = list(chains = 1, iter = 500, refresh = 0),
-#'   rule = "1se", verbose = FALSE
-#' )
-#' res$selected
-#' res$details$loo_comparison
+#' # Each candidate model compiles and samples its own Stan program, so the
+#' # call is guarded on brms being installed and kept to two single-predictor
+#' # fits via max_comb_size = 1.
+#' \donttest{
+#' if (requireNamespace("brms", quietly = TRUE)) {
+#'   x1 <- seq(-2, 2, length.out = 40)
+#'   x2 <- rep(c(-1, 1), 20)
+#'   d <- data.frame(
+#'     y  = 1 + 2 * x1 + sin(seq_len(40)),
+#'     x1 = x1,
+#'     x2 = x2
+#'   )
+#'   res <- fs_bayes(
+#'     d, target = "y", predictors = c("x1", "x2"),
+#'     max_comb_size = 1,
+#'     brm_args = list(chains = 1, iter = 500, refresh = 0),
+#'     rule = "1se", verbose = FALSE
+#'   )
+#'   print(res$selected)
+#'   print(res$details$loo_comparison)
+#' }
 #' }
 #' @export
 fs_bayes <- function(data,

inst/WORDLIST

@@ -3,8 +3,14 @@ AbsCoefficient
 Barnhill
 Boruta
 Boruta's
+Breiman
 CMRG
+Friedman
+Gabry
+Gelman
 Guyon
+Hastie
+Kursa
 L'Ecuyer
 MASS
 MLmetrics
@@ -18,7 +24,12 @@ Quinlan's
 RFE
 RSpectra
 Rsquared
+Rudnicki
+Tibshirani
 Vapnik
+Vehtari
+Weston
+Zou
 auc
 backticked
 bayes