Skip to content

TST: Test named aggregations with functions #29262

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

Merged
merged 2 commits into from
Oct 29, 2019

Conversation

gfyoung
Copy link
Member

@gfyoung gfyoung commented Oct 29, 2019

Closes #28467

@gfyoung gfyoung added Testing pandas testing functions or related to the test suite Apply Apply, Aggregate, Transform, Map labels Oct 29, 2019
@gfyoung gfyoung added this to the 1.0 milestone Oct 29, 2019
@pep8speaks
Copy link

pep8speaks commented Oct 29, 2019

Hello @gfyoung! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2019-10-29 06:55:20 UTC

@gfyoung gfyoung force-pushed the named-agg-lambda-func branch 3 times, most recently from 7896056 to 82649a1 Compare October 29, 2019 04:58
@gfyoung gfyoung force-pushed the named-agg-lambda-func branch from 82649a1 to f11fcb1 Compare October 29, 2019 06:55
@jreback jreback merged commit 181c8aa into pandas-dev:master Oct 29, 2019
@jreback
Copy link
Contributor

jreback commented Oct 29, 2019

thanks @gfyoung

@gfyoung gfyoung deleted the named-agg-lambda-func branch October 29, 2019 15:44
Reksbril pushed a commit to Reksbril/pandas that referenced this pull request Nov 18, 2019
proost pushed a commit to proost/pandas that referenced this pull request Dec 19, 2019
proost pushed a commit to proost/pandas that referenced this pull request Dec 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Apply Apply, Aggregate, Transform, Map Testing pandas testing functions or related to the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't use a lambda function in named aggregation
4 participants