-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
PERF: groupby-cummax,cummin #15048
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
Labels
Milestone
Comments
mroeschke
added a commit
to mroeschke/pandas
that referenced
this issue
Jan 4, 2017
4 tasks
mroeschke
added a commit
to mroeschke/pandas
that referenced
this issue
Jan 5, 2017
mroeschke
added a commit
to mroeschke/pandas
that referenced
this issue
Jan 6, 2017
mroeschke
added a commit
to mroeschke/pandas
that referenced
this issue
Jan 7, 2017
pep8, removed args, kwargs add whatsnew + test + changed logic small error in algo Use a more obvious test Fixed algo & test passed
mroeschke
added a commit
to mroeschke/pandas
that referenced
this issue
Jan 8, 2017
pep8, removed args, kwargs add whatsnew + test + changed logic small error in algo Use a more obvious test Fixed algo & test passed Add dtypes test Add additional tests
mroeschke
added a commit
to mroeschke/pandas
that referenced
this issue
Jan 8, 2017
pep8, removed args, kwargs add whatsnew + test + changed logic small error in algo Use a more obvious test Fixed algo & test passed Add dtypes test Add additional tests handle nan case
mroeschke
added a commit
to mroeschke/pandas
that referenced
this issue
Jan 9, 2017
pep8, removed args, kwargs add whatsnew + test + changed logic small error in algo Use a more obvious test Fixed algo & test passed Add dtypes test Add additional tests handle nan case Adapt max/min for different dtypes + tests
mroeschke
added a commit
to mroeschke/pandas
that referenced
this issue
Jan 9, 2017
pep8, removed args, kwargs add whatsnew + test + changed logic small error in algo Use a more obvious test Fixed algo & test passed Add dtypes test Add additional tests handle nan case Adapt max/min for different dtypes + tests remove uncessary comments
mroeschke
added a commit
to mroeschke/pandas
that referenced
this issue
Jan 11, 2017
pep8, removed args, kwargs add whatsnew + test + changed logic small error in algo Use a more obvious test Fixed algo & test passed Add dtypes test Add additional tests handle nan case Adapt max/min for different dtypes + tests remove uncessary comments Added test & adjust algorithm
mroeschke
added a commit
to mroeschke/pandas
that referenced
this issue
Jan 11, 2017
pep8, removed args, kwargs add whatsnew + test + changed logic small error in algo Use a more obvious test Fixed algo & test passed Add dtypes test Add additional tests handle nan case Adapt max/min for different dtypes + tests remove uncessary comments Added test & adjust algorithm Fix linting errors
This was referenced Mar 3, 2017
AnkurDedania
pushed a commit
to AnkurDedania/pandas
that referenced
this issue
Mar 21, 2017
closes pandas-dev#15048 Author: Matt Roeschke <[email protected]> Closes pandas-dev#15053 from mroeschke/improve_cummin_cummax and squashes the following commits: 5e8ba63 [Matt Roeschke] PERF: Cythonize Groupby.cummin/cummax (pandas-dev#15048)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
these could be implemented in cython, should be very straightforward as we already have much templated code
The text was updated successfully, but these errors were encountered: