CLN/API: move plotting funcs to pandas.plotting
State: closed

closes #12548, rebase of #13579

Labels:
name: Visualization
description: plotting
name: Clean

Comments:
# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/16005?src=pr&el=h1) Report
> Merging [#16005](https://codecov.io/gh/pandas-dev/pandas/pull/16005?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/9c56098a5fdf2ce1bec989671a077722e64d647f?src=pr&el=desc) will **decrease** coverage by `0.02%`.
> The diff coverage is `71.16%`.

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

```diff
@@            Coverage Diff             @@
##           master   #16005      +/-   ##
==========================================
- Coverage   91.02%   90.99%   -0.03%     
==========================================
  Files         145      154       +9     
  Lines       50391    50474      +83     
==========================================
+ Hits        45870    45931      +61     
- Misses       4521     4543      +22
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `88.8% <71.16%> (-0.03%)` | :arrow_down: |
| #single | `40.36% <14.82%> (+0.02%)` | :arrow_up: |

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/16005?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/tools/plotting.py](https://codecov.io/gh/pandas-dev/pandas/pull/16005?src=pr&el=tree#diff-cGFuZGFzL3Rvb2xzL3Bsb3R0aW5nLnB5) | `100% <ø> (+25.38%)` | :arrow_up: |
| [pandas/tseries/plotting.py](https://codecov.io/gh/pandas-dev/pandas/pull/16005?src=pr&el=tree#diff-cGFuZGFzL3RzZXJpZXMvcGxvdHRpbmcucHk=) | `0% <0%> (-61.79%)` | :arrow_down: |
| [pandas/util/doctools.py](https://codecov.io/gh/pandas-dev/pandas/pull/16005?src=pr&el=tree#diff-cGFuZGFzL3V0aWwvZG9jdG9vbHMucHk=) | `0% <0%> (ø)` | :arrow_up: |
| [pandas/tseries/converter.py](https://codecov.io/gh/pandas-dev/pandas/pull/16005?src=pr&el=tree#diff-cGFuZGFzL3RzZXJpZXMvY29udmVydGVyLnB5) | `0% <0%> (-62.96%)` | :arrow_down: |
| [pandas/\_\_init\_\_.py](https://codecov.io/gh/pandas-dev/pandas/pull/16005?src=pr&el=tree#diff-cGFuZGFzL19faW5pdF9fLnB5) | `93.02% <100%> (+0.34%)` | :arrow_up: |
| [pandas/core/config\_init.py](https://codecov.io/gh/pandas-dev/pandas/pull/16005?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvY29uZmlnX2luaXQucHk=) | `95.34% <100%> (ø)` | :arrow_up: |
| [pandas/core/groupby.py](https://codecov.io/gh/pandas-dev/pandas/pull/16005?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvZ3JvdXBieS5weQ==) | `92.03% <100%> (ø)` | :arrow_up: |
| [pandas/core/frame.py](https://codecov.io/gh/pandas-dev/pandas/pull/16005?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvZnJhbWUucHk=) | `97.65% <100%> (ø)` | :arrow_up: |
| [pandas/core/series.py](https://codecov.io/gh/pandas-dev/pandas/pull/16005?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvc2VyaWVzLnB5) | `95.09% <100%> (+0.01%)` | :arrow_up: |
| [pandas/plotting/\_misc.py](https://codecov.io/gh/pandas-dev/pandas/pull/16005?src=pr&el=tree#diff-cGFuZGFzL3Bsb3R0aW5nL19taXNjLnB5) | `38.98% <38.98%> (ø)` | |
| ... and [20 more](https://codecov.io/gh/pandas-dev/pandas/pull/16005?src=pr&el=tree-more) | |

------

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

@jorisvandenbossche I am fine with merging this before #15998 (or after). lmk.
Fixed the comments (apart from import clean-up, can do later a PR)

Would like to get this in before I have to rebase again (it was a rather painful rebase :-))

the tseries.plotting (also tseries.converter is kept as a shim) question, can maybe deal with that when moving/reorganizing the tseries module.
> Would like to get this in before I have to rebase again (it was a rather painful rebase :-))

merge as soon as you are ready.

> the tseries.plotting (also tseries.converter is kept as a shim) question, can maybe deal with that when moving/reorganizing the tseries module.

sure (though I would deprecate those modules as you moved them). can do as a followup though.
ready to merge?
When it is green, yes
thanks!
and thanks @sinhrks for the original work!