Merge pull request #446 from ishaan-arora-1/fix/deprecated-size-and-dead-code-444-445

Remove dead modify_aes_() and fix deprecated size usage in helpers

Author: jgabry

DESCRIPTION

@@ -34,6 +34,7 @@ Imports:
     ggplot2 (>= 3.4.0),
     ggridges (>= 0.5.5),
     glue,
+    lifecycle,
     posterior,
     reshape2,
     rlang (>= 0.3.0),

NAMESPACE

@@ -226,3 +226,6 @@ importFrom(dplyr,top_n)
 importFrom(dplyr,ungroup)
 importFrom(dplyr,vars)
 importFrom(ggplot2,"%+replace%")
+importFrom(lifecycle,deprecate_warn)
+importFrom(lifecycle,deprecated)
+importFrom(lifecycle,is_present)

R/bayesplot-helpers.R

@@ -446,19 +446,28 @@ plot_bg <- function(on = TRUE, ...) {
 
 #' @rdname bayesplot-helpers
 #' @export
-#' @param color,size Passed to [ggplot2::element_line()].
-#'
-grid_lines <- function(color = "gray50", size = 0.2) {
+#' @param color,linewidth Passed to [ggplot2::element_line()].
+#' @param size `r lifecycle::badge("deprecated")` Use `linewidth` instead.
+#'
+grid_lines <- function(color = "gray50", linewidth = 0.2, size = deprecated()) {
+  if (lifecycle::is_present(size)) {
+    lifecycle::deprecate_warn(
+      "1.16.0",
+      "grid_lines(size)",
+      "grid_lines(linewidth)"
+    )
+    linewidth <- size
+  }
   theme(
-    panel.grid.major = element_line(color = color, linewidth = size),
-    panel.grid.minor = element_line(color = color, linewidth = size * 0.5)
+    panel.grid.major = element_line(color = color, linewidth = linewidth),
+    panel.grid.minor = element_line(color = color, linewidth = linewidth * 0.5)
   )
 }
 
-grid_lines_y <- function(color = "gray50", size = 0.2) {
+grid_lines_y <- function(color = "gray50", linewidth = 0.2) {
   theme(
-    panel.grid.major.y = element_line(color = color, linewidth = size),
-    panel.grid.minor.y = element_line(color = color, linewidth = size * 0.5)
+    panel.grid.major.y = element_line(color = color, linewidth = linewidth),
+    panel.grid.minor.y = element_line(color = color, linewidth = linewidth * 0.5)
   )
 }
 

R/bayesplot-package.R

@@ -5,6 +5,7 @@
 #'
 #' @import ggplot2 stats rlang
 #' @importFrom dplyr %>% summarise group_by select
+#' @importFrom lifecycle deprecated deprecate_warn is_present
 #'
 #' @description
 #' \if{html}{

R/helpers-gg.R

@@ -24,17 +24,10 @@ geom_ignore <- function(...) {
 
 #' Add new aesthetic mappings to a list of aesthetic mappings
 #'
-#' @param mapping a list of `uneval` aesthetic mappings (created by `aes_()`)
-#' @param ... additional mappings to add, e.g., `color = ~ parameter`
+#' @param mapping a list of `uneval` aesthetic mappings (created by `aes()`)
+#' @param ... additional mappings to add using `.data$` syntax
 #' @return the updated list
 #' @noRd
-modify_aes_ <- function(mapping, ...) {
-  utils::modifyList(mapping, aes_(...))
-}
-
-#' Same as `modify_aes_` but using `aes()` instead of `aes_()` (now deprecated).
-#' Often `...` will need to contain expression of the form `.data$x` to avoid R cmd check warnings
-#' @noRd
 modify_aes <- function(mapping, ...) {
   utils::modifyList(mapping, aes(...))
 }

R/mcmc-diagnostics.R

@@ -77,7 +77,7 @@
 #' (p <- mcmc_acf_bar(x, pars = c("alpha", "beta[1]")))
 #'
 #' # add horiztonal dashed line at 0.5
-#' p + hline_at(0.5, linetype = 2, size = 0.15, color = "gray")
+#' p + hline_at(0.5, linetype = 2, linewidth = 0.15, color = "gray")
 #' }
 #'
 #' # fake rhat values to use for demonstration

R/mcmc-scatterplots.R

@@ -109,7 +109,7 @@
 #' color_scheme_set("pink")
 #' (p3 <- mcmc_scatter(x, pars = c("alpha", "beta[3]"), alpha = 0.25, size = 3))
 #' p3 + geom_smooth(method = "lm", se = FALSE, color = "gray20",
-#'                  size = .75, linetype = 2)
+#'                  linewidth = .75, linetype = 2)
 #'
 #' \donttest{
 #' if (requireNamespace("hexbin", quietly = TRUE)) {