check now finishes with some issues

[andrewelamb]

Hey @allaway:

I commented out this function call in the vignette:

library(survival)
goldStandard <- Surv(d.time, death) %>% as.character() %>% as.data.frame() %>% magrittr::set_colnames(c("gold"))
predictions <- age_new %>% as.data.frame() %>% magrittr::set_colnames(c("pred"))
prevPredictions <- age_previous %>% as.data.frame() %>% magrittr::set_colnames(c("pred"))

# TODO: fix this error: Error: `by` required, because the data sources have no common variables
# bootLadderBoot(predictions = predictions,
#                predictionColname = "pred", 
#                prevPredictions = prevPredictions,
#                goldStandard = goldStandard,
#                goldStandardColname = "gold",
#                scoreFun = c_statistic,
#                verbose = T)

To get check() to work. The error is not super helpful in this context(this could indicate some error handling could be added to this function).

[andrewelamb]

Oh yes check() is a function that goes through a package and does several things like run the tests, make sure the documentation is correct, runs the vignettes, etc...

[allaway]

IIRC the ability to run this function on Surv objects was never finished. This is related to #11