Skip to content

DOC: fix various typos in plotting documentation #51176

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
Feb 5, 2023
Merged

Conversation

partev
Copy link
Contributor

@partev partev commented Feb 5, 2023

  • closes #xxxx (Replace xxxx with the GitHub issue number)
  • Tests added and passed if fixing a bug or adding a new feature
  • All code checks passed.
  • Added type annotations to new arguments/methods/functions.
  • Added an entry in the latest doc/source/whatsnew/vX.X.X.rst file if fixing a bug or adding a new feature.

@@ -308,7 +308,7 @@ def hist_frame(

See Also
--------
Series.plot.hist: Make a histogram.
pandas.Series.plot.hist: Make a histogram.
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

@partev partev Feb 5, 2023

Choose a reason for hiding this comment

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

there are two different boxplot functions apparently. The one you linked to renders correctly. There is another one in "plotting" folder. That one is not rendering correctly: https://pandas.pydata.org/docs/dev/reference/api/pandas.plotting.boxplot.html

Copy link
Member

Choose a reason for hiding this comment

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

Ah nice catch, thx

@@ -274,12 +274,13 @@ def andrews_curves(

Andrews curves have the functional form:

f(t) = x_1/sqrt(2) + x_2 sin(t) + x_3 cos(t) +
x_4 sin(2t) + x_5 cos(2t) + ...
.. math::
Copy link
Member

Choose a reason for hiding this comment

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

Can you show a screen shot how this renders?

Copy link
Contributor Author

@partev partev Feb 5, 2023

Choose a reason for hiding this comment

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

@phofl phofl added this to the 2.0 milestone Feb 5, 2023
@phofl phofl merged commit aebcfdf into pandas-dev:main Feb 5, 2023
@phofl
Copy link
Member

phofl commented Feb 5, 2023

thx @partev

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.

2 participants