Add move_percentile() #30

Closed
kwgoodman opened this Issue Jun 23, 2011 · 3 comments

Comments

Projects
None yet
2 participants
@kwgoodman
Owner

kwgoodman commented Jun 23, 2011

Add a moving window scoreatpercentile function. It could be based on the move_median code with some slight modifications.

@ThomasLecocq

This comment has been minimized.

Show comment
Hide comment
@ThomasLecocq

ThomasLecocq Aug 17, 2015

👍 is this complicated to implement ?

👍 is this complicated to implement ?

@kwgoodman

This comment has been minimized.

Show comment
Hide comment
@kwgoodman

kwgoodman Aug 17, 2015

Owner

My four year old comment makes it sound easy. But you know how that goes. The move_median code is the most complicated code in bottleneck. And I'm not too excited about using it as the basis of other function until it can handle NaNs.

It is already a scoreatpercentile function if you want the score at the 50th percentile. You could hack something together to move the 50th percentile. If you do, let us know how it goes.

Owner

kwgoodman commented Aug 17, 2015

My four year old comment makes it sound easy. But you know how that goes. The move_median code is the most complicated code in bottleneck. And I'm not too excited about using it as the basis of other function until it can handle NaNs.

It is already a scoreatpercentile function if you want the score at the 50th percentile. You could hack something together to move the 50th percentile. If you do, let us know how it goes.

@kwgoodman kwgoodman closed this Mar 24, 2016

@kwgoodman

This comment has been minimized.

Show comment
Hide comment
@kwgoodman

kwgoodman May 21, 2016

Owner

bn.move_median now supports NaNs and min_count. So we are in good shape to add a move_percentile function.

Owner

kwgoodman commented May 21, 2016

bn.move_median now supports NaNs and min_count. So we are in good shape to add a move_percentile function.

@kwgoodman kwgoodman reopened this May 21, 2016

@kwgoodman kwgoodman changed the title from Add move_scoreatpercentile() to Add move_percentile() May 21, 2016

@kwgoodman kwgoodman closed this Jan 30, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment