-
-
Notifications
You must be signed in to change notification settings - Fork 18.7k
DOC: styling clean-up of docstrings (part 1, frame.py) #5560
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
DOC: styling clean-up of docstrings (part 1, frame.py) #5560
Conversation
@@ -2010,7 +2016,7 @@ def xs(self, key, axis=0, level=None, copy=True, drop_level=True): | |||
which levels are used. Levels can be referred by label or position. | |||
copy : boolean, default True | |||
Whether to make a copy of the data | |||
drop_level, default True | |||
drop_level : boolean, default True |
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.
bool vs. boolean?
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.
What do you prefer?
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.
A quick count: ca 6 times bool
is used and around 50 times boolan
in all docstrings in frame.py.
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.
okay so go with boolean
👍 |
@@ -2146,13 +2152,11 @@ def lookup(self, row_labels, col_labels): | |||
|
|||
Notes | |||
----- | |||
Akin to | |||
Akin to:: |
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.
double : intentional here?
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.
yes, so it's regarded as code block.
@jorisvandenbossche is this ready to be merged? (I'm going to look through one more time, but just wanted to make sure you aren't planning to do anything else) |
@jtratner Yes, ready to merge! |
…leanup DOC: styling clean-up of docstrings (part 1, frame.py)
Looks good - thanks! |
Some stylistic clean-up of the docstrings in frame.py.
Things changed are mainly:
:
inparam : type
method
keyword explanation in http://pandas.pydata.org/pandas-docs/dev/generated/pandas.DataFrame.corr.html