-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Fix SS06 formatting errors #29255
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
Fix SS06 formatting errors #29255
Conversation
SS06 errors fixed for the following: pandas.DataFrame.slice_shift pandas.DataFrame.to_period pandas.DataFrame.plot pandas.DataFrame.sparse.density pandas.DataFrame.style pandas.Series.as_blocks pandas.Series.ix pandas.Series.ptp pandas.DataFrame.as_blocks pandas.DataFrame.ix
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. Minor comments to improve styling / remove repetition
represented in the DataFrame. | ||
Ratio of non-sparse points to total (dense) data points. | ||
|
||
Ratio of non-sparse points to total (dense) data points represented |
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 think you can remove this extended summary; it says the same thing as the first line
pandas/core/generic.py
Outdated
|
||
.. deprecated:: 0.21.0 | ||
|
||
Convert the frame to a dict of dtype-> Constructor Types that each |
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.
Remove this as well; just duplicative
pandas/core/frame.py
Outdated
building a styled HTML representation fo the DataFrame. | ||
Returns a Styler object. | ||
|
||
Property returning a Styler object containing methods for building |
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 like the word of
is misspelled here
Co-Authored-By: William Ayd <[email protected]>
Co-Authored-By: William Ayd <[email protected]>
Co-Authored-By: William Ayd <[email protected]>
…andas into fix-docstring-format
Hi @WillAyd, added the changes you suggested. |
Great thanks @profwacko ! |
SS06 (Summary should fit in a single line) errors fixed (#29254) for the following:
Validated with
python scripts/validate_docstrings.py
Also referencing issue: #27977