Displaying 3 results from an estimated 3 matches for "r65926".
Did you mean:
r65296
2017 Nov 20
2
Small performance bug in [.Date
Hi all,
I think there's an unnecessary line in [.Date which has a considerable
impact on performance when subsetting large dates:
x <- Sys.Date() + 1:1e6
microbenchmark::microbenchmark(x[1])
#> Unit: microseconds
#> expr min lq mean median uq max neval
#> x[1] 920.651 1039.346 3624.833 2294.404 3786.881 41176.38 100
`[.Date` <- function(x, ...,
2017 Nov 21
0
Small performance bug in [.Date
...nd I'm pretty sure it's unnecessary as NextMethod() does not
> consult the class of x, but instead uses .Class.
Yes, at least so it looks in src/main/objects.c
Also, we had a very similar change a while ago :
------------------------------------------------------------------------
r65926 | luke | 2014-06-12 15:54:38 +0200 (Thu, 12. Jun 2014) | 2 Zeilen
Ge?nderte Pfade:
M src/library/base/R/datetime.R
Commented out class(x) <- NULL in [.POSIXct and [[.POSICct.
------------------------------------------------------------------------
and we never seemed to have followed up in...
2014 Jun 17
0
PATCH: Avoiding extra copies (NAMED bumped) with source(..., print.eval=FALSE) ...and with print.eval=TRUE?
...n most cases also with
print.eval=TRUE, e.g.
> withVisible(x[1] <- 0L)
$value
[1] 0
$visible
[1] FALSE
In other words, whenever withVisible() returns visible=FALSE, the
return values is not used by source().
Comments?
/Henrik
> sessionInfo()
R Under development (unstable) (2014-06-12 r65926)
Platform: x86_64-w64-mingw32/x64 (64-bit)
locale:
[1] LC_COLLATE=English_United States.1252
[2] LC_CTYPE=English_United States.1252
[3] LC_MONETARY=English_United States.1252
[4] LC_NUMERIC=C
[5] LC_TIME=English_United States.1252
attached base packages:
[1] stats graphics grDevices utils...