Skip to content

Dispatch IntervalIndex.argsort to IntervalArray #31582

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Dispatch IntervalIndex.argsort to IntervalArray #31582

wants to merge 1 commit into from

Conversation

@jbrockmendel
Copy link
Member

@jbrockmendel jbrockmendel commented Feb 2, 2020

cc @jschendel thoughts on if we can do better in the non-ascending/"quicksort" case?
@jreback jreback added the Interval Interval data type label Feb 2, 2020
@jreback
Copy link
Contributor

jreback commented Feb 2, 2020

any timings on the various cases?
@jreback jreback added this to the 1.1 milestone Feb 9, 2020
@jreback jreback requested a review from jschendel February 9, 2020 21:59
@jreback
Copy link
Contributor

jreback commented Feb 9, 2020

lgtm. perf timings?
@jbrockmendel
Copy link
Member Author

jbrockmendel commented Feb 10, 2020

havent done any timings, am hoping to get guidance from @jschendel on the remaining cases
@jbrockmendel
Copy link
Member Author

jbrockmendel commented Feb 19, 2020

Closing to clear the queue, will revisit, still hoping @jschendel can weigh in on how best to handle the remaining cases.
@jbrockmendel jbrockmendel added the Mothballed Temporarily-closed PR the author plans to return to label Apr 21, 2020
@jbrockmendel jbrockmendel deleted the ii-argsort branch January 20, 2021 00:13
@jbrockmendel jbrockmendel removed the Mothballed Temporarily-closed PR the author plans to return to label Jan 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Interval Interval data type

2 participants

@jbrockmendel @jreback