Re: [Rd] R 2.5.0 devel try issue in conjuntion with S4 method dispatch

2007-03-20 Thread ml-it-r-devel
Peter Dalgaard wrote: > Matthias Burger wrote: >> Hi Seth, >> > [...] >> > I have just committed my variation of Seth's patch, so please check the > current r-devel too. For the record: With R 2.5.0 devel (2007-03-18 r40854) the issue I was concerned about has been resolved. Thanks t

Re: [Rd] R 2.5.0 devel try issue in conjuntion with S4 method dispatch

2007-03-17 Thread Seth Falcon
Peter Dalgaard <[EMAIL PROTECTED]> writes: > I have just committed my variation of Seth's patch, so please check the > current r-devel too. Thanks, Peter. And I agree that your variation is cleaner. -- Seth Falcon | Computational Biology | Fred Hutchinson Cancer Research Center http://biocondu

Re: [Rd] R 2.5.0 devel try issue in conjuntion with S4 method dispatch

2007-03-17 Thread Peter Dalgaard
Matthias Burger wrote: > Hi Seth, > > I applied your patch and the issues seems to be resolved. Now I just wait > to see if all test case failures related to this disappear. > > Thank you for your kind help! > > Matthias > I have just committed my variation of Seth's patch, so please

Re: [Rd] R 2.5.0 devel try issue in conjuntion with S4 method dispatch

2007-03-17 Thread Matthias Burger
Hi Seth, >It isn't S4 specific. The issue seems more about anonymous >calls/functions. > > ll = list() > ll[[1]] = function(x) stop("died") > > v = try(do.call(ll[[1]], list(1)), silent=TRUE) > Error in as.list(call)[[1]] == "doTryCatch" : > comparison (1) is possible only for atom

Re: [Rd] R 2.5.0 devel try issue in conjuntion with S4 method dispatch

2007-03-16 Thread Seth Falcon
"Matthias Burger" <[EMAIL PROTECTED]> writes: > Well, I understand I should have spent more time trimming my example > to the minimal required code. Again this piece was copied & reduced. No need to have done anything differently. I commented on the code in hopes that it might be helpful for othe

Re: [Rd] R 2.5.0 devel try issue in conjuntion with S4 method dispatch

2007-03-16 Thread Seth Falcon
This is off-topic, but since the discussion moved towards coding style... Here are some comments on S4 style. [EMAIL PROTECTED] writes: > ## using S4 classes and methods > setClass("fooBase", > representation("VIRTUAL", > width = "numeric", >

Re: [Rd] R 2.5.0 devel try issue in conjuntion with S4 method dispatch

2007-03-16 Thread Martin Maechler
> "PD" == Peter Dalgaard <[EMAIL PROTECTED]> > on Fri, 16 Mar 2007 08:09:00 +0100 writes: PD> Seth Falcon wrote: >> [EMAIL PROTECTED] writes: >> >>> Investigating this new implementation I come across an issue in >>> conjuntion with using S4 classes and methods. t

Re: [Rd] R 2.5.0 devel try issue in conjuntion with S4 method dispatch

2007-03-15 Thread Peter Dalgaard
Seth Falcon wrote: > [EMAIL PROTECTED] writes: > >> Investigating this new implementation I come across an issue in >> conjuntion with using S4 classes and methods. try(expr) does not return an >> object with attribute 'try-error' in case of method dispatch failure >> in the wrapped expression w

Re: [Rd] R 2.5.0 devel try issue in conjuntion with S4 method dispatch

2007-03-15 Thread Seth Falcon
[EMAIL PROTECTED] writes: > Investigating this new implementation I come across an issue in > conjuntion with using S4 classes and methods. try(expr) does not return an > object with attribute 'try-error' in case of method dispatch failure > in the wrapped expression which to me seems not > quite c

[Rd] R 2.5.0 devel try issue in conjuntion with S4 method dispatch

2007-03-15 Thread ml-it-r-devel
Hi, after updating R 2.5.0 devel yesterday we today observed many new unexpected failures in our daily package build and test system runs, which can be traced to recent changes in the implementation in try() (as noted in NEWS). Investigating this new implementation I come across an issue in con