opentims icon indicating copy to clipboard operation
opentims copied to clipboard

add the na.rm argument required by the range generic. Closes #25

Open sneumann opened this issue 11 months ago • 3 comments

See getGeneric("range") for required method definition. Yours, Steffen

sneumann avatar Mar 18 '24 16:03 sneumann

Looks good to me, thanks. Matteo? You're the R guy here, mind having a look?

michalsta avatar Mar 19 '24 16:03 michalsta

Hi @MatteoLacki , one more background information is the entry in the R changelog mentioning the change: https://cran.r-project.org/bin/windows/base/old/4.3.3/NEWS.R-4.3.3.html I accidentally committed my wholesale change from name = function() {...} to name <- function() {...} which was an early attempt to tackle the issue (you never know in R ...). I didn't revert it because it is the style in the R manuals, e.g. https://adv-r.hadley.nz/functions.html, but I can do that if you want me to. Yours, Steffen

sneumann avatar Mar 20 '24 09:03 sneumann

Hi, This PR added na.rm in the OpenTIMS and fixed the installation error mentioned in #25 . It should be safe to merge for other users to use this package.

yufree avatar Jul 08 '24 12:07 yufree