Skip to content
Merged
Show file tree
Hide file tree
Changes from 13 commits
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
c11fa2a
various fixes cherrypicked
gmbecker Jun 4, 2026
7bea7c6
RowsVerticalSection and BBB vignettes plus needed fixes
gmbecker Jun 4, 2026
9ca6053
[no spell] update pkgdown file
gmbecker Jun 4, 2026
d76b167
[skip style] [skip vbump] Restyle files
github-actions[bot] Jun 4, 2026
e58e59b
reapply suggested vapply tweak
gmbecker Jun 4, 2026
756499f
Merge remote-tracking branch 'origin/1092_add_guided_advanced' into 1…
gmbecker Jun 4, 2026
8654a7a
[skip roxygen] [skip vbump] Roxygen Man Pages Auto Update
github-actions[bot] Jun 4, 2026
b6a59a3
Add vignette stubs for remaining advanced tour
gmbecker Jun 4, 2026
f59700e
collapse bbb vignettes into one
gmbecker Jun 4, 2026
eb24bd0
update pkgdown [no spell]
gmbecker Jun 4, 2026
4093d03
update NEWS.md
gmbecker Jun 4, 2026
ddfd1f5
Merge remote-tracking branch 'origin/1092_add_guided_advanced' into 1…
gmbecker Jun 4, 2026
1abc270
[skip style] [skip vbump] Restyle files
github-actions[bot] Jun 4, 2026
5bf9b99
new intro vignettes moving towards new nesting doc
gmbecker Jul 15, 2026
4fe7dc6
Merge branch 'main' into 1092_add_guided_advanced
gmbecker Jul 24, 2026
4b4efa1
[skip style] [skip vbump] Restyle files
github-actions[bot] Jul 24, 2026
2e497d3
add some rvs tests
gmbecker Jul 24, 2026
d1e6aea
fix guided_intro_basics.Rmd code, NEWS file, lint
gmbecker Jul 24, 2026
3df80a8
fix missing newline, few more tests.
gmbecker Jul 24, 2026
2c49c56
[skip style] [skip vbump] Restyle files
github-actions[bot] Jul 24, 2026
4327c8b
fixes to pkgdown and new vignettes
gmbecker Jul 24, 2026
a6edbff
Merge remote-tracking branch 'origin/1092_add_guided_advanced' into 1…
gmbecker Jul 24, 2026
dfdbf48
Merge branch 'main' into 1092_add_guided_advanced
gmbecker Jul 27, 2026
36718bd
Merge branch 'main' into 1092_add_guided_advanced
shajoezhu Jul 29, 2026
d291524
wrap RowsVerticalSection in tics in roxygen comments
gmbecker Jul 29, 2026
581ffb9
Wrap RowsVerticalSection in tics in NEWS
gmbecker Jul 29, 2026
0a8a364
fix lintr
shajoezhu Jul 31, 2026
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,18 +6,19 @@ default_language_version:
python: python3
repos:
- repo: https://github.com/lorenzwalthert/precommit
rev: v0.4.3.9021
rev: v0.4.3.9025
hooks:
- id: roxygenize
additional_dependencies:
- insightsengineering/formatters
- formatters
- magrittr
- methods
- checkmate
- htmltools
- lifecycle
- stats
- stringi
- roxygen2
- id: use-tidy-description
- id: spell-check
exclude: >
Expand Down
2 changes: 1 addition & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,6 @@ Config/Needs/verdepcheck: insightsengineering/formatters,
davidgohel/officer, Merck/r2rtf, rstudio/rmarkdown, therneau/survival,
r-lib/testthat, tidyverse/tibble, tidyverse/tidyr, r-lib/withr,
r-lib/xml2
Config/roxygen2/version: 8.0.0
Encoding: UTF-8
Language: en-US
LazyData: true
Expand Down Expand Up @@ -105,3 +104,4 @@ Collate:
'tt_from_df.R'
'validate_table_struct.R'
'zzz_constants.R'
Config/roxygen2/version: 8.0.0
1 change: 1 addition & 0 deletions NAMESPACE
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
# Generated by roxygen2: do not edit by hand

S3method(c,RowsVerticalSection)
S3method(print,CellValue)
S3method(print,RowsVerticalSection)
export("cell_footnotes<-")
Expand Down
22 changes: 20 additions & 2 deletions NEWS.md
Original file line number Diff line number Diff line change
@@ -1,8 +1,26 @@
## rtables 0.6.16.9001

