-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Rt05 documentation error fix issue 25108 #25309
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
Changes from 23 commits
777fe4c
a595cac
507d21e
950f272
ba5120f
3cd359f
0266006
86456dd
a86736b
7d57782
b47b886
63f0468
8d4c6b4
a15ec00
01a7b19
998f649
77bd88e
b090af1
20adfce
0514c6f
0038a73
021ede4
f3a5248
421bab7
24b20a3
ae87624
8b3d251
6b3ed7b
dd242ea
83e120f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2061,7 +2061,7 @@ def plot_series(data, kind='line', ax=None, # Series unique | |
For data grouped with ``by``: | ||
|
||
* :class:`~pandas.Series` | ||
* :class:`~numpy.array` (for ``return_type = None``) | ||
* :class:`~numpy.array` (for ``return_type = None``). | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Similar comment here. Maybe move the majority of this into Notes and have the description here just say "See Notes." There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. agreed, I moved the majority of the return documentation into notes. Let me know what you think. One alternative would be to simply say 'See Notes.' instead of including the return type. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same comment |
||
|
||
See Also | ||
-------- | ||
|
@@ -3332,7 +3332,7 @@ def area(self, x=None, y=None, **kwds): | |
Returns | ||
------- | ||
matplotlib.axes.Axes or numpy.ndarray | ||
Area plot, or array of area plots if subplots is True | ||
Area plot, or array of area plots if subplots is True. | ||
|
||
See Also | ||
-------- | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suppose this "works" but seems hacky and rather long for a return. Can you move some of this into a
Notes
section instead?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
agreed, I put the basic return documentation in the Return section. Everything else moved to a new Notes section
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good but can remove trailing period now