Skip to content

REF: use BlockManager.apply for DataFrameGroupBy.count #35924

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 17 commits into from
Sep 4, 2020
Merged

REF: use BlockManager.apply for DataFrameGroupBy.count #35924

merged 17 commits into from
Sep 4, 2020

Conversation

jbrockmendel
Copy link
Member

No description provided.

jbrockmendel and others added 17 commits August 20, 2020 21:19
@jbrockmendel
REF: remove unnecesary try/except
4c5eddd
@jbrockmendel
Merge branch 'master' of https://github.com/pandas-dev/pandas into re…
c632c9f 
…f-blockwise-3
@jbrockmendel
Merge branch 'master' of https://github.com/pandas-dev/pandas into re…
9e64be3 
…f-blockwise-3
@mathurk1 @jbrockmendel
TST: add test for agg on ordered categorical cols (pandas-dev#35630)
42649fb
@tkmz-n @jbrockmendel
TST: resample does not yield empty groups (pandas-dev#10603) (pandas-…
47121dd 
…dev#35799)
@jbrockmendel
revert accidental rebase
1decb3e
@jbrockmendel
Merge branch 'master' of https://github.com/pandas-dev/pandas into ma…
57c5dd3 
…ster
@jbrockmendel
Merge branch 'master' of https://github.com/pandas-dev/pandas into ma…
a358463 
…ster
@jbrockmendel
Merge branch 'master' of https://github.com/pandas-dev/pandas into ma…
ffa7ad7 
…ster
@jbrockmendel
Merge branch 'master' of https://github.com/pandas-dev/pandas into ma…
e5e98d4 
…ster
@jbrockmendel
Merge branch 'master' of https://github.com/pandas-dev/pandas into ma…
408db5a 
…ster
@jbrockmendel
Merge branch 'master' of https://github.com/pandas-dev/pandas into ma…
d3493cf 
…ster
@jbrockmendel
Merge branch 'master' of https://github.com/pandas-dev/pandas into ma…
75a805a 
…ster
@jbrockmendel
Merge branch 'master' of https://github.com/pandas-dev/pandas into ma…
9f61070 
…ster
@jbrockmendel
Merge branch 'master' of https://github.com/pandas-dev/pandas into ma…
2d10f6e 
…ster
@jbrockmendel
Merge branch 'master' of https://github.com/pandas-dev/pandas into ma…
3e20187 
…ster
@jbrockmendel
REF: use BlockManager.apply for DataFrameGroupBy.count
997393a
@jbrockmendel
Copy link
Member Author

@jreback gentle ping; this is the last place AFAICT where we can avoid directly iterating over blocks

@jreback jreback added the Internals Related to non-user accessible pandas implementation label Sep 4, 2020
@jreback jreback added this to the 1.2 milestone Sep 4, 2020
@jreback jreback merged commit 7d73641 into pandas-dev:master Sep 4, 2020
@jreback
Copy link
Contributor

jreback commented Sep 4, 2020

awesome!

@jbrockmendel jbrockmendel deleted the ref-count-apply branch September 4, 2020 20:50
@jbrockmendel jbrockmendel restored the ref-count-apply branch September 4, 2020 20:50
@jbrockmendel jbrockmendel deleted the ref-count-apply branch September 4, 2020 20:50
kesmit13 pushed a commit to kesmit13/pandas that referenced this pull request Nov 2, 2020
@jbrockmendel
REF: use BlockManager.apply for DataFrameGroupBy.count (pandas-dev#35924
dd0b7ad 
)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Internals Related to non-user accessible pandas implementation
Projects
None yet
Milestone
1.2
Development

Successfully merging this pull request may close these issues.
4 participants
@jbrockmendel @jreback @mathurk1 @tkmz-n