Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upAllow cvrisk( , grid = 0:mstop) #66
Comments
|
cvrisk for CoxPH is currently only working for grid = 1:mstop. Atm, it changes the grid if 0 is included and issues a warning. |
As we are now (see #64) able to fit models with
mstop = 0we should be also able topredictandfittedfor an offset model. Currently, a scalar is returned bu actually a constant vector of offsets might be more suitable. However, we then need to make sure that thesubsetfunction inmboost_fitworks as desired.The first point might introduce a lot of changes as we cannot assess a vectors zeroth element (hence all risks are shifted by one index). Consequently using the extractor function
risk.mboost()might avoid to many changes.