BrandonFors/pandas-github-issues huggingface dataset:
TST: drop tz-aware timestamp from DatetimIndex with DST transition
State: closed

- [x] closes #21761
- [x] tests added / passed
- [x] passes `git diff upstream/master -u -- "*.py" | flake8 --diff`
- [x] whatsnew entry


Labels:
name: Testing
description: pandas testing functions or related to the test suite
name: Datetime
description: Datetime data dtype
name: Indexing
description: Related to indexing on series/frames, not to indexes themselves

Comments:
Hello @mroeschke! Thanks for submitting the PR.

 - There are no PEP8 issues in the file [`pandas/tests/test_multilevel.py`](https://github.com/mroeschke/pandas/blob/e3d7d1abfb2dd7397641bdc4545235c19c60e2f9/pandas/tests/test_multilevel.py) !


# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/23479?src=pr&el=h1) Report
> Merging [#23479](https://codecov.io/gh/pandas-dev/pandas/pull/23479?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/8586644429a8c0054a74e645830601419da6613f?src=pr&el=desc) will **not change** coverage.
> The diff coverage is `n/a`.

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

```diff
@@           Coverage Diff           @@
##           master   #23479   +/-   ##
=======================================
  Coverage   92.23%   92.23%           
=======================================
  Files         161      161           
  Lines       51197    51197           
=======================================
  Hits        47220    47220           
  Misses       3977     3977
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.61% <ø> (ø)` | :arrow_up: |
| #single | `42.27% <ø> (ø)` | :arrow_up: |


------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/23479?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/23479?src=pr&el=footer). Last update [8586644...e3d7d1a](https://codecov.io/gh/pandas-dev/pandas/pull/23479?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

thanks @mroeschke
PERF: cython optimizations
State: closed

cython was emitting some warnings about untyped array lookups in `tslibs.conversion`; added types to those

`arr.fill(val)` is a python call, while `arr[:]` is a C call, so replaced all of those

modernized an old non-python for-loop, some other small cleanups along the way.

We're running out of available optimizations/cleanups in these files.  One thing I'd be interested in changing (but I'm not confident enough to do) is removing uses of `double` and `double_t`, which I _think_ are equivalent to `float64_t`.  But I'm not sure if that is architecture or platform dependent or something.

Labels:
name: Performance
description: Memory or execution speed performance
name: Internals
description: Related to non-user accessible pandas implementation

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

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

```diff
@@           Coverage Diff           @@
##           master   #23477   +/-   ##
=======================================
  Coverage   92.23%   92.23%           
=======================================
  Files         161      161           
  Lines       51197    51197           
=======================================
  Hits        47220    47220           
  Misses       3977     3977
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.61% <ø> (ø)` | :arrow_up: |
| #single | `42.27% <ø> (ø)` | :arrow_up: |


------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/23477?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/23477?src=pr&el=footer). Last update [aaaac86...a7b9aa0](https://codecov.io/gh/pandas-dev/pandas/pull/23477?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

> We're running out of available optimizations/cleanups in these files.

@jbrockmendel : Gosh, I wonder whose **"FAULT"** that is 😉 
> let's make this an AssertionError

Happy to do so, though now that I look closer, isn't this redundant with the check at the beginning of the function?
BTW any idea about the double_t thing?
what is the problem with double_t ?
this is just float64
> what is the problem with double_t ?  this is just float64

Yah that's the thing I want to double-check before changing all the places where we currently use `double` or `double_t` to `float64_t`
thanks!
DOC: Remove dead link and update links to https
State: closed

Removed one dead tutorial link and updated our doc links to https where available.


Labels:
name: Docs

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

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

```diff
@@           Coverage Diff           @@
##           master   #23476   +/-   ##
=======================================
  Coverage   92.23%   92.23%           
=======================================
  Files         161      161           
  Lines       51197    51197           
=======================================
  Hits        47220    47220           
  Misses       3977     3977
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.61% <ø> (ø)` | :arrow_up: |
| #single | `42.27% <ø> (ø)` | :arrow_up: |

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/23476?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/core/indexes/datetimes.py](https://codecov.io/gh/pandas-dev/pandas/pull/23476/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvaW5kZXhlcy9kYXRldGltZXMucHk=) | `96.41% <0%> (ø)` | :arrow_up: |
| [pandas/core/indexes/api.py](https://codecov.io/gh/pandas-dev/pandas/pull/23476/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvaW5kZXhlcy9hcGkucHk=) | `99% <0%> (ø)` | :arrow_up: |
| [pandas/core/generic.py](https://codecov.io/gh/pandas-dev/pandas/pull/23476/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvZ2VuZXJpYy5weQ==) | `96.81% <0%> (ø)` | :arrow_up: |
| [pandas/core/arrays/datetimes.py](https://codecov.io/gh/pandas-dev/pandas/pull/23476/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvYXJyYXlzL2RhdGV0aW1lcy5weQ==) | `98.84% <0%> (ø)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/23476?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/23476?src=pr&el=footer). Last update [d78bd7a...7b50cd0](https://codecov.io/gh/pandas-dev/pandas/pull/23476?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

@gfyoung just a subset, some of the links don't support https yet...
STY: standardize spacing for casting, with linting
State: closed

Casting in cython is done with the syntax `<type>obj` or `<type> obj`.  ATM the code uses both of these.  This PR standardizes on the first usage, with a lint check.

@datapythonista I'm not sure whether the grep check here belongs in the patterns section or in the non-python section.  LMK if you have a preference.

Labels:

Comments:
> prob should put all of these rules into a flake-cython package :)

If we can find someone else to maintain it, that'd be fantastic.
# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/23474?src=pr&el=h1) Report
> Merging [#23474](https://codecov.io/gh/pandas-dev/pandas/pull/23474?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/d78bd7a65f87022bae5021d36c8656c0ea5a7b76?src=pr&el=desc) will **not change** coverage.
> The diff coverage is `n/a`.

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

```diff
@@           Coverage Diff           @@
##           master   #23474   +/-   ##
=======================================
  Coverage   92.23%   92.23%           
=======================================
  Files         161      161           
  Lines       51197    51197           
=======================================
  Hits        47220    47220           
  Misses       3977     3977
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.61% <ø> (ø)` | :arrow_up: |
| #single | `42.27% <ø> (ø)` | :arrow_up: |


------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/23474?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/23474?src=pr&el=footer). Last update [d78bd7a...8d5e69c](https://codecov.io/gh/pandas-dev/pandas/pull/23474?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

Not for this PR, but a couple of other patterns that would be relatively easy to lint for:

`cimport numpy as np` --> should be `cnp` (not a widely-used convention, but it should be)
`*` imports
number of lines after function/class

For these purposes (and for this PR) we just assume away comments and docstrings.
Thanks @jbrockmendel, nice addition.

There are reasons for having it with both the linting and the patters, happy with any.
Fixing flake8 problems new to flake8 3.6.0
State: closed

- [ ] closes #xxxx
- [ ] tests added / passed
- [ ] passes `git diff upstream/master -u -- "*.py" | flake8 --diff`
- [ ] whatsnew entry

Seems like flake8 3.6.0 has just been released to conda. And when the CI uses the new version, it fails on the linting because a new validation `W605` which checks invalid escape characters. We have few in scripts and the conf of the documentation. Fixing those, so CI passes again.


Labels:
name: CI
description: Continuous Integration
name: Code Style
description: Code style, linting, code_checks

Comments:
Hello @datapythonista! Thanks for submitting the PR.

 - There are no PEP8 issues in the file [`doc/source/conf.py`](https://github.com/datapythonista/pandas/blob/9a5f5b6b57697a96ef0353e4fefc624d6a8e7b48/doc/source/conf.py) !

 - There are no PEP8 issues in the file [`scripts/find_commits_touching_func.py`](https://github.com/datapythonista/pandas/blob/9a5f5b6b57697a96ef0353e4fefc624d6a8e7b48/scripts/find_commits_touching_func.py) !

 - There are no PEP8 issues in the file [`scripts/find_undoc_args.py`](https://github.com/datapythonista/pandas/blob/9a5f5b6b57697a96ef0353e4fefc624d6a8e7b48/scripts/find_undoc_args.py) !


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

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

```diff
@@           Coverage Diff           @@
##           master   #23472   +/-   ##
=======================================
  Coverage   92.22%   92.22%           
=======================================
  Files         161      161           
  Lines       51187    51187           
=======================================
  Hits        47209    47209           
  Misses       3978     3978
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.61% <ø> (ø)` | :arrow_up: |
| #single | `42.26% <ø> (ø)` | :arrow_up: |


------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/23472?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/23472?src=pr&el=footer). Last update [f7f0c59...9a5f5b6](https://codecov.io/gh/pandas-dev/pandas/pull/23472?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

thanks @datapythonista nice that these are now being caught in flake
DOC: Updating DataFrame.join docstring
State: closed

- [X] closes #22897
- [ ] tests added / passed
- [ ] passes `git diff upstream/master -u -- "*.py" | flake8 --diff`
- [ ] whatsnew entry

Original work from @JustinZhengBC in #22904. The remote branch does not exist anymore, and I need to create this new PR to rebase.

Merging on green.

Labels:
name: Docs
name: Reshaping
description: Concat, Merge/Join, Stack/Unstack, Explode

Comments:
Hello @datapythonista! Thanks for submitting the PR.

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


# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/23471?src=pr&el=h1) Report
> Merging [#23471](https://codecov.io/gh/pandas-dev/pandas/pull/23471?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/528527b20f085524f484fef70dcde3c0b051782b?src=pr&el=desc) will **not change** coverage.
> The diff coverage is `n/a`.

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

```diff
@@           Coverage Diff           @@
##           master   #23471   +/-   ##
=======================================
  Coverage   92.25%   92.25%           
=======================================
  Files         161      161           
  Lines       51207    51207           
=======================================
  Hits        47239    47239           
  Misses       3968     3968
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.63% <ø> (ø)` | :arrow_up: |
| #single | `42.28% <ø> (ø)` | :arrow_up: |

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

------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/23471?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/23471?src=pr&el=footer). Last update [528527b...ebe5935](https://codecov.io/gh/pandas-dev/pandas/pull/23471?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

This should be ready to be merged.
DOC: clean doc string for NDFrame.astype
State: closed

Using astype(ordered..., categories=...) has been deprecated. This PR cleans a usage of this pattern in a doc string.


Labels:
name: Docs

Comments:
Hello @topper-123! Thanks for submitting the PR.

 - There are no PEP8 issues in the file [`pandas/core/generic.py`](https://github.com/topper-123/pandas/blob/3dc5b0d2cdcf5f03b05b2d1a9efca4475aceca97/pandas/core/generic.py) !


thanks @topper-123
BUG: fix groupby.transform rename bug (#23461)
State: closed

- [x] closes #23461
- [x] tests added / passed
- [x] passes `git diff upstream/master -u -- "*.py" | flake8 --diff`
- [x] whatsnew entry


Labels:
name: Bug
name: Groupby

Comments:
Hello @DiegoAlbertoTorres! Thanks for updating the PR.

 - There are no PEP8 issues in the file [`pandas/core/groupby/generic.py`](https://github.com/DiegoAlbertoTorres/pandas/blob/2aa4dc0430e60e6b190959511dd16c5f1fa8c398/pandas/core/groupby/generic.py) !

 - There are no PEP8 issues in the file [`pandas/tests/groupby/test_groupby.py`](https://github.com/DiegoAlbertoTorres/pandas/blob/2aa4dc0430e60e6b190959511dd16c5f1fa8c398/pandas/tests/groupby/test_groupby.py) !



##### Comment last updated on November 02, 2018 at 19:00 Hours UTC
Ugh, had forgotten to amend my commit. Pep8 bot should be happy now.
# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/23463?src=pr&el=h1) Report
> :exclamation: No coverage uploaded for pull request base (`master@ce8e05d`). [Click here to learn what that means](https://docs.codecov.io/docs/error-reference#section-missing-base-commit).
> The diff coverage is `100%`.

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

```diff
@@            Coverage Diff            @@
##             master   #23463   +/-   ##
=========================================
  Coverage          ?   92.25%           
=========================================
  Files             ?      161           
  Lines             ?    51171           
  Branches          ?        0           
=========================================
  Hits              ?    47209           
  Misses            ?     3962           
  Partials          ?        0
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.64% <100%> (?)` | |
| #single | `42.28% <0%> (?)` | |

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/23463?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/core/groupby/generic.py](https://codecov.io/gh/pandas-dev/pandas/pull/23463/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvZ3JvdXBieS9nZW5lcmljLnB5) | `87.07% <100%> (ø)` | |

------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/23463?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/23463?src=pr&el=footer). Last update [ce8e05d...5f76e40](https://codecov.io/gh/pandas-dev/pandas/pull/23463?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

@DiegoAlbertoTorres merge master and push again
Done (rebased).
lgtm. ping on green.
thanks @DiegoAlbertoTorres
BUG: Allow freq conversion from dt64 to period
State: closed

Closes #23438

In the PeriodArray refactor, I disallowed changing freq when converting from datetime64 to period. I didn't realize that was supported.

cc @thequackdaddy, does this close all the issues you ran into?

Labels:
name: Datetime
description: Datetime data dtype
name: Period
description: Period data type

Comments:
# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/23460?src=pr&el=h1) Report
> Merging [#23460](https://codecov.io/gh/pandas-dev/pandas/pull/23460?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/4f717551b8e49ecd978bb4f5a985d6f6a96d86e9?src=pr&el=desc) will **increase** coverage by `<.01%`.
> The diff coverage is `100%`.

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

```diff
@@            Coverage Diff             @@
##           master   #23460      +/-   ##
==========================================
+ Coverage   92.22%   92.23%   +<.01%     
==========================================
  Files         161      161              
  Lines       51187    51179       -8     
==========================================
- Hits        47209    47204       -5     
+ Misses       3978     3975       -3
```

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

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/23460?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/core/arrays/period.py](https://codecov.io/gh/pandas-dev/pandas/pull/23460/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvYXJyYXlzL3BlcmlvZC5weQ==) | `98.08% <100%> (+0.59%)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/23460?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/23460?src=pr&el=footer). Last update [4f71755...f765d21](https://codecov.io/gh/pandas-dev/pandas/pull/23460?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

thanks @TomAugspurger 

@jbrockmendel if you want to make an issue / note about the de-duplication would be great.
REF: cython cleanup, typing, optimizations
State: closed

Easy parts split off from #23368, #23382.

Labels:
name: Refactor
description: Internal refactoring of code
name: Internals
description: Related to non-user accessible pandas implementation
name: Clean

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

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

```diff
@@           Coverage Diff           @@
##           master   #23456   +/-   ##
=======================================
  Coverage   92.22%   92.22%           
=======================================
  Files         161      161           
  Lines       51191    51191           
=======================================
  Hits        47210    47210           
  Misses       3981     3981
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.6% <ø> (ø)` | :arrow_up: |
| #single | `42.26% <ø> (ø)` | :arrow_up: |


------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/23456?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/23456?src=pr&el=footer). Last update [93aba79...afdd39e](https://codecov.io/gh/pandas-dev/pandas/pull/23456?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

thanks