m

rhijmans · rhijmans · commit a3a15fcf4d14 · 2025-09-21T11:24:25.000-07:00
diff --git a/NEWS.md b/NEWS.md
@@ -16,7 +16,7 @@
 
 - when computing aggregated time steps such as days from POSIXct (seconds) time, terra now uses the date in the specified time zone, unlike base `as.Date` that seems to return the date in the UTC time zone [#1896](https://github.com/rspatial/terra/issues/1896) by Kodi Arfer
 - better support for creating a SpatVector with an EMPTY wkt geometry [#1903](https://github.com/rspatial/terra/issues/1903) by Anatolii Tsyplenkov
-- `makeTiles` gains argument "value" to allow the return value to be a SpatRaster or SpatRasterCollection [#1894](https://github.com/rspatial/terra/issues/1894) by Márcia Barbosa
+- `makeTiles` gains argument "value" to allow the return value to be the filenames (default), a SpatRaster or a SpatRasterCollection [#1894](https://github.com/rspatial/terra/issues/1894) by Márcia Barbosa
 
 ## new
 
diff --git a/R/tiles.R b/R/tiles.R
@@ -1,8 +1,8 @@
 
 setMethod("makeTiles", signature(x="SpatRaster"),
-	function(x, y, filename="tile_.tif", extend=FALSE, na.rm=FALSE, buffer=0, value="files", overwrite=FALSE, ...) {
+	function(x, y, filename="tile_.tif", extend=FALSE, na.rm=FALSE, buffer=0, ret="files", overwrite=FALSE, ...) {
 	
-		value <- match.arg(value, c("filenames", "raster", "collection"))
+		ret <- match.arg(ret, c("files", "raster", "collection"))
 		filename <- trimws(filename[1])
 		filename <- filename[!is.na(filename)]
 		if (filename == "") error("makeTiles", "filename cannot be empty")
@@ -22,9 +22,9 @@ setMethod("makeTiles", signature(x="SpatRaster"),
 			error("makeTiles", "y must be numeric or a SpatRaster or SpatVector")
 		}
 		messages(x, "makeTiles")
-		if (value == "files") {
+		if (ret == "files") {
 			ff
-		} else if (value == "raster") {
+		} else if (ret == "raster") {
 			filename <- gsub("_\\.", "s.", filename)
 			vrt(ff, filename=filename)
 		} else {
diff --git a/man/makeTiles.Rd b/man/makeTiles.Rd
@@ -20,7 +20,7 @@ Divide a SpatRaster into "tiles". The cells of another SpatRaster (normally with
 
 \usage{
 \S4method{makeTiles}{SpatRaster}(x, y, filename="tile_.tif", extend=FALSE,
-		na.rm=FALSE, buffer=0, value="files", overwrite=FALSE, ...)
+		na.rm=FALSE, buffer=0, ret="files", overwrite=FALSE, ...)
 
 
 \S4method{getTileExtents}{SpatRaster}(x, y, extend=FALSE, buffer=0)
@@ -33,7 +33,7 @@ Divide a SpatRaster into "tiles". The cells of another SpatRaster (normally with
   \item{extend}{logical. If \code{TRUE}, the extent of \code{y} is expanded to assure that it covers all of \code{x}}
   \item{na.rm}{logical. If \code{TRUE}, tiles with only missing values are ignored}
   \item{buffer}{integer. The number of additional rows and columns added to each tile. Can be a single number, or two numbers to specify a separate number of rows and columns. This allows for creating overlapping tiles that can be used for computing spatial context dependent values with e.g. \code{\link{focal}}. The expansion is only inside \code{x}, no rows or columns outside of \code{x} are added}
-  \code{value}{character. The type of rerturn value desired. Either "filenames", "raster" (for a SpatRaster), or "collection" (for a SpatRasterCollection)}
+  \code{ret}{character. The type of return value desired. Either "files" (for the filenames), "raster" (for a SpatRaster), or "collection" (for a SpatRasterCollection)}
   \item{overwrite}{logical. If \code{TRUE}, existing tiles are overwritten; otherwise they are skipped (without error or warning)}
   \item{...}{additional arguments for writing files as in \code{\link{writeRaster}}}
 }
