arkouda
arkouda copied to clipboard
Separate grouping and sorting semantics
This issue contains the requested functionality previously captured in Issue #846 that was not addressed by PR #1330
- Separate grouping and sorting semantics
-
argsortandcoargsortshould actually sort the array(s). Currently, callingcoargsorton a list including a Strings or Categorical will only group, not sort. -
GroupByshould guarantee grouping, but not necessarily sorting. Strings and Categorical should have separate APIs for sorting and grouping, andGroupByshould call the latter.
-
The high-level actions required by this issue are:
- [x] Create a server message specifically for
GroupBythat takes over the grouping logic currently employed bycoargsort - [ ] Modify
coargsortto actually sort the data, rather than just grouping it (for strings)
@reuster986 or anyone else, feel free to add anything I missed from the original issue
Note this issue is related to and summarizes the conversation in #677. I would recommend closing that issue in favor of this one