search

KarlaMonterrubioG / CompRisksVignettes

Supplementary material for the paper: A review on competing risks methods for survival analysis
GNU General Public License v3.0
7 stars 2 forks source link

Update to timereg has broken CIF report #9

Closed nathansam closed 1 year ago

nathansam commented 1 year ago

The newest CRAN version of timereg has broken the CIF report. For some reason, cens.model = cox without explicitly specifying the weights is no longer supported.

https://github.com/scheike/timereg/blob/ca44719f4650a84ea981fd6c964897294ae5c31f/R/comprisk.r#L394-L418

This affects the following code which will now error.

https://github.com/KarlaMonterrubioG/Competing_risks/blob/7178c981e6673dafc45105ecaaff7c4905d5a4e7/Source/CIF_specification.Rmd#L384-L402

The options seem to be:

  1. Set cens.model = "KM" or cens.model = "stratKM" instead of "cox".
  2. Use Docker to ensure an older version is used (presumably this feature was removed for a reason though).
  3. Remove the entire timereg section.
KarlaMonterrubioG commented 1 year ago

I have pushed a change just setting cens.model to the default value