From rpeng at jhsph.edu  Mon Mar  9 14:18:55 2009
From: rpeng at jhsph.edu (Roger D. Peng)
Date: Mon, 9 Mar 2009 09:18:55 -0400
Subject: [Rd] patch for 'merge' docs
In-Reply-To: <49B50FC5.7080206@biostat.ku.dk>
References: <66f3bd910903081820g75a3260ew1ce1c5864b17c711@mail.gmail.com>
	<93D7EDD5-4C4F-422A-9324-DF18E37FE269@stat.berkeley.edu>
	<66f3bd910903090517j334ec015u99ef366cc56f668c@mail.gmail.com>
	<49B50FC5.7080206@biostat.ku.dk>
Message-ID: <66f3bd910903090618k1c22d5c9xcad4a0d52133fe87@mail.gmail.com>

My patch was not particularly great. I think the Peter's alternative
makes (more) sense.

-roger

On Mon, Mar 9, 2009 at 8:47 AM, Peter Dalgaard <P.Dalgaard at biostat.ku.dk> wrote:
> Roger D. Peng wrote:
>> Hmm, I see what you mean, and I'd be willing to accept that logic if I
>> could find a single other instance in the R documentation where that
>> shorthand was used. But I suppose this might be the only instance
>> where such a shorthand is necessary.
>
> Could you repeat what the patch was (presumably your mailer claimed that
> it was non-text)?
>
> The text could probably improved if it confuses the reader. Most of what
> it says is actually implied by the argument defaults (all.x=all,
> all.y=all), and it is not perfectly logical anyway (all sets the
> _default_ for all.x and all.y, but you can still set e.g. all=T, all.y=F).
>
> I think we could simplify the text to something like
>
> ? ? all: logical; Provides a convenient way to set both 'all.x' and
> ? ? ? ? ? ? 'all.y' (defined below).
>
>> -roger
>>
>> On Sun, Mar 8, 2009 at 9:30 PM, Kasper Daniel Hansen
>> <khansen at stat.berkeley.edu> wrote:
>>> Roger
>>>
>>> (I think) L is shorthand for some logical value, ie. TRUE or FALSE. That has
>>> always been pretty clear to me. Your patch was stripped.
>>>
>>> Kasper
>>>
>>> On Mar 8, 2009, at 18:20 , Roger D. Peng wrote:
>>>
>>>> I've never quite understood the documentation for the 'all' argument
>>>> to 'merge'. I'm pretty sure using 'all = L' doesn't work but I'm open
>>>> to correction here. In any event, I've attached a patch.
>>>>
>>>> -roger
>>>>
>>>> --
>>>> Roger D. Peng ?| ?http://www.biostat.jhsph.edu/~rpeng/
>>>> ______________________________________________
>>>> R-devel at r-project.org mailing list
>>>> https://stat.ethz.ch/mailman/listinfo/r-devel
>>> ______________________________________________
>>> R-devel at r-project.org mailing list
>>> https://stat.ethz.ch/mailman/listinfo/r-devel
>>>
>>
>>
>>
>
>
> --
> ? O__ ?---- Peter Dalgaard ? ? ? ? ? ? ?ster Farimagsgade 5, Entr.B
> ?c/ /'_ --- Dept. of Biostatistics ? ? PO Box 2099, 1014 Cph. K
> ?(*) \(*) -- University of Copenhagen ? Denmark ? ? ?Ph: ?(+45) 35327918
> ~~~~~~~~~~ - (p.dalgaard at biostat.ku.dk) ? ? ? ? ? ? ?FAX: (+45) 35327907
>
> ______________________________________________
> R-devel at r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/r-devel
>



-- 
Roger D. Peng  |  http://www.biostat.jhsph.edu/~rpeng/



[Rd] patch for 'merge' docs
Roger D. Peng