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