Skip to content

Fix mixup between mean and median #33106

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.

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 1 commit into from
Mar 29, 2020
Merged

Fix mixup between mean and median #33106

merged 1 commit into from
Mar 29, 2020

Conversation

oefe
Copy link
Contributor

@oefe oefe commented Mar 28, 2020

Text said "median", but code uses "mean". Make text match the code

  • closes #xxxx
  • tests added / passed
  • passes black pandas
  • passes git diff upstream/master -u -- "*.py" | flake8 --diff
  • whatsnew entry

Text said "median", but code uses "mean". Make text match the code
@@ -386,4 +386,4 @@ A full overview on time series is given in the pages on :ref:`time series and da

.. raw:: html

</div>
Copy link
Member

Choose a reason for hiding this comment

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

Hmm what changed here? Could you revert this diff?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The editor (I used the GitHub website) appended a newline here, so that the file ends with a newline. I'd suggest to leave it in, otherwise the next person trying to edit this file with GitHub will run into the same issue.

Or would you prefer a separate PR to fix the file endings? It looks like most, but not all, other files in the Pandas docs already end with a newline. (Some also with multiple newlines...)

Copy link
Contributor

Choose a reason for hiding this comment

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

this is ok

@mroeschke mroeschke added the Docs label Mar 29, 2020
@jreback jreback added this to the 1.1 milestone Mar 29, 2020
@jreback jreback merged commit 053c207 into pandas-dev:master Mar 29, 2020
@jreback
Copy link
Contributor

jreback commented Mar 29, 2020

thanks @oefe

@oefe oefe deleted the patch-1 branch March 29, 2020 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants