Skip to content

Conversation

@GYHHAHA
Copy link
Contributor

@GYHHAHA GYHHAHA commented Nov 30, 2020

@jreback jreback added this to the 1.2 milestone Dec 2, 2020
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.

lgtm. @TomAugspurger @jbrockmendel if you have a moment to review

)
.T.flatten()
)
return result.take(indices, axis=1)
Copy link
Member

Choose a reason for hiding this comment

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

could a lot of this be shared with the axis=0 version by replacing result.columns with result.axes[1] and above replacing result.index with result.axes[0]? (which in both cases would then become result.axes[axis])

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sounds good, I will change this.

- Bug in :meth:`DataFrame.groupby` dropped ``nan`` groups from result with ``dropna=False`` when grouping over a single column (:issue:`35646`, :issue:`35542`)
- Bug in :meth:`.DataFrameGroupBy.head`, :meth:`.DataFrameGroupBy.tail`, :meth:`SeriesGroupBy.head`, and :meth:`SeriesGroupBy.tail` would raise when used with ``axis=1`` (:issue:`9772`)
- Bug in :meth:`.DataFrameGroupBy.transform` would raise when used with ``axis=1`` and a transformation kernel (e.g. "shift") (:issue:`36308`)
- Bug in :meth:`.DataFrameGroupBy.quantile` couldn't handle with arraylike ``q`` when grouping by columns (:issue:`35269`)
Copy link
Member

Choose a reason for hiding this comment

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

extra period snuck in

@pep8speaks
Copy link

pep8speaks commented Dec 2, 2020

Hello @GYHHAHA! 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 2020-12-02 12:33:48 UTC
@jreback jreback merged commit 5fdf642 into pandas-dev:master Dec 2, 2020
@jreback
Copy link
Contributor

jreback commented Dec 2, 2020

thanks @GYHHAHA very nice

@GYHHAHA GYHHAHA deleted the fix-quantile branch December 3, 2020 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

5 participants