Skip to content

BUG: Fix KeyError with Groupby label#5467

Merged
quasiben merged 3 commits intodask:masterfrom
ryankarlos:groupby-support-label-bug-fix
Oct 7, 2019
Merged

BUG: Fix KeyError with Groupby label#5467
quasiben merged 3 commits intodask:masterfrom
ryankarlos:groupby-support-label-bug-fix

Conversation

@ryankarlos
Copy link
Copy Markdown
Contributor

@ryankarlos ryankarlos commented Oct 7, 2019

@ryankarlos ryankarlos changed the title Groupby support label bug fix BUG: Allow label with Groupby Oct 7, 2019
@ryankarlos
Copy link
Copy Markdown
Contributor Author

@jrbourbeau had a go at this as my first dask PR - let me know if any changes need to be made

@ryankarlos ryankarlos changed the title BUG: Allow label with Groupby BUG: Fix KeyError with Groupby label Oct 7, 2019
@quasiben
Copy link
Copy Markdown
Member

quasiben commented Oct 7, 2019

Thanks for the contribution @ryankarlos !

@quasiben
Copy link
Copy Markdown
Member

quasiben commented Oct 7, 2019

Looks good to me -- merging in

@quasiben quasiben merged commit 57a749b into dask:master Oct 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

dask.dataframe groupby doesn work when column

2 participants