Merge branch 'master' into devel

nt-williams · Jun 26, 2024 · 5dfc790 · 5dfc790
2 parents 7a2f31a + 432551c
commit 5dfc790
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -38,7 +38,7 @@ Imports:
     checkmate (>= 2.1.0),
     SuperLearner,
     schoolmath
-URL: https://github.com/nt-williams/lmtp
+URL: http://beyondtheate.com, https://github.com/nt-williams/lmtp
 BugReports: https://github.com/nt-williams/lmtp/issues
 Suggests: 
     testthat (>= 2.1.0),

diff --git a/R/density_ratios.R b/R/density_ratios.R
@@ -1,5 +1,11 @@
 cf_r <- function(task, learners, mtp, control, pb) {
   out <- vector("list", length = length(task$folds))
+
+  if (length(learners) == 1 && learners == "SL.mean") {
+    warning("Using 'SL.mean' as the only learner of the density ratios will always result in a misspecified model! If your exposure is randomized, consider using `c('SL.glm', 'SL.glmnet')`.",
+            call. = FALSE)
+  }
+
   for (fold in seq_along(task$folds)) {
     out[[fold]] <- future::future({
       estimate_r(