Skip to content

dispatch Series[datetime64] comparison ops to DatetimeIndex #19800

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

Merged
merged 16 commits into from
Mar 1, 2018
Merged

dispatch Series[datetime64] comparison ops to DatetimeIndex #19800

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
1990e3f
dispatch Series[datetime64] comparison ops to DatetimeIndex
jbrockmendel Feb 20, 2018
f48e6d5
remove unsupported case
jbrockmendel Feb 20, 2018
bd909b0
fix error in older numpys
jbrockmendel Feb 20, 2018
003c4ff
Fixup copy/paste
jbrockmendel Feb 21, 2018
f942be9
unwrap Series to try to debug appveyor breakage
jbrockmendel Feb 21, 2018
59c96e8
port test for None comparison
jbrockmendel Feb 21, 2018
25534c6
troubleshoot appveyor build
jbrockmendel Feb 21, 2018
3d15998
continue troubleshooting Appveyor
jbrockmendel Feb 21, 2018
db487e8
test for date comparison
jbrockmendel Feb 22, 2018
ba290ab
just use isna
jbrockmendel Feb 22, 2018
f8d5bdd
comments
jbrockmendel Feb 24, 2018
9e59598
Merge branch 'master' of https://github.com/pandas-dev/pandas into dt…
jbrockmendel Feb 24, 2018
b618a82
comment
jbrockmendel Feb 24, 2018
35db0a2
Merge branch 'master' of https://github.com/pandas-dev/pandas into dt…
jbrockmendel Feb 25, 2018
3911541
Merge branch 'master' of https://github.com/pandas-dev/pandas into dt…
jbrockmendel Mar 1, 2018
ff51634
Update ops.py
jbrockmendel Mar 1, 2018
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
unwrap Series to try to debug appveyor breakage
  • Loading branch information
@jbrockmendel
jbrockmendel committed Feb 21, 2018
commit f942be918738ece803b0368c08720a2f25b50afd
4 changes: 4 additions & 0 deletions pandas/core/indexes/base.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3949,6 +3949,10 @@ def _evaluate_compare(self, other):
result = _comp_method_OBJECT_ARRAY(
op, self.values, other)
else:
if isinstance(other, ABCSeries):
Copy link
Contributor

@TomAugspurger TomAugspurger Feb 21, 2018

Choose a reason for hiding this comment

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

What are the potential types of other here? Series, Index, anything else? If it's just those two, I think unconditionally doing other = other._values will work. If other may also be an ndarray then ignore this.
Copy link
Member Author

@jbrockmendel jbrockmendel Feb 21, 2018

Choose a reason for hiding this comment

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

This was an unsuccessful attempt to debug a failing appveyor build. Let's circle back to the other._values idea once that problem is sorted out.
# Windows builds with some numpy versions (1.13)
# require specifically unwrapping Series GH#19800
other = other.values
with np.errstate(all='ignore'):
result = op(self.values, np.asarray(other))

Expand Down