Skip to content
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

Create all aggregation helpers like .var(), .std() #171

Closed
sethmlarson opened this issue Apr 10, 2020 · 3 comments · Fixed by #175
Closed

Create all aggregation helpers like .var(), .std() #171

sethmlarson opened this issue Apr 10, 2020 · 3 comments · Fixed by #175
Labels
good first issue Good for newcomers topic:dataframe Issue or PR about eland.DataFrame
Milestone

Comments

@sethmlarson
Copy link
Contributor

sethmlarson commented Apr 10, 2020

Currently we have NDFrame.mean(), .min(), etc but also need .var(), .std(), etc to be in line with Pandas.

@sethmlarson sethmlarson added good first issue Good for newcomers topic:dataframe Issue or PR about eland.DataFrame labels Apr 10, 2020
@sethmlarson sethmlarson added this to the v7.9.0 milestone Apr 10, 2020
@sethmlarson sethmlarson changed the title Create all aggregation helpers like .mean(), .var() etc Create all aggregation helpers like .mean(), .var() Apr 10, 2020
@mesejo
Copy link
Contributor

mesejo commented Apr 10, 2020

I think there is already one function in ndframe that it is mean. How is this different?

@sethmlarson
Copy link
Contributor Author

@mesejo Ah, then we only need to implement the ones that are missing. :)

@sethmlarson sethmlarson changed the title Create all aggregation helpers like .mean(), .var() Create all aggregation helpers like .var(), .std() Apr 10, 2020
@sethmlarson
Copy link
Contributor Author

Updated the title and issue to reflect this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers topic:dataframe Issue or PR about eland.DataFrame
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants