DOC: Fix order section in docstrings (part II)
State: closed

- [X] closes #24125 
- [x] passes `git diff upstream/master -u -- "*.py" | flake8 --diff`

This is to build on #24126 .

As I commented in the issue, this pull request will fix additional 10 out of 24 errors. 14 errors will remain and are referenced in the issue.

Labels:
name: Docs
name: Clean

Comments:
Hello @alexander-ponomaroff! Thanks for submitting the PR.

 - There are no PEP8 issues in the file [`pandas/core/accessor.py`](https://github.com/alexander-ponomaroff/pandas/blob/6efd5f5d82640b879ae35a6e69de734e4bff45ac/pandas/core/accessor.py) !

 - There are no PEP8 issues in the file [`pandas/core/groupby/groupby.py`](https://github.com/alexander-ponomaroff/pandas/blob/6efd5f5d82640b879ae35a6e69de734e4bff45ac/pandas/core/groupby/groupby.py) !

 - There are no PEP8 issues in the file [`pandas/core/resample.py`](https://github.com/alexander-ponomaroff/pandas/blob/6efd5f5d82640b879ae35a6e69de734e4bff45ac/pandas/core/resample.py) !

 - There are no PEP8 issues in the file [`pandas/core/window.py`](https://github.com/alexander-ponomaroff/pandas/blob/6efd5f5d82640b879ae35a6e69de734e4bff45ac/pandas/core/window.py) !


# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/24132?src=pr&el=h1) Report
> Merging [#24132](https://codecov.io/gh/pandas-dev/pandas/pull/24132?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/03134cb92ed61d073438daac7a7c2a0e094efede?src=pr&el=desc) will **decrease** coverage by `49.18%`.
> The diff coverage is `n/a`.

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

```diff
@@             Coverage Diff             @@
##           master   #24132       +/-   ##
===========================================
- Coverage    92.2%   43.02%   -49.19%     
===========================================
  Files         162      162               
  Lines       51701    51701               
===========================================
- Hits        47671    22244    -25427     
- Misses       4030    29457    +25427
```

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

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/24132?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/core/window.py](https://codecov.io/gh/pandas-dev/pandas/pull/24132/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvd2luZG93LnB5) | `28.93% <ø> (-67.47%)` | :arrow_down: |
| [pandas/core/accessor.py](https://codecov.io/gh/pandas-dev/pandas/pull/24132/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvYWNjZXNzb3IucHk=) | `63.85% <ø> (-34.94%)` | :arrow_down: |
| [pandas/core/groupby/groupby.py](https://codecov.io/gh/pandas-dev/pandas/pull/24132/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvZ3JvdXBieS9ncm91cGJ5LnB5) | `25.03% <ø> (-71.48%)` | :arrow_down: |
| [pandas/core/resample.py](https://codecov.io/gh/pandas-dev/pandas/pull/24132/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvcmVzYW1wbGUucHk=) | `23.36% <ø> (-73.63%)` | :arrow_down: |
| [pandas/io/formats/latex.py](https://codecov.io/gh/pandas-dev/pandas/pull/24132/diff?src=pr&el=tree#diff-cGFuZGFzL2lvL2Zvcm1hdHMvbGF0ZXgucHk=) | `0% <0%> (-100%)` | :arrow_down: |
| [pandas/core/categorical.py](https://codecov.io/gh/pandas-dev/pandas/pull/24132/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvY2F0ZWdvcmljYWwucHk=) | `0% <0%> (-100%)` | :arrow_down: |
| [pandas/io/sas/sas\_constants.py](https://codecov.io/gh/pandas-dev/pandas/pull/24132/diff?src=pr&el=tree#diff-cGFuZGFzL2lvL3Nhcy9zYXNfY29uc3RhbnRzLnB5) | `0% <0%> (-100%)` | :arrow_down: |
| [pandas/tseries/plotting.py](https://codecov.io/gh/pandas-dev/pandas/pull/24132/diff?src=pr&el=tree#diff-cGFuZGFzL3RzZXJpZXMvcGxvdHRpbmcucHk=) | `0% <0%> (-100%)` | :arrow_down: |
| [pandas/tseries/converter.py](https://codecov.io/gh/pandas-dev/pandas/pull/24132/diff?src=pr&el=tree#diff-cGFuZGFzL3RzZXJpZXMvY29udmVydGVyLnB5) | `0% <0%> (-100%)` | :arrow_down: |
| [pandas/io/formats/html.py](https://codecov.io/gh/pandas-dev/pandas/pull/24132/diff?src=pr&el=tree#diff-cGFuZGFzL2lvL2Zvcm1hdHMvaHRtbC5weQ==) | `0% <0%> (-98.62%)` | :arrow_down: |
| ... and [123 more](https://codecov.io/gh/pandas-dev/pandas/pull/24132/diff?src=pr&el=tree-more) | |

------

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

# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/24132?src=pr&el=h1) Report
> Merging [#24132](https://codecov.io/gh/pandas-dev/pandas/pull/24132?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/040f06f731a09fc6e0663cada6697f6602b36f1d?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/24132/graphs/tree.svg?width=650&token=eZ4WkYLtcO&height=150&src=pr)](https://codecov.io/gh/pandas-dev/pandas/pull/24132?src=pr&el=tree)

```diff
@@           Coverage Diff           @@
##           master   #24132   +/-   ##
=======================================
  Coverage   92.22%   92.22%           
=======================================
  Files         162      162           
  Lines       51828    51828           
=======================================
  Hits        47798    47798           
  Misses       4030     4030
```

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

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/24132?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/core/accessor.py](https://codecov.io/gh/pandas-dev/pandas/pull/24132/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvYWNjZXNzb3IucHk=) | `98.79% <ø> (ø)` | :arrow_up: |
| [pandas/core/groupby/groupby.py](https://codecov.io/gh/pandas-dev/pandas/pull/24132/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvZ3JvdXBieS9ncm91cGJ5LnB5) | `96.67% <ø> (ø)` | :arrow_up: |
| [pandas/core/resample.py](https://codecov.io/gh/pandas-dev/pandas/pull/24132/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvcmVzYW1wbGUucHk=) | `96.58% <ø> (ø)` | :arrow_up: |
| [pandas/core/window.py](https://codecov.io/gh/pandas-dev/pandas/pull/24132/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvd2luZG93LnB5) | `96.39% <ø> (ø)` | :arrow_up: |

------

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

can you rebase
@alexander-ponomaroff seems like you've got unrelated changes to this PR, can you take a look please
`0cfd1c2f000ad9c4cf8e7a170e85c2971a99cf02` and `f74fc59793ce1f01e1afcdd29bcdf0dceccaf244` commits were added to this pull request by @jreback right? Or did I mess something up?
When we ask for a rebase, it mainly means to merge master into your branch, so you have your changes on top of the latest version. So, you should have a merge commit, but not commits from master directly or other PRs.
@datapythonista I guess my experimentation went wrong :/. I reverted everything and merged everything properly now right?

Last night I just merged master into my branch but it failed the CI checks, so I thought I was doing something wrong and tried some experimentation, which was not a good idea.
Looks great, but the failures in the CI are because of a test that checks a docstring. Can you take a look and update the test please?
@jreback I fixed the problems in this PR, and all green now, should be ready to go.

There are still few docstrings with the wrong order, we'll be adding the validation to the CI in the follow up issue to fix them: #24280
@jreback if you have time to take a quick look at this. Should be ready to be merged, and will block #24288 soon, which will add the linting for this fix. Thanks!
thanks @alexander-ponomaroff for the fixes

and thanks @jreback for the review
@datapythonista No problem. Sorry for not fixing the failures that happened in the tests quickly, I had exams and was very busy studying for those.
No worries. Are you planning to work on moving the CircleCI build to Travis? I was thinking on taking that myself if you don't.
@datapythonista It would be great if you could take that over. I've been struggling with how to go about moving CicleCI build to Travis and didn't want to be annoying and ask too many questions, so I haven't gotten very far when I was working on it. Then my exams took over all of my time and I had to stop working on it, so if you are ok with taking it over from now, it would be much appreciated. Sorry about this.
no worries, thanks for giving it a try, and for letting me know