<div>Yeah, this sounds like a better way to do it. Thanks for doing all this!</div><div><br></div><div>How about "compare.cols" or "<a href="http://compare.by">compare.by</a>"?</div><div><br></div><div>

--Frank</div><div><br><div class="gmail_quote">On Mon, Aug 12, 2013 at 12:51 PM, Steve Lianoglou <span dir="ltr"><<a href="mailto:mailinglist.honeypot@gmail.com" target="_blank">mailinglist.honeypot@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi folks,<br>
<br>
I actually want to revisit the fix I made here.<br>
<br>
Instead of having `use.key` in the signature to unique.data.table (and<br>
duplicated.data.table) to be:<br>
<br>
function(x,<br>
             incomparables=FALSE,<br>
             tolerance=.Machine$double.eps ^ 0.5,<br>
             use.key=TRUE, ...)<br>
<br>
How about we switch out use.key for a parameter that specifies the<br>
column names to use in the uniqueness check, which defaults to key(x)<br>
to keep backwards compatibility.<br>
<br>
For argument's sake (like that?), lets call this parameter `columns`<br>
(by.columns? with.columns? whatever) so:<br>
<br>
function(x,<br>
             incomparables=FALSE,<br>
             tolerance=.Machine$double.eps ^ 0.5,<br>
             columns=key(x), ...)<br>
<br>
Then:<br>
<br>
(1) leaving it alone is the backward compatibile behavior;<br>
(2) Perhaps setting it to NULL will use all columns, and make it<br>
equivalent to unique.data.frame (also the same when x has no key); and<br>
(3) setting it to any other combo of columns uses those columns as the<br>
uniqueness key and filters the rows (only) out of x accordingly.<br>
<br>
What do you folks think? Personally I think this is better on all<br>
accounts then just specifying to use the key or not and the only<br>
question in my mind is the name of the argument -- happy to hear other<br>
world views, however, so don't be shy.<br>
<br>
Thanks,<br>
<div class="HOEnZb"><div class="h5">-steve<br>
<br>
--<br>
Steve Lianoglou<br>
Computational Biologist<br>
Bioinformatics and Computational Biology<br>
Genentech<br>
_______________________________________________<br>
datatable-help mailing list<br>
<a href="mailto:datatable-help@lists.r-forge.r-project.org">datatable-help@lists.r-forge.r-project.org</a><br>
<a href="https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/datatable-help" target="_blank">https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/datatable-help</a><br>
</div></div></blockquote></div><br></div>