postProcess operations not being cached when nested in prepInputs call?

[CeresBarros]

I've noticed that my crop/mask/projection operations do not seem to be cached when running a prepInputs call:

reprex:

Require::Require("PredictiveEcology/reproducible")

options(reproducible.destinationPath = "~/reproducible_test/data",
        reproducible.cachePath = "~/reproducible_test/cache")

shpUrl <- "http://sis.agr.gc.ca/cansis/nsdb/ecostrat/zone/ecozone_shp.zip"

studyArea <- SpaDES.tools::randomStudyArea(size = 20000)

ecozoneFilename <- file.path(options("reproducible.cachePath"), "ecozones.shp")
ecozoneFiles <- c(
  "ecozones.dbf", "ecozones.prj",
  "ecozones.sbn", "ecozones.sbx", "ecozones.shp", "ecozones.shx"
)

shpEcozone <- prepInputs(
  url = shpUrl,
  targetFile = reproducible::asPath(ecozoneFilename),
  alsoExtract = reproducible::asPath(ecozoneFiles),
  to = studyArea
)

shpEcozone <- prepInputs(
  url = shpUrl,
  targetFile = reproducible::asPath(ecozoneFilename),
  alsoExtract = reproducible::asPath(ecozoneFiles),
  to = studyArea
)

[eliotmcintire]

When terra was used under the hood, most (all?) of the internal caching was removed as it tended to be less effective as the terra operations were so much faster. I believe the best option is to cache the entire call:

shpEcozone <- prepInputs(
  url = shpUrl,
  targetFile = reproducible::asPath(ecozoneFilename),
  alsoExtract = reproducible::asPath(ecozoneFiles),
  to = studyArea
) |>
Cache()

[CeresBarros]

I see -- I either forgot or didn't realise that. In that case the documentation for prepInputs(..., useCache) needs to be clarified as it currently suggests caching will happen up on these operations. I'm happy to PR a doc fix -- is development still the best branch to PR to?

[eliotmcintire]

Yes. Please use @modsForLargeArchives for PR.

[CeresBarros]

please ignore first PR