### New Features
* Added `restrict_facets` function factory for use with `make_split_fun`
* Exportd previously internal `make_subset_expr` for use when constructing custom splitting behavior
* Added `restrict_facets` function factory for use with `make_split_fun` @gmbecker
* Exported previously internal `make_subset_expr` for use when constructing custom splitting behavior
* Added accessor methods for RowsVerticalSection objects: `row_cells`, `obj_format`, `obj_format<-`, `obj_na_str`, `obj_na_str<-`, `cell_values`
* Added `c` method for directly combining `RowsVerticalSection` objects
* Added vignette: Guided Tour (Advanced) @gmbecker
* Added vignette: Guided Tour (Advanced) - Custom Analysis And Summary Functions @gmbecker
* Added vignette: Guided Tour (Advanced) - Analysis Functions - Split Context @gmbecker
* Added vignette: Guided Tour (Advanced) - Analysis Functions - Combining Existing `afun`s @gmbecker
* Added vignette: Guided Tour (Advanced) - Analysis Functions - (stub) Useful Building Blocks @gmbecker
* Added vignette: Guided Tour (Advanced) - Custom Split Functions @gmbecker
* Added vignette: Guided Tour (Advanced) - Custom Split Functions - (stub) `make_split_fun` @gmbecker
* Added vignette: Guided Tour (Advanced) - Custom Split Functions - Behavioral Building Block @gmbecker
* Added vignette: Guided Tour (Advanced) - Custom Split Functions - (stub) Worked Examples @gmbecker
* Added vignette: Guided Tour (Advanced) - (stub) `TableTree` Objects @gmbecker
* Added vignette: Guided Tour (Advanced) - (stub) `TableTree` Objects - (stub) Accessing Table Values @gmbecker
* Added vignette: Guided Tour (Advanced) - (stub) `TableTree` Objects - (stub) Custom Scoring Functions For Sorting @gmbecker
* Added vignette: Guided Tour (Advanced) - (stub) `TableTree` Objects - (stub) Custom Pruning Functions @gmbecker

### Bug Fixes
* `obj_na_str<-` RowsVerticalSection method now correctly recycles length 1 values @gmbecker
Comment thread
gmbecker marked this conversation as resolved.
Outdated

## rtables 0.6.15

Expand Down
37 changes: 37 additions & 0 deletions R/00tabletrees.R
Original file line number Diff line number Diff line change
Expand Up @@ -2182,6 +2182,43 @@ print.RowsVerticalSection <- function(x, ...) {
invisible(x)
}

#' Combine RowsVerticalSection objects
Comment thread
gmbecker marked this conversation as resolved.
Outdated
#' Combine two or more RowsVerticalSection objects (as returned
#' by [in_rows()]) into a single object
#'
#' @param ... RowsVerticalSection objects
#' @returns A single RowsVerticalSection object containing all
#' row sections from the objects passed to `...`
#' @export
c.RowsVerticalSection <- function(...) {
Comment thread
gmbecker marked this conversation as resolved.
lst <- list(...)
if (!all(vapply(lst, function(x) inherits(x, "RowsVerticalSection"), TRUE))) {
stop("Cannot use c() to combine RowsVerticalSection objects with objects of other classes")
}

out <- NextMethod(generic = "c")
out <- RowsVerticalSection(
out,
names = comb_attr_w_dflt(lst, "row_names"),
labels = comb_attr_w_dflt(lst, "row_labels"),
indent_mods = comb_attr_w_dflt(lst, "indent_mods", 0L),
formats = comb_attr_w_dflt(lst, "row_formats", "xx"),
footnotes = comb_attr_w_dflt(lst, "row_footnotes"),
format_na_strs = comb_attr_w_dflt(lst, "row_na_strs", NA_character_)
)
out
}

comb_attr_w_dflt <- function(lst, attrname, dflt = NULL) {
unlist(
lapply(lst, function(x) {
attr(x, attrname, exact = TRUE) %||% rep(dflt, length(x))
}),
recursive = FALSE,
use.names = FALSE
)
}

#### Empty default objects to avoid repeated calls
## EmptyColInfo <- InstantiatedColumnInfo()
## EmptyElTable <- ElementaryTable()
Expand Down
2 changes: 1 addition & 1 deletion R/make_split_fun.R
Original file line number Diff line number Diff line change
Expand Up @@ -394,7 +394,7 @@ add_overall_facet <- function(name, label, extra = list()) {
#'
#' @param innervar (`character`)\cr the variable(s) to trim (remove unobserved levels) independently within each facet.
#'
#' @return A function suitable for use in the `pre` (list) argument of `make_split_fun`.
#' @return A function suitable for use in the `post` (list) argument of `make_split_fun`.
#'
#' @seealso [make_split_fun()]
#'
Expand Down
30 changes: 28 additions & 2 deletions R/tree_accessors.R
Original file line number Diff line number Diff line change
Expand Up @@ -999,6 +999,10 @@ setGeneric("row_cells", function(obj) standardGeneric("row_cells"))
#' @exportMethod row_cells
setMethod("row_cells", "TableRow", function(obj) obj@leaf_value)

#' @rdname row_accessors
#' @exportMethod row_cells
setMethod("row_cells", "RowsVerticalSection", function(obj) as(obj, "list", strict = TRUE))

#' @rdname row_accessors
setGeneric("row_cells<-", function(obj, value) standardGeneric("row_cells<-"))

Expand All @@ -1017,7 +1021,6 @@ setGeneric("row_values", function(obj) standardGeneric("row_values"))
#' @exportMethod row_values
setMethod("row_values", "TableRow", function(obj) rawvalues(obj@leaf_value))


#' @rdname row_accessors
#' @exportMethod row_values<-
setGeneric("row_values<-", function(obj, value) standardGeneric("row_values<-"))
Expand Down Expand Up @@ -1153,13 +1156,18 @@ setMethod("obj_format", "CellValue", function(obj) attr(obj, "format", exact = T
#' @export
setMethod("obj_format", "Split", function(obj) obj@split_format)

#' @rdname formatters_methods
#' @export
setMethod("obj_format", "RowsVerticalSection", function(obj) attr(obj, "row_formats", exact = TRUE))

#' @rdname formatters_methods
#' @export
setMethod("obj_format<-", "VTableNodeInfo", function(obj, value) {
obj@format <- value
obj
})


#' @rdname formatters_methods
#' @export
setMethod("obj_format<-", "Split", function(obj, value) {
Expand All @@ -1174,13 +1182,27 @@ setMethod("obj_format<-", "CellValue", function(obj, value) {
obj
})

#' @rdname formatters_methods
#' @export
setMethod("obj_format<-", "RowsVerticalSection", function(obj, value) {
attr(obj, "row_formats") <- value
obj
})

#' @rdname int_methods
#' @export
setMethod("obj_na_str<-", "CellValue", function(obj, value) {
attr(obj, "format_na_str") <- value
obj
})

#' @rdname int_methods
#' @export
setMethod("obj_na_str<-", "RowsVerticalSection", function(obj, value) {
attr(obj, "row_na_strs") <- value
obj
})

#' @rdname int_methods
#' @export
setMethod("obj_na_str<-", "VTableNodeInfo", function(obj, value) {
Expand All @@ -1199,6 +1221,10 @@ setMethod("obj_na_str<-", "Split", function(obj, value) {
#' @export
setMethod("obj_na_str", "VTableNodeInfo", function(obj) obj@na_str)

#' @rdname int_methods
#' @export
setMethod("obj_na_str", "RowsVerticalSection", function(obj) attr(obj, "row_na_strs", exact = TRUE))

#' @rdname formatters_methods
#' @export
setMethod("obj_na_str", "Split", function(obj) obj@split_na_str)
Expand Down Expand Up @@ -1678,7 +1704,7 @@ setMethod(
"must have length 1 or the number of rows"
)
}
attr(obj, "indent_mods") <- as.integer(value)
attr(obj, "indent_mods") <- rep(as.integer(value), length.out = length(obj))
obj

## obj@indent_mods <- value
Expand Down
8 changes: 8 additions & 0 deletions R/tt_pos_and_access.R
Original file line number Diff line number Diff line change
Expand Up @@ -1584,6 +1584,14 @@ setMethod(
}
)

#' @rdname int_methods
#' @keywords internal
#' @exportMethod cell_values
setMethod(
"cell_values", "RowsVerticalSection",
function(tt, rowpath, colpath = NULL, omit_labrows = TRUE) rawvalues(tt)
)

#' @rdname int_methods
#' @keywords internal
#' @exportMethod cell_values
Expand Down
14 changes: 14 additions & 0 deletions _pkgdown.yml
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,19 @@ articles:
- guided_intermediate_translating_shells
- guided_intermediate_afun_reqs
- guided_intermediate_split_reqs
- guided_advanced
- guided_advanced_afuns
- guided_advanced_afuns_spl_context
- guided_advanced_afuns_rowsverticalsection
- guided_advanced_afuns_building_blocks
- guided_advanced_split_funs
- guided_advanced_split_funs_make_split_fun
- guided_advanced_split_funs_new_bbbs
- guided_advanced_split_funs_worked_ex
- guided_advanced_tt
- guided_advanced_tt_access
- guided_advanced_tt_score_funs
- guided_advanced_tt_prune_funs

- title: Clinical Trials
navbar: Clinical Trials
Expand Down Expand Up @@ -323,3 +336,4 @@ reference:
- length,CellValue-method
- names,VTableNodeInfo-method
- insert_rrow
- c.RowsVerticalSection
2 changes: 2 additions & 0 deletions inst/WORDLIST
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ amongst
ARD
ard
ARDs
BBBs
biomarker
BMEASIFL
Bov
Expand Down Expand Up @@ -80,6 +81,7 @@ responder
Resync
reusability
roadmap
RowsVerticalSection
RStudio
rtables
Rua
Expand Down
22 changes: 22 additions & 0 deletions man/c.RowsVerticalSection.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 6 additions & 0 deletions man/formatters_methods.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 9 additions & 0 deletions man/int_methods.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 3 additions & 0 deletions man/row_accessors.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion man/trim_levels_in_facets.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Binary file added vignettes/analysis_basics_a.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Loading