cards icon indicating copy to clipboard operation
cards copied to clipboard

Architectural Decision Records (ADR)

Open ddsjoberg opened this issue 10 months ago • 2 comments

https://github.com/insightsengineering/chevron.adr

For now, each comment below will be one decision made.

ddsjoberg avatar Aug 07 '23 13:08 ddsjoberg

Rounding in {cards}

  • Date: 2023-10-31

Context and Problem Statement

Default rounding in R is different from SAS and from what everyone expects.

Considered Options

  • Export function to implement standard rounding
  • Rely on default R rounding, and continue to explain to people that the formatted values are "correct" in the context of base R's default rounding

Decision Outcome

After discussion with @shajoezhu, we decided we will export a function for human rounding, so we don't have to continue to explain how the rounding is performed and results from other languages will match.

ddsjoberg avatar Oct 31 '23 16:10 ddsjoberg

Difference between by and strata

  • Date: 2023-10-27

Context and Problem Statement

In some cases, tabulations and summaries need to be calculated by every combination of variables, including observed and unobserved combinations (e.g. tabulating subject characteristics by treatment arm: even if a characteristic is unobserved in a treatment arm), and in other cases, we need summaries by observed levels (e.g. tabulating preferred term stratified by SOC).

Considered Options

  • We already export functions for summarizing by all combinations (ard_categorical(by)) and we could add a strata argument to tabulate by observed levels.
  • Instead of adding an argument to the existing function, we could export another function that can do stratified summaries.

Decision Outcome

@bzkrouse and @ddsjoberg discussed and we will add a strata argument to supplement the existing by argument.

ddsjoberg avatar Oct 31 '23 16:10 ddsjoberg