[ 
https://issues.apache.org/jira/browse/ARROW-6232?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sutou Kouhei resolved ARROW-6232.
---------------------------------
    Fix Version/s: 0.15.0
       Resolution: Fixed

Issue resolved by pull request 5080
[https://github.com/apache/arrow/pull/5080]

> [C++] Rename Argsort kernel to SortToIndices
> --------------------------------------------
>
>                 Key: ARROW-6232
>                 URL: https://issues.apache.org/jira/browse/ARROW-6232
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>            Reporter: Sutou Kouhei
>            Assignee: Sutou Kouhei
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 0.15.0
>
>          Time Spent: 3h 40m
>  Remaining Estimate: 0h
>
> "Argsort" is NumPy specific name. Other languages/libraries use
> different name:
>   * R: order
>     * 
> https://cran.r-project.org/doc/manuals/r-release/fullrefman.pdf#Rfn.order
>   * MATLAB: sort
>     * https://mathworks.com/help/matlab/ref/sort.html
>     * "sort" returns sorted array and indices to sort array
>   * Julia: sortperm
>     * 
> https://pkg.julialang.org/docs/julia/THl1k/1.1.1/base/sort.html#Base.sortperm
> It's better that we use general name because Arrow C++ isn't a NumPy
> compatible library.
> "SortToIndices" means "sort that returns indices array".



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

Reply via email to