-
-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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
BUG: Handle infinite values correctly in Series.argmax #13625
Conversation
@@ -1290,3 +1290,4 @@ Bug Fixes | |||
- Bug when specifying a UTC ``DatetimeIndex`` by setting ``utc=True`` in ``.to_datetime`` (:issue:`11934`) | |||
- Bug when increasing the buffer size of CSV reader in ``read_csv`` (:issue:`12494`) | |||
- Bug when setting columns of a ``DataFrame`` with duplicate column names (:issue:`12344`) | |||
- Bug when calling Series.argmax with infinite values (:issue: `13595`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move to 0.19.0
@@ -549,3 +549,5 @@ Bug Fixes | |||
- Bug in ``groupby`` with ``as_index=False`` returns all NaN's when grouping on multiple columns including a categorical one (:issue:`13204`) | |||
|
|||
- Bug where ``pd.read_gbq()`` could throw ``ImportError: No module named discovery`` as a result of a naming conflict with another python package called apiclient (:issue:`13454`) | |||
|
|||
- Bug when calling Series.argmax with infinite values (:issue: `13595`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use double-back ticks around Series.argmax
no space after ``:issue:13595
)
@matthewwillian You have some PEP8 issues (see the travis log at the bottom: https://travis-ci.org/pydata/pandas/jobs/144182270), and a failing test. |
Okay, thank you. I will take a look. |
@matthewwillian Can you update/rebase? |
It's great if you can include the fix for
|
can you rebase |
can you rebase / update? |
can you rebase and move release notes to 0.19.2 |
can you rebase / update |
closing, but if you want to update, pls comment. |
git diff upstream/master | flake8 --diff