search

chjackson / flexsurv

The flexsurv R package for flexible parametric survival and multi-state modelling
http://chjackson.github.io/flexsurv/
53 stars 28 forks source link

partial argument matching #149

Closed sebffischer closed 1 year ago

sebffischer commented 1 year ago

Your package depends on partial argument matching. Instead of the correct name length.out in the function seq() you are using length

consider the code

old_opts = options(
  warnPartialMatchArgs = TRUE,
  warnPartialMatchAttr = TRUE,
  warnPartialMatchDollar = TRUE
)
seq(length = 10)
chjackson commented 1 year ago

I think I've removed all the partial matches in flexsurv now. Running flexsurv's tests with these options set also revealed use of partial matching in the survival package, which I filed an issue about. In the meantime, I've forced the calls to survival::survreg to be run with these options switched off. So flexsurv's tests should run cleanly with these options on now.