Fix findings of AI reviewer

mayer79 · mayer79 · commit 1f373fdefcd8 · 2025-07-19T22:00:23.000+02:00
diff --git a/R/kernelshap.R b/R/kernelshap.R
@@ -27,9 +27,8 @@
 #'
 #' This is repeated multiple times until convergence, see CL21 for details.
 #'
-#' To avoid the evaluation of
-#' This inefficiency can be fixed by a hybrid strategy, combining exact calculations
-#' with sampling.
+#' To avoid the re-evaluation of identical coalition vectors, we have implemented
+#' a hybrid strategy, combining exact calculations with sampling.
 #'
 #' The hybrid algorithm has two steps:
 #' 1. Step 1 (exact part): There are \eqn{2p} different on-off vectors \eqn{z} with
diff --git a/man/kernelshap.Rd b/man/kernelshap.Rd
diff --git a/tests/testthat/test-basic.R b/tests/testthat/test-basic.R
@@ -284,7 +284,7 @@ test_that("kernelshap and permshap work for models with high-order interactions"
     hybrid_degree = 2,
     exact = FALSE,
     m = 1000,
-    max_iter = 100, ,
+    max_iter = 100,
     tol = 0.001,
     verbose = FALSE
   )

Original file line number	Diff line number	Diff line change
`@@ -284,7 +284,7 @@ test_that("kernelshap and permshap work for models with high-order interactions"`
`284`	`284`	`hybrid_degree = 2,`
`285`	`285`	`exact = FALSE,`
`286`	`286`	`m = 1000,`
`287`		`- max_iter = 100, ,`
	`287`	`+ max_iter = 100,`
`288`	`288`	`tol = 0.001,`
`289`	`289`	`verbose = FALSE`
`290`	`290`	`)`