Skip to content

Commit 358ef54

Browse files
committed
refactor: add p-value and correlation coefficient thresholds to AF time correlation rule
1 parent a0b3c90 commit 358ef54

2 files changed

Lines changed: 6 additions & 1 deletion

File tree

workflow/rules/vaf.smk

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -286,6 +286,8 @@ rule af_time_correlation_data:
286286
params:
287287
cor_method=config["COR"]["METHOD"],
288288
cor_exact=config["COR"]["EXACT"],
289+
max_p_adj_threshold = 0.05,
290+
min_abs_cor_threshold = 0.0,
289291
input:
290292
variants=OUTDIR / f"{OUTPUT_NAME}.variants.all_sites.tsv",
291293
metadata=config["METADATA"],

workflow/scripts/report/af_time_correlation_data.R

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,10 @@ write_csv(
111111

112112
log_info("Selecting variants whose allele frequency is significantly correlated with time")
113113
significant.variants <- correlations %>%
114-
filter(p.value.adj < 0.05) %>%
114+
filter(
115+
p.value.adj <= snakemake@params$max_p_adj_threshold,
116+
abs(coefficient) >= snakemake@params$min_abs_cor_threshold
117+
) %>%
115118
pull(variant) %>%
116119
unique()
117120

0 commit comments

Comments
 (0)