Skip to content

Potential bug in cv_sl (hardcoded [[1]] instead of [[i]]) #437

Description

@ctesta01

Dear maintainers of {sl3},

I wanted to suggest that I may have found a bug in the cv_sl function.

On line 85 there appears to be a hardcoded [[1]] where I would expect that it should be [[i]].

The code references cv_sl_fit$fit_object$fold_fits[[i]] throughout in several places above, but on line 85 uses cv_sl_fit$fit_object$fold_fits[[1]].

sl3/R/cv_sl.R

Line 85 in 0e8f236

sl_pred_i <- cv_sl_fit$fit_object$fold_fits[[1]]$predict(valid_task_i)

I spoke about this with @nhejazi earlier today and he encouraged me to file an issue to document this.

Best,
Christian

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions