Skip to content
Snippets Groups Projects
Verified Commit 97f0e591 authored by Gärber, Florian's avatar Gärber, Florian
Browse files

fix: Remove param `t` from `MFI`, it is never used

parent 3e144bba
No related branches found
No related tags found
No related merge requests found
#' MFI Shortcut
#' Mutual Forest Impact (Corrected Mean Adjusted Agreement).
#'
#' This method corrects the mean adjusted agreement by a permutation approach
#' and generates the relation parameter mutual forest impact.
#' Subsequently p-values are determined and related variables are selected.
#'
#' @inheritParams RandomForestSurrogates
#' @inheritDotParams RandomForestSurrogates
#' @param variables Vector of variable names for **which related variables should be searched**.
#' @param candidates Vector of variable names that **are candidates to be related to the variables**.
#' @param t Used to calculate threshold. (Default: 5)
#'
#' @returns A [`MutualForestImpact`] list object.
#' * `REL`: The [`MeanAdjustedAgreement`] object.
......@@ -28,7 +31,6 @@ MFI <- function(
variables,
candidates,
num.threads = 1,
t = 5,
...
) {
rels <- lapply(list(norm = FALSE, perm = TRUE), function(permutate) {
......@@ -38,7 +40,9 @@ MFI <- function(
permutate = permutate,
...
),
related = FALSE, t = t, num.threads = num.threads
# Relations are not calculated, leaving `t` unused.
related = FALSE, t = NA_integer_,
num.threads = num.threads
)
})
......
......@@ -6,7 +6,7 @@
#' @param variables vector of variable names.
#' @param allvariables vector of all variable names (strings)
#' @param candidates vector of variable names (strings) that are candidates to be related to the variables (has to be contained in allvariables)
#' @param t variable to calculate threshold. Default is 3.
#' @param t variable to calculate threshold. Default is 3. Used if `select.var = TRUE`.
#' @param s.a average number of surrogate variables (ideally calculated by count.surrogates function).
#' @param select.var set False if only relations should be calculated and no related variables should be selected.
#' @param num.threads number of threads used for parallel execution. Default is number of CPUs available.
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment