Re: [Rd] as.matrix.dist patch (performance)

2024-01-16 Thread Tim Taylor
Cheers Ivan Heather Turner has improved in it further in the R contributors slack. I’ll take another look and ensure something is added to Bugzilla, with attribution, in the next few days. Tim > On 16 Jan 2024, at 14:36, Ivan Krylov wrote: > > Dear Tim, > > В Thu, 10 Aug 2023 22:38:44 +010

Re: [Rd] as.matrix.dist patch (performance)

2024-01-16 Thread Ivan Krylov
Dear Tim, В Thu, 10 Aug 2023 22:38:44 +0100 Tim Taylor пишет: > Submitting here in the first instance but happy to move to Bugzilla > if more appropriate. It's a fine patch. The 1.7 times speed up from not transposing the return value shouldn't be sneezed at. I think it's time to move it to Bug

[Rd] as.matrix.dist patch (performance)

2023-08-10 Thread Tim Taylor
Please find attached a small patch to improve the performance of as.matrix.dist().  It's a tiny bit more involved than the current code but does bring a reasonable speed improvement for larger objects (remaining comparable for smaller ones). Example: set.seed(1) dat <- matrix(rnorm(2), n