(tm_a_mmrm) Name of the column could be improved

[cicdguy]

all obs could be All observations unless this is a regulatory thing...

Provenance:

Creator: kpagacz

[cicdguy]

THe all obs name does not appear to be assigned in the TMC module. It may come from build_table of rtables.

Provenance:

Creator: junlue

[cicdguy]

THe all obs name does not appear to be assigned in the TMC module. It may come from build_table of rtables.

@junlue yeah you are right. It is a by default name coming from rtables when there is no split_cols_by

Provenance:

Creator: imazubi

[cicdguy]

Here is workaround for changing "all obs": insightsengineering/rtables#110

Provenance:

Creator: anajens

[cicdguy]

So it probably should be implemented somewhere in tern, right?

Provenance:

Creator: kpagacz

[yangx61]

The name of the column should really be 'All Patients'. We can simply update MMRMT01 1.2 following the workaround provided above.

df_no_arm <- tidy(mmrm_results_no_arm)
attr(df_no_arm$AVISIT, "label") <- "Visit"

# Define the split function
split_fun <- drop_split_levels

basic_table() %>%
  add_overall_col("All Patients") %>% 
  add_colcounts() %>%
  split_rows_by("AVISIT", split_fun = split_fun, label_pos = "topleft", split_label = obj_label(df_no_arm$AVISIT)) %>%
  summarize_lsmeans(arms = FALSE) %>%
  append_topleft("  Statistics") %>%
  build_table(df_no_arm, alt_counts_df = adsl_sub)

[danielinteractive]

Yeah nice small improvement, low priority but also low complexity, can update both TLGC entry as well as template function in tm_a_mmrm()