Skip to content

Commit

Permalink
Exported the aggregate functions for rm_main_result and rm_confusion_…
Browse files Browse the repository at this point in the history
…matrix to get it to pass the Windows checks. Hopefully this will not make these methods public.
  • Loading branch information
emeyers committed Mar 15, 2024
1 parent 205a987 commit 826e211
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 0 deletions.
4 changes: 4 additions & 0 deletions NAMESPACE
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# Generated by roxygen2: do not edit by hand

S3method(aggregate_CV_split_results,rm_confusion_matrix)
S3method(aggregate_CV_split_results,rm_main_results)
S3method(aggregate_resample_run_results,rm_confusion_matrix)
S3method(aggregate_resample_run_results,rm_main_results)
S3method(get_data,ds_basic)
S3method(get_data,ds_generalization)
S3method(get_parameters,cl_max_correlation)
Expand Down
2 changes: 2 additions & 0 deletions R/rm_confusion_matrix.R
Original file line number Diff line number Diff line change
Expand Up @@ -88,6 +88,7 @@ new_rm_confusion_matrix <- function(the_data = data.frame(),
# Not going to export this since it should never be directly called by users of the NDR.
#' @inherit aggregate_CV_split_results
#' @keywords internal
#' @export
aggregate_CV_split_results.rm_confusion_matrix <- function(rm_obj, prediction_results) {

# include a warning if the state is not initial
Expand Down Expand Up @@ -190,6 +191,7 @@ aggregate_CV_split_results.rm_confusion_matrix <- function(rm_obj, prediction_re
# Not going to export this since it should never be directly called by users of the NDR.
#' @inherit aggregate_resample_run_results
#' @keywords internal
#' @export
aggregate_resample_run_results.rm_confusion_matrix <- function(resample_run_results) {

confusion_matrix <- resample_run_results
Expand Down
2 changes: 2 additions & 0 deletions R/rm_main_results.R
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ new_rm_main_results <- function(the_data = data.frame(), state = "initial", opti
# The aggregate_CV_split_results method needed to fulfill the results metric interface.
#' @inherit aggregate_CV_split_results
#' @keywords internal
#' @export
aggregate_CV_split_results.rm_main_results <- function(rm_obj, prediction_results) {


Expand Down Expand Up @@ -247,6 +248,7 @@ if ((sum(grepl("decision", names(prediction_results))) == 0) & (include_norm_ran
# The aggregate_resample_run_results method needed to fulfill the results metric interface
#' @inherit aggregate_resample_run_results
#' @keywords internal
#' @export
aggregate_resample_run_results.rm_main_results <- function(resample_run_results) {


Expand Down

0 comments on commit 826e211

Please sign in to comment.