Skip to content

[WIP]: Add quantile shortcut #15032

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.

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

Closed
wants to merge 2 commits into from
Closed

[WIP]: Add quantile shortcut #15032

wants to merge 2 commits into from

Conversation

srinivasreddy
Copy link
Contributor

@srinivasreddy srinivasreddy commented Jan 1, 2017

@jreback
Copy link
Contributor

jreback commented Jan 1, 2017

tests!

Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

look at how .mean is implemented

@codecov-io
Copy link

codecov-io commented Jan 1, 2017

Current coverage is 84.76% (diff: 33.33%)

Merging #15032 into master will decrease coverage by <.01%

@@             master     #15032   diff @@
==========================================
  Files           145        145          
  Lines         51131      51138     +7   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits          43346      43347     +1   
- Misses         7785       7791     +6   
  Partials          0          0          

Powered by Codecov. Last update 6141754...a3b713b

@@ -1026,6 +1026,22 @@ def mean(self, *args, **kwargs):

@Substitution(name='groupby')
@Appender(_doc_template)
def quantile(self, *args, **kwargs):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure what you are doing here, this doesn't affect .resample.

but more importantly w/o tests it is impossible to tell.

@srinivasreddy srinivasreddy changed the title Add quantile shortcut [WIP]: Add quantile shortcut Jan 2, 2017
@jreback jreback added API Design Resample resample method labels Jan 2, 2017
@jreback
Copy link
Contributor

jreback commented Feb 27, 2017

can you rebase / update

@jreback
Copy link
Contributor

jreback commented Mar 28, 2017

if you'd like to add tests and finish this up, pls ping.

@jreback jreback closed this Mar 28, 2017
@pwaller
Copy link
Contributor

pwaller commented Apr 28, 2017

Ping @srinivasreddy. Would have used this :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Design Resample resample method
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ENH: add quantile method to resample
4 participants