-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DOC: Fixes to docstrings, mainly PR09 errors #29530
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 52 commits
fb76a17
0f40afb
94ac074
e8a5ab1
90cc836
9e6d290
64b7c75
d84b2a1
214add3
705f441
b3f1074
9d7f405
cb524fd
9a517aa
b99d417
8095d30
e8c4c50
a86a34b
76daea6
e6094eb
04bc5a4
2e427e9
735e6c2
0292b1e
2f3b875
405da2b
4f718eb
84ab4da
60d9a1f
7d1bcdd
9585d6e
9d5ed04
a049e8d
af1d4d9
abe0eb8
44755f1
796828f
6e703aa
4525b08
1fd931a
a5eb1a9
3ff70b4
080a77b
de22262
c56b0f9
18f492e
0826a6f
25167fb
c4848eb
0f9c6d1
2f72294
d434831
fdc670e
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 |
---|---|---|
|
@@ -132,11 +132,12 @@ | |
Name or list of names to sort by. | ||
|
||
- if `axis` is 0 or `'index'` then `by` may contain index | ||
levels and/or column labels | ||
levels and/or column labels. | ||
- if `axis` is 1 or `'columns'` then `by` may contain column | ||
levels and/or index labels | ||
levels and/or index labels. | ||
|
||
.. versionchanged:: 0.23.0 | ||
|
||
Allow specifying index or column level names.""", | ||
versionadded_to_excel="", | ||
optional_labels="""labels : array-like, optional | ||
|
@@ -2112,9 +2113,10 @@ def to_html( | |
A ``border=border`` attribute is included in the opening | ||
`<table>` tag. Default ``pd.options.display.html.border``. | ||
encoding : str, default "utf-8" | ||
Set character encoding | ||
Set character encoding. | ||
|
||
.. versionadded:: 1.0 | ||
|
||
table_id : str, optional | ||
A css id is included in the opening `<table>` tag if specified. | ||
|
||
|
@@ -7140,17 +7142,16 @@ def corr(self, method="pearson", min_periods=1): | |
Parameters | ||
---------- | ||
method : {'pearson', 'kendall', 'spearman'} or callable | ||
Method of correlation: | ||
|
||
Method of correlation. | ||
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. You need to keep the blank line between the first sentence and the bullet points (I think that's the reason sphinx is complaining). BTW, I think the colon ( 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. yup, thanks. That was just me playing with it to see if I could get the validation to accept it |
||
* pearson : standard correlation coefficient | ||
* kendall : Kendall Tau correlation coefficient | ||
* spearman : Spearman rank correlation | ||
* callable: callable with input two 1d ndarrays | ||
and returning a float. Note that the returned matrix from corr | ||
will have 1 along the diagonals and will be symmetric | ||
regardless of the callable's behavior. | ||
and returning a float. Note that the returned matrix from corr | ||
will have 1 along the diagonals and will be symmetric | ||
regardless of the callable's behavior. | ||
|
||
.. versionadded:: 0.24.0 | ||
.. versionadded:: 0.24.0 | ||
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. did you check if the html version of this docstring looks good? those indentations are always tricky 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. This is correct (it has the same indentation as the list item) |
||
|
||
min_periods : int, optional | ||
Minimum number of observations required per pair of columns | ||
|
@@ -7716,7 +7717,7 @@ def idxmin(self, axis=0, skipna=True): | |
Parameters | ||
---------- | ||
axis : {0 or 'index', 1 or 'columns'}, default 0 | ||
The axis to use. 0 or 'index' for row-wise, 1 or 'columns' for column-wise | ||
The axis to use. 0 or 'index' for row-wise, 1 or 'columns' for column-wise. | ||
skipna : bool, default True | ||
Exclude NA/null values. If an entire row/column is NA, the result | ||
will be NA. | ||
|
@@ -7754,7 +7755,7 @@ def idxmax(self, axis=0, skipna=True): | |
Parameters | ||
---------- | ||
axis : {0 or 'index', 1 or 'columns'}, default 0 | ||
The axis to use. 0 or 'index' for row-wise, 1 or 'columns' for column-wise | ||
The axis to use. 0 or 'index' for row-wise, 1 or 'columns' for column-wise. | ||
skipna : bool, default True | ||
Exclude NA/null values. If an entire row/column is NA, the result | ||
will be NA. | ||
|
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.
you should add the error you fixed here too, the previous is only a message for the log