From 45eb76638518c2fc23f6f16b07ffc9e0ff2d9847 Mon Sep 17 00:00:00 2001 From: sorhawell Date: Sat, 14 Oct 2023 12:40:47 +0200 Subject: [PATCH] try remove braces --- R/expr__expr.R | 15 ++++++++------- man/Expr_rolling_std.Rd | 8 ++++---- 2 files changed, 12 insertions(+), 11 deletions(-) diff --git a/R/expr__expr.R b/R/expr__expr.R index e0060c21b..23ad981ed 100644 --- a/R/expr__expr.R +++ b/R/expr__expr.R @@ -3042,10 +3042,10 @@ Expr_rolling_sum = function( #' @param center #' Set the labels at the center of the window #' @param by -#' If the `window_size` is temporal for instance `"5h"` or `"3s`, you must +#' If the `window_size` is temporal for instance `"5h"` or `"3s"`, you must #' set the column that will be used to determine the windows. This column must -#' be of dtype `{Date, Datetime}` -#' @param closed : `{'left', 'right', 'both', 'none'}` +#' be of DataType: Date or DateTime. +#' @param closed string option `c("left", "right", "both", "none")`. #' Define whether the temporal window interval is closed or not. #' #' @@ -3066,13 +3066,14 @@ Expr_rolling_std = function( min_periods = NULL, center = FALSE, # :bool, by = NULL, # : Nullable, - closed = "left" # ;: Nullable, + closed = c("left", "right", "both", "none") ) { wargs = prepare_rolling_window_args(window_size, min_periods) - unwrap(.pr$Expr$rolling_std( + .pr$Expr$rolling_std( self, wargs$window_size, weights, - wargs$min_periods, center, by, closed - )) + wargs$min_periods, center, by, closed[1L] + ) |> + unwrap("in $rolling_std(): ") } #' Rolling var diff --git a/man/Expr_rolling_std.Rd b/man/Expr_rolling_std.Rd index bad50284a..b42617e09 100644 --- a/man/Expr_rolling_std.Rd +++ b/man/Expr_rolling_std.Rd @@ -10,7 +10,7 @@ Expr_rolling_std( min_periods = NULL, center = FALSE, by = NULL, - closed = "left" + closed = c("left", "right", "both", "none") ) } \arguments{ @@ -40,11 +40,11 @@ a result. If None, it will be set equal to window size.} \item{center}{Set the labels at the center of the window} -\item{by}{If the \code{window_size} is temporal for instance \code{"5h"} or \verb{"3s}, you must +\item{by}{If the \code{window_size} is temporal for instance \code{"5h"} or \code{"3s"}, you must set the column that will be used to determine the windows. This column must -be of dtype \verb{\{Date, Datetime\}}} +be of DataType: Date or DateTime.} -\item{closed}{: \verb{\{'left', 'right', 'both', 'none'\}} +\item{closed}{string option \code{c("left", "right", "both", "none")}. Define whether the temporal window interval is closed or not.} } \value{