<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span style="font-size:13px;font-family:arial,sans-serif">I think if there is a `j` expression that computes on an .SD that has</span><br style="font-size:13px;font-family:arial,sans-serif">

<span style="font-size:13px;font-family:arial,sans-serif">duplicated colnames, I think we just stop().</span></blockquote><div><br></div><div>I'm not entirely sure what you mean by this. The following *should* work imo:</div>

<div><br></div><div>    dt = data.table(x = 1:10, x = 10:1)</div><div><br></div><div>    dt[, lapply(.SD, sum)]</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Nov 8, 2013 at 2:56 PM, Steve Lianoglou <span dir="ltr"><<a href="mailto:lianoglou.steve@gene.com" target="_blank">lianoglou.steve@gene.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Right, agreed.<br>
<br>
On Fri, Nov 8, 2013 at 12:53 PM, Arunkumar Srinivasan<br>
<div class="HOEnZb"><div class="h5"><<a href="mailto:aragorn168b@gmail.com">aragorn168b@gmail.com</a>> wrote:<br>
> Sorry, forget the j-value. For `.SDcols`, even when we provide integers<br>
> (column numbers), internally, we compute the column name and subset to get<br>
> `.SD`. And this'll have to change.<br>
><br>
> Arun<br>
><br>
> On Friday, November 8, 2013 at 9:47 PM, Steve Lianoglou wrote:<br>
><br>
> On Fri, Nov 8, 2013 at 12:45 PM, Arunkumar Srinivasan<br>
> <<a href="mailto:aragorn168b@gmail.com">aragorn168b@gmail.com</a>> wrote:<br>
><br>
> Oh I can certainly agree with that. I guess we'll have to make some changes<br>
> to the code to use index based subsetting when .SDcols or j-value is number<br>
> then.<br>
><br>
><br>
> Not sure what you mean by j-value -- the examples you gave didn't<br>
> compute on the .SD, it just returned it.<br>
><br>
> I think if there is a `j` expression that computes on an .SD that has<br>
> duplicated colnames, I think we just stop().<br>
><br>
> Or did you mean something else?<br>
><br>
> -steve<br>
><br>
> On Friday, November 8, 2013 at 9:41 PM, Steve Lianoglou wrote:<br>
><br>
> My gut reaction is:<br>
><br>
> On Fri, Nov 8, 2013 at 12:37 PM, Arunkumar Srinivasan<br>
> <<a href="mailto:aragorn168b@gmail.com">aragorn168b@gmail.com</a>> wrote:<br>
><br>
> Sure, here's an example of what I was trying to explain:<br>
><br>
> Suppose:<br>
> DT <- data.table(x=1:5, y=1:5, x=6:10)<br>
><br>
> Then,<br>
><br>
> DT[, c(1,3), with=FALSE] # gives correct subset<br>
><br>
><br>
> This is "OK", we just do what the user asks, here, as they are being<br>
> very specific.<br>
><br>
> DT[, c("x", "x"), with=FALSE] # gives column 1 twice - wrong<br>
><br>
><br>
> stop() -- we don't try to disambiguate (even if it "seems" specific)<br>
><br>
> DT[, .SD, .SDcols=c("x", "x")] # gives column 1 twice - wrong result<br>
><br>
><br>
> stop()<br>
><br>
> Also stop() on DT[, ..., .SDcols="x"]<br>
><br>
> DT[, .SD, .SDcols=c(1,3)] # gives column 1 twice - wrong result - but<br>
><br>
><br>
> Do what the user asks for.<br>
><br>
> No?<br>
><br>
> -steve<br>
><br>
> --<br>
> Steve Lianoglou<br>
> Computational Biologist<br>
> Bioinformatics and Computational Biology<br>
> Genentech<br>
><br>
><br>
><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>
><br>
><br>
><br>
><br>
> --<br>
> Steve Lianoglou<br>
> Computational Biologist<br>
> Bioinformatics and Computational Biology<br>
> Genentech<br>
><br>
><br>
><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>
<br>
<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>