insightsengineering / cards

CDISC Analysis Results Data
https://insightsengineering.github.io/cards/
24 stars 0 forks source link

Architectural Decision Records (ADR) #11

Open ddsjoberg opened 11 months ago

ddsjoberg commented 11 months ago

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

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

ddsjoberg commented 8 months ago

Rounding in {cards}

Context and Problem Statement

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

Considered Options

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 commented 8 months ago

Difference between by and strata

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

Decision Outcome

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