Skip to content

Commit

Permalink
Documentation updates and typo corrections
Browse files Browse the repository at this point in the history
  • Loading branch information
ShiyuC committed Dec 14, 2023
1 parent eb1b6de commit 33df5cc
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
8 changes: 4 additions & 4 deletions R/apply_conditional_format.R
Original file line number Diff line number Diff line change
Expand Up @@ -11,15 +11,15 @@
#' @noRd
validate_conditional_format_params <- function(string, format_group, condition, replacement, full_string) {
if (!inherits(string, "character")) {
stop("Paramter `string` must be a character vector", call.=FALSE)
stop("Parameter `string` must be a character vector", call.=FALSE)
}

if (!inherits(format_group, "numeric") || (inherits(format_group, "numeric") && format_group %% 1 != 0)) {
stop("Paramter `format_group` must be an integer", call.=FALSE)
stop("Parameter `format_group` must be an integer", call.=FALSE)
}

if (!inherits(replacement, "character")) {
stop("Paramter `replacement` must be a string", call.=FALSE)
stop("Parameter `replacement` must be a string", call.=FALSE)
}

# Condition statement must use the variable name 'x'
Expand All @@ -28,7 +28,7 @@ validate_conditional_format_params <- function(string, format_group, condition,
}

if (!inherits(full_string, "logical")) {
stop("Paramter `full_string` must be bool", call.=FALSE)
stop("Parameter `full_string` must be bool", call.=FALSE)
}

}
Expand Down
2 changes: 2 additions & 0 deletions vignettes/table.Rmd
Original file line number Diff line number Diff line change
Expand Up @@ -115,6 +115,8 @@ header_n(t) %>%
kable()
```

Note: it’s expected the set_distinct_by() function is used with population data. This is because it does not make sense to use population data denominators unless you have distinct counts. The entire point of population data is to use subject counts, so non-distinct counts would potentially count multiple records per subject and then the percentage doesn’t make any sense.

With the table level settings under control, now you're ready to learn more about what **Tplyr** has to offer in each layer.

- Learn more about descriptive statistics layers in `vignette("desc")`
Expand Down

0 comments on commit 33df5cc

Please sign in to comment.