Clarified documentation for prepInputs and postProcess

R/postProcessTo.R

@@ -666,21 +666,18 @@ projectTo <- function(from, projectTo, overwrite = FALSE,
           if (!terra::is.lonlat(from)) {
             # if (sf::st_crs("epsg:4326") != sf::st_crs(from)) {
             newRes <- terra::res(from)
-            messagePreProcess("Using resolution of ", paste(newRes, collapse = "x"), "m; ",
+            messagePreProcess("Using original resolution (", paste(newRes, collapse = "x"), "m) ",
                               verbose = verbose
             )
             projectTo <- terra::rast(projectTo, resolution = newRes)
           } else {
             projectTo <- terra::crs(projectTo)
           }
 
-          messagePreProcess("in the projection of `projectTo`, using the origin and extent",
+          messagePreProcess("Using the origin and extent from `ext(from)` in the projection from `crs(projectTo)`.",
                             verbose = verbose
           )
-          messagePreProcess("from `ext(from)` (in the projection from `projectTo`).",
-                            verbose = verbose
-          )
-          messagePreProcess("If this is not correct, create a template gridded object and pass that to projectTo...",
+          messagePreProcess("If this is not correct, create a template gridded object and pass that to `projectTo`...",
                             verbose = verbose
           )
           messagePreProcess("",
@@ -758,7 +755,7 @@ projectTo <- function(from, projectTo, overwrite = FALSE,
   from
 }
 
-#' @param needBuffer Logical. Defaults to `TRUE`, meaning nothing is done out
+#' @param needBuffer Logical. Defaults to `FALSE`, meaning nothing is done out
 #'   of the ordinary. If `TRUE`, then a buffer around the cropTo, so that if a reprojection
 #'   has to happen on the `cropTo` prior to using it as a crop layer, then a buffer
 #'   of 1.5 * res(cropTo) will occur prior, so that no edges are cut off.

R/prepInputs.R

@@ -191,9 +191,11 @@ utils::globalVariables(c(
 #'   Defaults to `getOption("reproducible.useCache", 2L)` in `prepInputs`, and
 #'   `getOption("reproducible.useCache", FALSE)` if calling any of the inner
 #'   functions manually. For `prepInputs`, this mean it will use `Cache`
-#'   only up to 2 nested levels, which will generally including `postProcess` and
-#'   the first level of `*Input` functions, e.g., `cropInputs`, `projectInputs`,
-#'   `maskInputs`, but not `fixErrors`.
+#'   only up to 2 nested levels, which includes `preProcess`. `postProcess` and
+#'   its nested `*Input` functions (e.g., `cropInputs`, `projectInputs`,
+#'   `maskInputs`) are no longer internally cached, as `terra` processing speeds
+#'   mean internal caching is more time consuming. We recommend caching the full
+#'   `prepInputs` call instead (e.g. `prepInputs(...) |> Cache()`).
 #'
 #' @param .tempPath Optional temporary path for internal file intermediate steps.
 #'   Will be cleared on.exit from this function.