search

nlmixr2 / rxode2

rxode2
https://nlmixr2.github.io/rxode2/
GNU General Public License v3.0
30 stars 7 forks source link

Minor suggestion: Deprecate either `rxInits()` or `rxInit()` #294

Open billdenney opened 2 years ago

billdenney commented 2 years ago

The rxInits() and rxInit() functions do the same thing. I'm guessing that they are uncommonly used (based on my bias that I'd not heard of them). (I only found them by looking for anther function within rxode2.)

As a very small code simplification, could one of them be removed?

mattfidler commented 2 years ago

Probably remove rxInit(), but it may have far reaching implications. I would suggest to wait until nlmixr2 is completely updated on CRAN.