Re: [Rd] bug with mapply() on an S4 object

2012-11-28 Thread Hervé Pagès
Hi Martin, Malcolm, Thanks Martin for applying the fix. On 11/28/2012 05:47 AM, Cook, Malcolm wrote: Yes, yes, excellent and great , I am tracking this development with great interest. Am I correct that the implications for BioConductor is the tearing out of the Xapply from generics and the

Re: [Rd] bug with mapply() on an S4 object

2012-11-28 Thread Cook, Malcolm
Yes, yes, excellent and great , I am tracking this development with great interest. Am I correct that the implications for BioConductor is the tearing out of the Xapply from generics and the expecations that List and descendents would now "just work" with {t,mc,mcl,...}apply? That would be a g

Re: [Rd] bug with mapply() on an S4 object

2012-11-28 Thread Martin Maechler
> Hervé Pagès > on Tue, 27 Nov 2012 17:03:05 -0800 writes: > Some formatting issues when copy/pasting the patch in the > body of the email so I've attached the diff file. Thank you, Hervé. I have committed (a slightly simplified version of) your patch to R-devel (to become 2

Re: [Rd] bug with mapply() on an S4 object

2012-11-27 Thread Hervé Pagès
Some formatting issues when copy/pasting the patch in the body of the email so I've attached the diff file. Cheers, H. On 11/27/2012 04:56 PM, Hervé Pagès wrote: Hi, Here is a patch for this (against current R-devel). The "caching" of the .Primitive for 'length' is taken from seq_along() C cod

Re: [Rd] bug with mapply() on an S4 object

2012-11-27 Thread Hervé Pagès
Hi, Here is a patch for this (against current R-devel). The "caching" of the .Primitive for 'length' is taken from seq_along() C code (in R-devel/src/main/seq.c). hpages@thinkpad:~/svn/R$ svn diff R-devel Index: R-devel/src/main/mapply.c ==

[Rd] bug with mapply() on an S4 object

2012-11-14 Thread Hervé Pagès
Hi, Starting with ordinary vectors, so we know what to expect: > mapply(function(x, y) {x * y}, 101:106, rep(1:3, 2)) [1] 101 204 309 104 210 318 > mapply(function(x, y) {x * y}, 101:106, 1:3) [1] 101 204 309 104 210 318 Now with an S4 object: setClass("A", representation(aa="intege