[
https://issues.apache.org/jira/browse/ARROW-6232?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sutou Kouhei updated ARROW-6232:
--------------------------------
Description:
"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".
was:
"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.
"SortIndices" means "sort that returns indices array". We can add
"SortValues" or something for sort kernel that returns values array.
"SortIndices" may be easily mistaken for "sort by indices".
> [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
> Time Spent: 3h
> 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)
