Skip to content

Commit 6ea0450

Browse files
shorten variable names
1 parent 4bdb0e2 commit 6ea0450

File tree

6 files changed

+63
-63
lines changed

6 files changed

+63
-63
lines changed

R/survival.R

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ h_lr_test_via_score <- function(score_fun, ...) {
8686
tau_l = tau_l,
8787
pval = pval,
8888
n = n,
89-
give_randomization_strata_warning = score_attrs$give_randomization_strata_warning
89+
give_rand_strat_warning = score_attrs$give_rand_strat_warning
9090
)
9191
}
9292

@@ -162,7 +162,7 @@ robin_surv_comparison <- function(
162162
if (!is.null(unadj_score_fun)) {
163163
assert_function(unadj_score_fun)
164164
assert_true(length(vars$covariates) > 0)
165-
args_to_drop <- c("model", "hr_se_plugin_adjusted", "check_randomization_strata_warning")
165+
args_to_drop <- c("model", "hr_se_plugin_adjusted", "check_rand_strat_warning")
166166
unadj_args <- args[!(names(args) %in% args_to_drop)]
167167
unadj_args$score_fun <- unadj_score_fun
168168
# Get theta_hat from the unadjusted score function.
@@ -194,7 +194,7 @@ robin_surv_comparison <- function(
194194
test_score = test_result$u_l,
195195
test_n = test_result$n,
196196
test_sigma_l2 = test_result$sigma_l2,
197-
give_randomization_strata_warning = test_result$give_randomization_strata_warning
197+
give_rand_strat_warning = test_result$give_rand_strat_warning
198198
)
199199
}
200200

@@ -218,7 +218,7 @@ robin_surv_no_strata_no_cov <- function(
218218
exp_level,
219219
control_level,
220220
contrast,
221-
check_randomization_strata_warning = FALSE
221+
check_rand_strat_warning = FALSE
222222
) {
223223
robin_surv_comparison(
224224
score_fun = h_lr_score_no_strata_no_cov,
@@ -231,7 +231,7 @@ robin_surv_no_strata_no_cov <- function(
231231
time = vars$time,
232232
status = vars$status,
233233
randomization_strata = vars$randomization_strata,
234-
check_randomization_strata_warning = check_randomization_strata_warning
234+
check_rand_strat_warning = check_rand_strat_warning
235235
)
236236
}
237237

@@ -243,7 +243,7 @@ robin_surv_strata <- function(
243243
exp_level,
244244
control_level,
245245
contrast,
246-
check_randomization_strata_warning = FALSE
246+
check_rand_strat_warning = FALSE
247247
) {
248248
robin_surv_comparison(
249249
score_fun = h_lr_score_strat,
@@ -257,7 +257,7 @@ robin_surv_strata <- function(
257257
status = vars$status,
258258
strata = vars$strata,
259259
randomization_strata = vars$randomization_strata,
260-
check_randomization_strata_warning = check_randomization_strata_warning
260+
check_rand_strat_warning = check_rand_strat_warning
261261
)
262262
}
263263

@@ -516,7 +516,7 @@ robin_surv <- function(
516516
# randomization strata in the analysis model has already been required in a comparison.
517517
# We want to avoid checking for the need of a warning, or giving a warning,
518518
# multiple times.
519-
give_randomization_strata_warning <- FALSE
519+
give_rand_strat_warning <- FALSE
520520

521521
estimates <- lapply(
522522
seq_len(n_comparisons),
@@ -529,19 +529,19 @@ robin_surv <- function(
529529
exp_level = exp_level,
530530
control_level = control_level,
531531
contrast = contrast,
532-
check_randomization_strata_warning = !give_randomization_strata_warning,
532+
check_rand_strat_warning = !give_rand_strat_warning,
533533
...
534534
)
535-
if (!give_randomization_strata_warning) {
535+
if (!give_rand_strat_warning) {
536536
# Only update if we have checked for it in this iteration, otherwise
537537
# we could overwrite a TRUE value with the default FALSE.
538-
give_randomization_strata_warning <<- result$give_randomization_strata_warning
538+
give_rand_strat_warning <<- result$give_rand_strat_warning
539539
}
540540
result
541541
}
542542
)
543543

544-
if (give_randomization_strata_warning) {
544+
if (give_rand_strat_warning) {
545545
missing_vars <- setdiff(input$randomization_strata, c(input$covariates, input$strata))
546546
cov_string <- if (length(missing_vars) > 1) {
547547
paste0("interaction(", toString(missing_vars), ")")

R/survival_score.R

Lines changed: 21 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ h_lr_score_no_strata_no_cov <- function(
5353
n = nrow(df),
5454
use_ties_factor = TRUE,
5555
calculate_variance = TRUE,
56-
check_randomization_strata_warning = FALSE
56+
check_rand_strat_warning = FALSE
5757
) {
5858
assert_numeric(theta, min.len = 1L, finite = TRUE)
5959
assert_data_frame(df)
@@ -69,10 +69,10 @@ h_lr_score_no_strata_no_cov <- function(
6969
assert_true(n >= nrow(df))
7070
assert_flag(use_ties_factor)
7171
assert_flag(calculate_variance)
72-
assert_flag(check_randomization_strata_warning)
72+
assert_flag(check_rand_strat_warning)
7373

7474
# Check whether to warn about insufficient inclusion of randomization strata.
75-
give_randomization_strata_warning <- (check_randomization_strata_warning &&
75+
give_rand_strat_warning <- (check_rand_strat_warning &&
7676
length(randomization_strata) > 0)
7777

7878
# Standardize data set format, subset to relevant variables.
@@ -141,7 +141,7 @@ h_lr_score_no_strata_no_cov <- function(
141141
sigma_l2 = sigma_l2,
142142
se_theta_l = se_theta_l,
143143
n = n,
144-
give_randomization_strata_warning = give_randomization_strata_warning
144+
give_rand_strat_warning = give_rand_strat_warning
145145
)
146146
}
147147

@@ -156,7 +156,7 @@ h_lr_score_strat <- function(
156156
randomization_strata = character(),
157157
use_ties_factor = TRUE,
158158
calculate_variance = TRUE,
159-
check_randomization_strata_warning = FALSE
159+
check_rand_strat_warning = FALSE
160160
) {
161161
assert_string(treatment)
162162
assert_string(time)
@@ -166,9 +166,9 @@ h_lr_score_strat <- function(
166166
lapply(df[strata], assert_factor)
167167
lapply(df[randomization_strata], assert_factor)
168168
assert_flag(calculate_variance)
169-
assert_flag(check_randomization_strata_warning)
169+
assert_flag(check_rand_strat_warning)
170170

171-
give_randomization_strata_warning <- if (check_randomization_strata_warning) {
171+
give_rand_strat_warning <- if (check_rand_strat_warning) {
172172
length(randomization_strata) > 0 &&
173173
!all(randomization_strata %in% strata) &&
174174
!h_are_factors_nested(interaction(df[strata]), interaction(df[randomization_strata]))
@@ -193,7 +193,7 @@ h_lr_score_strat <- function(
193193
randomization_strata = randomization_strata,
194194
use_ties_factor = use_ties_factor,
195195
calculate_variance = calculate_variance,
196-
check_randomization_strata_warning = FALSE
196+
check_rand_strat_warning = FALSE
197197
)
198198

199199
u_sl <- sum_vectors_in_list(strata_results)
@@ -204,7 +204,7 @@ h_lr_score_strat <- function(
204204
sigma_l2 = sigma_sl2,
205205
se_theta_l = se_theta_sl,
206206
n = n,
207-
give_randomization_strata_warning = give_randomization_strata_warning
207+
give_rand_strat_warning = give_rand_strat_warning
208208
)
209209
}
210210

@@ -221,7 +221,7 @@ h_lr_score_cov <- function(
221221
use_ties_factor = TRUE,
222222
hr_se_plugin_adjusted = TRUE,
223223
calculate_variance = TRUE,
224-
check_randomization_strata_warning = FALSE
224+
check_rand_strat_warning = FALSE
225225
) {
226226
assert_data_frame(df)
227227
assert_string(treatment)
@@ -234,7 +234,7 @@ h_lr_score_cov <- function(
234234
assert_factor(df[[treatment]], n.levels = 2L, any.missing = FALSE)
235235
assert_flag(hr_se_plugin_adjusted)
236236
assert_flag(calculate_variance)
237-
assert_flag(check_randomization_strata_warning)
237+
assert_flag(check_rand_strat_warning)
238238

239239
# Subset to complete records here.
240240
required_cols <- unique(c(treatment, time, status, covariates, randomization_strata))
@@ -256,7 +256,7 @@ h_lr_score_cov <- function(
256256
lm_results <- h_get_lm_results(lm_input)
257257
beta_est <- lm_results$beta_est
258258

259-
give_randomization_strata_warning <- if (check_randomization_strata_warning) {
259+
give_rand_strat_warning <- if (check_rand_strat_warning) {
260260
length(randomization_strata) > 0 &&
261261
!h_unbiased_means_across_strata(
262262
residuals_per_group = lm_results$residuals,
@@ -278,7 +278,7 @@ h_lr_score_cov <- function(
278278
randomization_strata = randomization_strata,
279279
use_ties_factor = use_ties_factor,
280280
calculate_variance = calculate_variance,
281-
check_randomization_strata_warning = FALSE
281+
check_rand_strat_warning = FALSE
282282
)
283283

284284
# We assume here that the observed proportion of treatment 1 in the data set corresponds to the preplanned
@@ -314,7 +314,7 @@ h_lr_score_cov <- function(
314314
randomization_strata = randomization_strata,
315315
use_ties_factor = use_ties_factor,
316316
calculate_variance = TRUE,
317-
check_randomization_strata_warning = FALSE
317+
check_rand_strat_warning = FALSE
318318
)
319319
attr(unadj_score_theta_hat, "sigma_l2")
320320
}
@@ -335,7 +335,7 @@ h_lr_score_cov <- function(
335335
se_theta_l = if (calculate_variance) se_theta_cl else NA_real_,
336336
sigma_l2 = if (calculate_variance) sigma_cl2 else NA_real_,
337337
n = n,
338-
give_randomization_strata_warning = give_randomization_strata_warning
338+
give_rand_strat_warning = give_rand_strat_warning
339339
)
340340
}
341341

@@ -353,7 +353,7 @@ h_lr_score_strat_cov <- function(
353353
use_ties_factor = TRUE,
354354
hr_se_plugin_adjusted = TRUE,
355355
calculate_variance = TRUE,
356-
check_randomization_strata_warning = FALSE
356+
check_rand_strat_warning = FALSE
357357
) {
358358
assert_data_frame(df)
359359
assert_string(treatment)
@@ -366,7 +366,7 @@ h_lr_score_strat_cov <- function(
366366
assert_subset(c(treatment, time, status, strata, covariates), names(df))
367367
assert_flag(hr_se_plugin_adjusted)
368368
assert_flag(calculate_variance)
369-
assert_flag(check_randomization_strata_warning)
369+
assert_flag(check_rand_strat_warning)
370370

371371
# Subset to complete records here.
372372
required_cols <- unique(c(treatment, time, status, covariates, strata, randomization_strata))
@@ -397,7 +397,7 @@ h_lr_score_strat_cov <- function(
397397
lm_results <- h_get_strat_lm_results(strat_lm_input)
398398
beta_est <- lm_results$beta_est
399399

400-
give_randomization_strata_warning <- if (check_randomization_strata_warning) {
400+
give_rand_strat_warning <- if (check_rand_strat_warning) {
401401
length(randomization_strata) > 0 &&
402402
!h_unbiased_means_across_strata(
403403
residuals_per_group = lm_results$residuals,
@@ -419,7 +419,7 @@ h_lr_score_strat_cov <- function(
419419
randomization_strata,
420420
use_ties_factor = use_ties_factor,
421421
calculate_variance = calculate_variance,
422-
check_randomization_strata_warning = FALSE
422+
check_rand_strat_warning = FALSE
423423
)
424424

425425
# We assume here that the observed proportion of treatment 1 in the data set
@@ -481,7 +481,7 @@ h_lr_score_strat_cov <- function(
481481
randomization_strata = randomization_strata,
482482
use_ties_factor = use_ties_factor,
483483
calculate_variance = TRUE,
484-
check_randomization_strata_warning = FALSE
484+
check_rand_strat_warning = FALSE
485485
)
486486
attr(strat_unadj_score_theta_hat, "sigma_l2")
487487
}
@@ -497,6 +497,6 @@ h_lr_score_strat_cov <- function(
497497
se_theta_l = if (calculate_variance) se_theta_csl else NA_real_,
498498
sigma_l2 = if (calculate_variance) sigma_csl2 else NA_real_,
499499
n = n,
500-
give_randomization_strata_warning = give_randomization_strata_warning
500+
give_rand_strat_warning = give_rand_strat_warning
501501
)
502502
}

man/survival_comparison_functions.Rd

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

man/survival_score_functions.Rd

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

tests/testthat/_snaps/survival.md

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -16,49 +16,49 @@
1616
# h_lr_test_via_score works as expected
1717

1818
list(u_l = 0.0895537761860842, sigma_l2 = 0.17706769289317, tau_l = 3.21352484896035,
19-
pval = 0.0013111645203555, n = 228L, give_randomization_strata_warning = FALSE)
19+
pval = 0.0013111645203555, n = 228L, give_rand_strat_warning = FALSE)
2020

2121
# robin_surv_comparison works as expected without covariate adjustment
2222

2323
list(estimate = 0.53342923961288, se = 0.167270504465807, hr_n = 228L,
2424
hr_sigma_l2 = 0.156756810948865, test_stat = 3.21352484896035,
2525
p_value = 0.0013111645203555, test_score = 0.0895537761860842,
26-
test_n = 228L, test_sigma_l2 = 0.17706769289317, give_randomization_strata_warning = FALSE)
26+
test_n = 228L, test_sigma_l2 = 0.17706769289317, give_rand_strat_warning = FALSE)
2727

2828
# robin_surv_comparison can skip the hazard ratio estimation
2929

3030
list(estimate = NA_real_, se = NA_real_, hr_n = NA_integer_,
3131
hr_sigma_l2 = NA_real_, test_stat = 3.21352484896035, p_value = 0.0013111645203555,
3232
test_score = 0.0895537761860842, test_n = 228L, test_sigma_l2 = 0.17706769289317,
33-
give_randomization_strata_warning = FALSE)
33+
give_rand_strat_warning = FALSE)
3434

3535
# robin_surv_no_strata_no_cov works as expected
3636

3737
list(estimate = -0.53342923961288, se = 0.167270504465807, hr_n = 228L,
3838
hr_sigma_l2 = 0.156756810948865, test_stat = -3.21352484896035,
3939
p_value = 0.0013111645203555, test_score = -0.0895537761860842,
40-
test_n = 228L, test_sigma_l2 = 0.17706769289317, give_randomization_strata_warning = FALSE)
40+
test_n = 228L, test_sigma_l2 = 0.17706769289317, give_rand_strat_warning = FALSE)
4141

4242
# robin_surv_strata works as expected
4343

4444
list(estimate = -0.554820697243091, se = 0.170628735365298, hr_n = 227L,
4545
hr_sigma_l2 = 0.151310755205329, test_stat = -3.2855836062253,
4646
p_value = 0.00101771334472415, test_score = -0.0896871248297144,
47-
test_n = 227L, test_sigma_l2 = 0.169145720705825, give_randomization_strata_warning = FALSE)
47+
test_n = 227L, test_sigma_l2 = 0.169145720705825, give_rand_strat_warning = FALSE)
4848

4949
# robin_surv_cov works as expected
5050

5151
list(estimate = -0.503821677843015, se = 0.1656799127954, hr_n = 228L,
5252
hr_sigma_l2 = 0.156990532256188, test_stat = -3.06610317830763,
5353
p_value = 0.0021686846372504, test_score = -0.0851266162073163,
54-
test_n = 228L, test_sigma_l2 = 0.175748674116587, give_randomization_strata_warning = FALSE)
54+
test_n = 228L, test_sigma_l2 = 0.175748674116587, give_rand_strat_warning = FALSE)
5555

5656
# robin_surv_strata_cov works as expected
5757

5858
list(estimate = -0.0136495961352817, se = 0.158066245456898,
5959
hr_n = 227L, hr_sigma_l2 = 0.172478508438683, test_stat = -0.0835488066000914,
6060
p_value = 0.933415170960403, test_score = -0.00229971887336489,
61-
test_n = 227L, test_sigma_l2 = 0.171986696328211, give_randomization_strata_warning = FALSE)
61+
test_n = 227L, test_sigma_l2 = 0.171986696328211, give_rand_strat_warning = FALSE)
6262

6363
# h_log_hr_coef_mat works as expected
6464

0 commit comments

Comments
 (0)