Bug: Logical operator of Series with Index (#22092)
State: closed

- [ ] closes #22092 
- [ ] 1 test added for &, | and ^ logical operator of series
- [ ] whatsnew entry


Labels:
name: Bug
name: Dtype Conversions
description: Unexpected or buggy dtype conversions
name: Algos
description: Non-arithmetic algos: value_counts, factorize, sorting, isin, clip, shift, diff

Comments:
# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22293?src=pr&el=h1) Report
> Merging [#22293](https://codecov.io/gh/pandas-dev/pandas/pull/22293?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/996f361f8e6986ea1c65ccb164a4c585e1f4a027?src=pr&el=desc) will **not change** coverage.
> The diff coverage is `100%`.

[![Impacted file tree graph](https://codecov.io/gh/pandas-dev/pandas/pull/22293/graphs/tree.svg?width=650&token=eZ4WkYLtcO&height=150&src=pr)](https://codecov.io/gh/pandas-dev/pandas/pull/22293?src=pr&el=tree)

```diff
@@           Coverage Diff           @@
##           master   #22293   +/-   ##
=======================================
  Coverage   92.16%   92.16%           
=======================================
  Files         169      169           
  Lines       50708    50708           
=======================================
  Hits        46734    46734           
  Misses       3974     3974
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.57% <100%> (ø)` | :arrow_up: |
| #single | `42.35% <0%> (ø)` | :arrow_up: |

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/22293?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/core/ops.py](https://codecov.io/gh/pandas-dev/pandas/pull/22293/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvb3BzLnB5) | `96.94% <100%> (ø)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22293?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/22293?src=pr&el=footer). Last update [996f361...aaf8349](https://codecov.io/gh/pandas-dev/pandas/pull/22293?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

Changing from ``ABCIndex`` to ``ACBIndexClass``, it can handle ``Int64Index`` too. Thanks for your advice. @jbrockmendel 

The tests on ``Int64Index`` are also added. @jreback 
@jreback @jbrockmendel Would you tell me if there is anything to be implemented? Or failing one of the tests does matter?
Pending minor comments, LGTM.
Changed. Thanks for your reply
ping @jreback Could this be merged? 
ping @jreback  @jbrockmendel Is there anything to be implemented before this PR can be merged?
> Is there anything to be implemented before this PR can be merged?

@makbigc I can't speak for jreback, but rebasing will definitely be necessary
Hello @makbigc! Thanks for updating the PR.

 - There are no PEP8 issues in the file [`pandas/core/ops.py`](https://github.com/makbigc/pandas/blob/aaf8349e4bf6f5b8551e428643b6b29c748c2851/pandas/core/ops.py) !

 - There are no PEP8 issues in the file [`pandas/tests/series/test_operators.py`](https://github.com/makbigc/pandas/blob/aaf8349e4bf6f5b8551e428643b6b29c748c2851/pandas/tests/series/test_operators.py) !


@jreback do you anything else to be implemented? Please tell me.
looks good, can you rebase, ping on green.
@jreback changed. No conflict right now. 
lgtm. @jbrockmendel over to you
@jbrockmendel ok if ok, pls approve and merge