Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Architectural Decision Records (ADR) #11

Open
ddsjoberg opened this issue Aug 7, 2023 · 2 comments
Open

Architectural Decision Records (ADR) #11

ddsjoberg opened this issue Aug 7, 2023 · 2 comments

Comments

@ddsjoberg
Copy link
Collaborator

ddsjoberg commented Aug 7, 2023

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

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

@ddsjoberg
Copy link
Collaborator Author

ddsjoberg commented Oct 31, 2023

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
Copy link
Collaborator Author

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 ddsjoberg pinned this issue Oct 31, 2023
@ddsjoberg ddsjoberg changed the title Set up Architectural Decision Records (ADR) Architectural Decision Records (ADR) Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant