<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><p>I’ve gone ahead and fixed <span class="externalcitation">[#696]</span>(https://github.com/Rdatatable/data.table/issues/696) to be consistent with base, even though I think this is not necessary in almost all cases. Either one could do:</p>

<pre><code>DT[order(DT[["a"]])]

</code></pre>

<p>Or simply use <code>copy</code> along with <code>setorderv</code>:</p>

<pre><code>setorderv(copy(DT), cols="a", order=1L, na.last=FALSE)

</code></pre>

<p>I find the latter much more cleaner, and can be used if one wants to reorder by reference as well, by just removing <code>copy</code>.</p>

<p><style>body{font-family:Helvetica,Arial;font-size:13px}</style><style>body {

        font-family: "Helvetica Neue", Helvetica, Arial, sans-serif;

        padding:1em;

        margin:auto;

        background:#fefefe;

}

h1, h2, h3, h4, h5, h6 {

        font-weight: bold;

}

h1 {

        color: #000000;

        font-size: 28pt;

}

h2 {

        border-bottom: 1px solid #CCCCCC;

        color: #000000;

        font-size: 24px;

}

h3 {

        font-size: 18px;

}

h4 {

        font-size: 16px;

}

h5 {

        font-size: 14px;

}

h6 {

        color: #777777;

        background-color: inherit;

        font-size: 14px;

}

hr {

        height: 0.2em;

        border: 0;

        color: #CCCCCC;

        background-color: #CCCCCC;

}

p, blockquote, ul, ol, dl, li, table, pre {

        margin: 15px 0;

}

a, a:visited {

        color: #4183C4;

        background-color: inherit;

        text-decoration: none;

}

#message {

        border-radius: 6px;

        border: 1px solid #ccc;

        display:block;

        width:100%;

        height:60px;

        margin:6px 0px;

}

button, #ws {

        font-size: 12 pt;

        padding: 4px 6px;

        border-radius: 5px;

        border: 1px solid #bbb;

        background-color: #eee;

}

code, pre, #ws, #message {

        font-family: Monaco;

        font-size: 10pt;

        border-radius: 3px;

        background-color: #F8F8F8;

        color: inherit;

}

code {

        border: 1px solid #EAEAEA;

        margin: 0 2px;

        padding: 0 5px;

}

pre {

        border: 1px solid #CCCCCC;

        overflow: auto;

        padding: 4px 8px;

}

pre > code {

        border: 0;

        margin: 0;

        padding: 0;

}

#ws { background-color: #f8f8f8; }

table {

border-collapse: collapse;  

font-family: Helvetica, arial, freesans, clean, sans-serif;  

color: rgb(51, 51, 51);  

font-size: 15px; line-height: 25px;

padding: 0; }

table tr {

border-top: 1px solid #cccccc;

background-color: white;

margin: 0;

padding: 0; }

table tr:nth-child(2n) {

background-color: #f8f8f8; }

table tr th {

font-weight: bold;

border: 1px solid #cccccc;

margin: 0;

padding: 6px 13px; }

table tr td {

border: 1px solid #cccccc;

margin: 0;

padding: 6px 13px; }

table tr th :first-child, table tr td :first-child {

margin-top: 0; }

table tr th :last-child, table tr td :last-child {

margin-bottom: 0; }

.send { color:#77bb77; }

.server { color:#7799bb; }

.error { color:#AA0000; }</style></p><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div> <div id="bloop_sign_1403567401415302912" class="bloop_sign"><div style="font-family:helvetica,arial;font-size:13px">Arun</div></div> <div style="color:black"><br>From: <span style="color:black">Michael Smith</span> <a href="mailto:my.r.help@gmail.com">my.r.help@gmail.com</a><br>Reply: <span style="color:black">Michael Smith</span> <a href="mailto:my.r.help@gmail.com">my.r.help@gmail.com</a><br>Date: <span style="color:black">June 21, 2014 at 5:19:04 AM</span><br>To: <span style="color:black">Arunkumar Srinivasan</span> <a href="mailto:aragorn168b@gmail.com">aragorn168b@gmail.com</a><br>Cc: <span style="color:black">datatable-help@lists.r-forge.r-project.org</span> <a href="mailto:datatable-help@lists.r-forge.r-project.org">datatable-help@lists.r-forge.r-project.org</a><br>Subject: <span style="color:black"> Re: [datatable-help] `with=F` in the `i` Argument <br></span></div><br> <blockquote type="cite" class="clean_bq"><span><div><div></div><div>Hi Arun,

<br>

<br>If `is.object` gives `FALSE` and you just have a list, you could wrap it

<br>in `unlist` as follows. It gives the same result for your cases. (These

<br>are just my two cents, maybe someone else has a different opinion.)

<br>

<br>require(data.table)

<br>DT <- data.table(x=c(1,4,3,2), y=c(8,6,5,7), z=c(10,12,11,9))

<br>

<br>## Case A.

<br>DT[base::order(DT[, "x", with=FALSE])]  # OK.

<br>## Case B.

DT[base::order(list(x))]                # Not OK.

<br>DT[base::order(unlist(list(x)))]        # Same as case A.

<br>

<br># Case C.

<br>DT[base::order(DT[, "x", with=FALSE], DT[, "y", with=FALSE])] # OK.

<br># Case D.

DT[base::order(list(x), list(y))]         # Not OK.

<br>DT[base::order(unlist(list(x), list(y)))] # Same as case C.

<br>

<br>## Case E.

<br>DT[base::order(DT[, c("x", "y"), with=FALSE])] # Pads NA for `y`.

<br>## Case F.

<br>DT[base::order(list(x, y))]             # Not OK.

<br>DT[base::order(unlist(list(x, y)))]     # Same as case E.

<br>

<br>

<br>Thanks,

<br>M

<br>

<br>

<br>

<br>

<br>On 06/21/2014 08:25 AM, Arunkumar Srinivasan wrote:

<br>> Michael,

<br>>  

<br>> Note that in your case, you can also do:

<br>>  

<br>> |DT <- data.table(a = 1:4, b = 8:5)

<br>> for (i in c("a", "b"))

<br>>     DT[order(DT[[i]])]

<br>> |

<br>>  

<br>> At the moment, I’m more inclined towards giving an error when any of the

<br>> arguments to |order(.)| results in a |list|. The message could be

<br>> something like:

<br>>  

> |DT[order(.)] on data.tables is optimised internally to use data.table's fast ordering. Since the behaviour of base:::order seems inconsistent in the way it handles list input - for ex: compare DT[order(list(x))] and DT[order(data.table(x))], we do not support list columns as input here. If you're sure, you can use `DT[base:::order(.)]` explicitly. However, this can be avoided most of the times by using `[[` to access specified columns to result in a vector.

<br>> |

<br>>  

> What do you (all) think?

<br>>  

<br>>  

<br>> Arun

<br>>  

<br>> From: Arunkumar Srinivasan aragorn168b@gmail.com

<br>> <mailto:aragorn168b@gmail.com>

<br>> Reply: Arunkumar Srinivasan aragorn168b@gmail.com

<br>> <mailto:aragorn168b@gmail.com>

<br>> Date: June 20, 2014 at 11:47:22 PM

<br>> To: Michael Smith my.r.help@gmail.com <mailto:my.r.help@gmail.com>

<br>> Cc: datatable-help@lists.r-forge.r-project.org

<br>> datatable-help@lists.r-forge.r-project.org

<br>> <mailto:datatable-help@lists.r-forge.r-project.org>

<br>> Subject: Re: [datatable-help] `with=F` in the `i` Argument

<br>>  

<br>>> This is a really tricky one. I was just trying to fix it when I

<br>>> recollected the issues with |base:::order| from the time during

<br>>> implementation.

<br>>>

<br>>> Consider this case:

<br>>>

<br>>> |require(data.table)

<br>>> DT <- data.table(x=c(1,4,3,2), y=c(8,6,5,7), z=c(10,12,11,9))

<br>>> |

<br>>>

<br>>> Consider the cases A and B below:

<br>>>

<br>>> |# case A

<br>>> DT[base:::order(DT[, "x", with=FALSE])]

<br>>> #    x y  z

<br>>> # 1: 1 8 10

<br>>> # 2: 2 7  9

<br>>> # 3: 3 5 11

<br>>> # 4: 4 6 12

<br>>> |

<br>>>

>> Intended right result. Great!

<br>>>

<br>>>

<br>>>       B:

<br>>>

<br>>> |# case B

<br>>> DT[base:::order(list(x))]

<br>>> #    x y  z

<br>>> # 1: 1 8 10

<br>>> |

<br>>>

<br>>> What just happened?!? So, basically if the list gives |TRUE| for

<br>>> |is.object(.)|, it understands what the opeation is, correctly. But if

<br>>> it’s /just/ a list, no idea how to deal with it. Also it silently

<br>>> returns undesirable result (imo).

<br>>>

<br>>> Similar to the above cases, compare these two:

<br>>>

<br>>> |# case C

<br>>> DT[base:::order(DT[, "x", with=FALSE], DT[, "y", with=FALSE])]

<br>>> # vs

<br>>> # case D

<br>>> DT[base:::order(list(x), list(y))]

<br>>> |

<br>>>

<br>>>

<br>>>       Even more crazy case:

<br>>>

<br>>> |# case E

<br>>> DT[base:::order(DT[, c("x", "y"), with=FALSE])]

<br>>> # vs

<br>>> # case F

<br>>> DT[base:::order(list(x,y))]

<br>>> |

<br>>>

<br>>> While we were testing and implementing |forder|, obviously it dint

>> occur to check with the argument to |order(.)| with a |data.table|.

<br>>> And in spite of the fact that the output for |DT[order(list(x))]| is a

<br>>> bit strange and even dangerous, to be consistent with |base:::order|,

>> we had implemented it the same way.

<br>>>

>> Now, I’m not so sure.. Any ideas justifying these differences?

<br>>>

<br>>>

<br>>>

<br>>> Arun

<br>>>

<br>>> From: Michael Smith my.r.help@gmail.com <mailto:my.r.help@gmail.com>

<br>>> Reply: Michael Smith my.r.help@gmail.com <mailto:my.r.help@gmail.com>

<br>>> Date: June 15, 2014 at 5:02:46 AM

<br>>> To: G See gsee000@gmail.com <mailto:gsee000@gmail.com>

<br>>> Cc: datatable-help@lists.r-forge.r-project.org

<br>>> datatable-help@lists.r-forge.r-project.org

<br>>> <mailto:datatable-help@lists.r-forge.r-project.org>

<br>>> Subject:  Re: [datatable-help] `with=F` in the `i` Argument

<br>>>

<br>>>> Devs,

<br>>>>

<br>>>> Is this a bug? It works in 1.9.2 but not in the 1.9.3 development

<br>>>> version:

<br>>>>

<br>>>> DT <- data.table(a = 1:4, b = 8:5)

<br>>>> for (i in c("a", "b"))

<br>>>> print(DT[order(DT[, i, with = FALSE])])

<br>>>>

<br>>>> Error in forder(DT, DT[, i, with = FALSE]) :

>>> Column '1' is type 'list' which is not supported for ordering currently.

<br>>>>

<br>>>>

<br>>>> Thanks,

<br>>>>

<br>>>> M

<br>>>>

<br>>>>

<br>>>> On 05/31/2014 12:44 PM, G See wrote:

<br>>>> > Hi Michael,

<br>>>> >

<br>>>> > I would use get()

<br>>>> >

<br>>>> > DT <- data.table(a = 1:4, b = 8:5)

<br>>>> > for (i in c("a", "b"))

<br>>>> > print(DT[order(get(i))])

<br>>>> >

<br>>>> > For what it's worth, your solution doesn't seem to work in data.table

<br>>>> > 1.9.3 (svn rev. 1278):

<br>>>> >

<br>>>> >> for (i in c("a", "b"))

<br>>>> > + print(DT[order(DT[, i, with = FALSE])])

<br>>>> > Error in forder(DT, DT[, i, with = FALSE]) :

>>> > Column '1' is type 'list' which is not supported for ordering currently.

<br>>>> >

<br>>>> >

<br>>>> > HTH,

<br>>>> > Garrett

<br>>>> >

<br>>>> > On Fri, May 30, 2014 at 11:01 PM, Michael Smith <my.r.help@gmail.com> wrote:

<br>>>> >> All,

<br>>>> >>

<br>>>> >> I'm trying to order the rows according to several columns at a time:

<br>>>> >>

<br>>>> >> DT <- data.table(a = 1:4, b = 8:5)

<br>>>> >> for (i in c("a", "b"))

<br>>>> >> print(DT[order(i), with = FALSE])

<br>>>> >>

<br>>>> >> It doesn't work, since `with` seems to be about the `j` argument, but

<br>>>> >> not the `i` argument, according to `?data.table`.

<br>>>> >>

<br>>>> >> I found the following workaround, but wonder whether there is a more

<br>>>> >> elegant way to do it:

<br>>>> >>

<br>>>> >> for (i in c("a", "b"))

<br>>>> >> print(DT[order(DT[, i, with = FALSE])])

<br>>>> >>

<br>>>> >> Thanks,

<br>>>> >> M

<br>>>> >> _______________________________________________

<br>>>> >> datatable-help mailing list

<br>>>> >> datatable-help@lists.r-forge.r-project.org

<br>>>> >> https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/datatable-help

<br>>>> _______________________________________________

<br>>>> datatable-help mailing list

<br>>>> datatable-help@lists.r-forge.r-project.org

<br>>>> https://lists.r-forge.r-project.org/cgi-bin/mailman/listinfo/datatable-help

<br>>  

<br></div></div></span></blockquote><p></p></body></html>