Re: [Rd] legitimate use of :::

2013-08-25 Thread Gabriel Becker
Yihui, On Sun, Aug 25, 2013 at 7:53 PM, Yihui Xie wrote: > I know it is really bad, but the so-called good approach can be more > expensive than that > It is more expensive for you, yes. On the other hand, I'm pretty sure if everyone were running around circumventing whatever CRAN tests they w

Re: [Rd] legitimate use of :::

2013-08-25 Thread Yihui Xie
I know it is really bad, but the so-called good approach can be more expensive than that, primarily because a package with a NOTE in R CMD check is likely to be rejected by CRAN, or authors have to justify the NOTE in the email. For this particular case, I can imagine at least one case which can b

Re: [Rd] legitimate use of :::

2013-08-25 Thread Duncan Murdoch
On 13-08-24 12:46 PM, Kasper Daniel Hansen wrote: On Thu, Aug 22, 2013 at 8:27 PM, Peter Meilstrup Using ::: on a package you don't control can be more dangerous. For a package author to choose to export a function to the public interface represents at least some assurance that that interface w

Re: [Rd] legitimate use of :::

2013-08-25 Thread Duncan Murdoch
On 13-08-22 11:54 PM, Yihui Xie wrote: Maybe it is not a good idea for R CMD check to check ::: at all, and a warning in R-exts and ?':::' may well be enough. On the other hand, it is just so easy to get around :::, because everybody can see its source code: It's a really bad idea to write tric

Re: [Rd] Correct NAMESPACE approach when writing an S3 method for a generic in another package

2013-08-25 Thread Gabriel Becker
The combination of importFrom(, ) S3method(, , ) works fine for me in a toy example. I don't have OpenGL on my laptop to test your actual case but I see no reason it wouldn't work. ~G On Sun, Aug 25, 2013 at 7:29 AM, Hadley Wickham wrote: > What I do, which is probably wrong, but at least it

Re: [Rd] Correct NAMESPACE approach when writing an S3 method for a generic in another package

2013-08-25 Thread Hadley Wickham
What I do, which is probably wrong, but at least it works, is export the s3 method as a function. i.e. instead of S3method(genericfunction, myclass) I do export(genericfunction.myclass) Hadley On Fri, Aug 23, 2013 at 11:01 PM, Gavin Simpson wrote: > Dear List, > > In one of my packages I have

Re: [Rd] cyclic namespace dependency detected when loading ...

2013-08-25 Thread Uwe Ligges
On 21.08.2013 16:34, Jannis wrote: Hi R users, I am developing two packages. Each package uses some functions from the other package. Now when I define these dependencies in the NAMESPACE file (via importFrom(,function1,)), i get this error (when building one package): cyclic namespa