-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DOC: Fix EX01 issues in docstrings #37875
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
Comments
I'll take all |
I'll take all |
Out of curiosity, do types below need examples in docstrings?
|
Hi @xinrong-databricks,
I have seen that other dtype classes ( Nevertheless, modifying de docstrings in these classes might require a change in string variable |
Hi @lucasrodes, that really makes sense. Thank you for your clear explanations! |
It looks like GroupBy.apply example section was unintentionally removed here: https://github.com/pandas-dev/pandas/pull/24126/files#diff-db4158473de1d73771c56100a135f2bb80b2b29e2b573a763781a1a25e69c6dc I can send a patch to add it back. |
I've opened an example of how to make PRs for this issue now: #51091 Before submitting, you should run (replace
and check that You should then also remove the function you've fixed up to the exclusions in Feel free to take a few related functions per PR - I just did a single one to show an example |
I'll write examples for pandas.Series.shape |
Location of the documentation
In various pandas modules. See list below for the complete list of functions raising EX01 errors.
Documentation problem
The listed functions are missing the example section, i.e. code EX01.
Suggested fix for documentation
As proposed in #27977, I attach the list of raised errors below. Use this thread to comment on which function/module you will be working so we don't duplicate work. Please mention this issue in your PR so that the list below can be updated.
Current errors:
see the list here
pandas/ci/code_checks.sh
Lines 86 to 578 in 77e97d2
When you work on this issue, please choose a handful of functions, fix them up, and remove them from the exclusions list above
The text was updated successfully, but these errors were encountered: