outcome_type is meaningless to Lrnr_glm_fast learner

[osofr]

All attempts give an error. Perhaps we should add this to test set once we figured out what is causing this. Seems like the same issue is true for Lrnr_glm, not sure if all learners are affected though.

  data(cpp_imputed)
  cpp_haz_01range <- cpp_imputed
  cpp_haz_01range[["haz_01range"]] <- rep_len(c(0.1, 0.9), nrow(cpp_imputed))
  task_01range <- sl3_Task$new(cpp_haz_01range, covariates = covars, outcome = "haz_01range")
  fglm_learner <- Lrnr_glm_fast$new(outcome_type = "continuous")
  fglm_learner$train(task_01range)

Failed on Lrnr_glm_fast_TRUE_Cholesky_continuous
Error in outcome_type$glm_family : 
  $ operator is invalid for atomic vectors

  fglm_learner <- Lrnr_glm_fast$new(outcome_type = "binomial")
  fglm_learner$train(task_01range)

Failed on Lrnr_glm_fast_TRUE_Cholesky_binomial
Error in outcome_type$glm_family : 
  $ operator is invalid for atomic vectors

  fglm_learner <- Lrnr_glm_fast$new(outcome_type = "quasibinomial")
  fglm_learner$train(task_01range)

Failed on Lrnr_glm_fast_TRUE_Cholesky_quasibinomial
Error in outcome_type$glm_family : 
  $ operator is invalid for atomic vectors

[osofr]

Per @jeremy

this line: https://github.com/jeremyrcoyle/sl3/blob/master/R/sl3_Task.R#L63
needs to be added to this function: https://github.com/jeremyrcoyle/sl3/blob/master/R/Lrnr_base.R#L45
for the outcome_type as character to be converted to the outcome_type object