Merge pull request #255 from StochasticTree/ci-check-hotfix

Fix python unit tests and R package code for CI checks

Author: andrewherren

R/posterior_transformation.R

@@ -419,9 +419,11 @@ compute_contrast_bart_model <- function(
       if (!is.matrix(rfx_basis_0) || !is.matrix(rfx_basis_1)) {
         stop("'rfx_basis_0' and 'rfx_basis_1' must be matrices")
       }
-      if ((nrow(rfx_basis_0) != nrow(X)) || (nrow(rfx_basis_1) != nrow(X))) {
+      if (
+        (nrow(rfx_basis_0) != nrow(X_0)) || (nrow(rfx_basis_1) != nrow(X_1))
+      ) {
         stop(
-          "'rfx_basis_0' and 'rfx_basis_1' must have the same number of rows as 'X'"
+          "'rfx_basis_0' and 'rfx_basis_1' must have the same number of rows as 'X_0' and 'X_1'"
         )
       }
       if (

test/R/testthat/test-api-combinations.R

@@ -372,7 +372,7 @@ test_that("Quick check of interactions between components of BART functionality"
   skip_on_cran()
   # Code from: https://github.com/r-lib/testthat/blob/main/R/skip.R
   skip_if(
-    isTRUE(as.logical(Sys.getenv("RUN_SLOW_TESTS", "false"))),
+    isFALSE(as.logical(Sys.getenv("RUN_SLOW_TESTS", "false"))),
     "skipping slow tests"
   )
 
@@ -554,7 +554,7 @@ test_that("Quick check of interactions between components of BCF functionality",
   skip_on_cran()
   # Code from: https://github.com/r-lib/testthat/blob/main/R/skip.R
   skip_if(
-    isTRUE(as.logical(Sys.getenv("RUN_SLOW_TESTS", "false"))),
+    isFALSE(as.logical(Sys.getenv("RUN_SLOW_TESTS", "false"))),
     "skipping slow tests"
   )