Update modules' code for reproducibility #1399
Draft
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request
Related to https://github.com/insightsengineering/coredev-tasks/issues/646 and insightsengineering/teal.gallery#212
{tern}
is a dependency but now that the qenv doesn't inherit the environment from theparent.env
it must be loaded or functions prefixed to make code fully reproducible in a fresh session (without previous loading anything.append_varlabels
summarize_num_patients
count_occurrences
count_occurrences_by_grade
score_occurrences_cont_cols
summarize_num_patients
summarize_occurrences_by_grade
extract_survival_subgroups
combine_levels
tabulate_survival_subgroups
g_forest
extract_rsp_subgroups
tabulate_rsp_subgroups
g_km
control_surv_timepoint
: Thre is one call withindo.call("control_surv_timepoint"
that I didn't prefix it because it would fail, we might need to load the package for that module.estimate_multinomial_response
estimate_odds_ratio
estimate_proportion_diff
test_proportion_diff
summarize_coxreg
fit_logistic
default_na_str
control_coxreg
!control_incidence_rate
estimate_incidence_rate
estimate_proportions
summarize_ancova
{rtables}
isn't loaded by default: prefixing for:sort_at_path
split_fun
drop_split_levels
add_overall_level
trim_levels_to_map
keep_split_levels
{formatters}
is not available on a fresh session for:obj_label
tm_t_ancova
andtm_t_mmrm
depended on the option set on.onLoad
for TMC. Instead of removing the call for the default viadefault_na_str
the modules set the default viatern::set_default_na_str
tm_t_mmrm
,tm_g_barchart_simple
,tm_t_g_forest_tte
andtm_t_coxreg
now use the label instead in the modal on "Show R Code" to be consistent with other modules.