Skip to content

PERF: imporves performance in SeriesGroupBy.value_counts #10909

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 1 commit into from
Aug 27, 2015

Conversation

behzadnouri
Copy link
Contributor

Test name                                    | head[ms] | base[ms] |  ratio   |
-------------------------------------------------------------------------------
groupby_ngroups_10000_value_counts           |  11.4880 | 6698.5627 |   0.0017 |
groupby_ngroups_100_value_counts             |   1.3397 |  69.8457 |   0.0192 |
-------------------------------------------------------------------------------
Test name                                    | head[ms] | base[ms] |  ratio   |
-------------------------------------------------------------------------------

Ratio < 1.0 means the target commit is faster then the baseline.
Seed used: 1234

Target [d587be8] : PERF: imporves performance in SeriesGroupBy.value_counts
Base   [a3c4b59] : TST: pythonxs link seems to have changed in test_html.py, skip tests

@jreback jreback added Groupby Performance Memory or execution speed performance labels Aug 27, 2015
@jreback jreback added this to the 0.17.0 milestone Aug 27, 2015
@jreback
Copy link
Contributor

jreback commented Aug 27, 2015

wow, you made this crazy fast! thanks!

jreback added a commit that referenced this pull request Aug 27, 2015
PERF: imporves performance in SeriesGroupBy.value_counts
@jreback jreback merged commit c3a4de3 into pandas-dev:master Aug 27, 2015
@behzadnouri behzadnouri deleted the grby-val-cnt branch August 27, 2015 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Groupby Performance Memory or execution speed performance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants