-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
BUG: validate_docstrings has many warnings #44642
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 a look; if no answer or Pr from my side in 12 hours call a real developer pls 🥲 |
Sorry, but I can't replicate this issue on my computer... Anyway, if anyone would like to contribute they just have to go to scripts/validate_docstrings.py and :
When i try to run the command it throw me back: |
@IsNotMyIP - Just a guess, but I think you need to recompile the libs |
@TomAugspurger can you assign this issue to me |
@rhshadrach I'll check if I can reproduce this on my Windows, I tried in a MacOS.... @gourcool Go for it! If you can replicate the bug in your computer should be easy to fix, let me know if you have any doubt. |
@gourcool Any updates on this? I am thinking to pick this issue up if there isn't much progress yet. |
yeah, I couldn't reproduce this issue on my machine either |
take |
Fixed some of these in #47080, and opened #47079 due to some resample warnings. However many that remain I don't believe can or should be fixed. Here are some categories of warnings I don't believe we can/should fix:
|
Is this issue still open, or is there something I can work on? |
I think these don't appear anymore since we enforced all the deprecations in the 1.x series so closing. Can reopen if these appear again |
I have checked that this issue has not already been reported.
I have confirmed this bug exists on the latest version of pandas.
I have confirmed this bug exists on the master branch of pandas.
Reproducible Example
Running
./ci/code_checks.sh docstrings
results in many warnings:Issue Description
We shouldn't have any unintentional warnings in our docstrings. These examples should be updated to not use the deprecated behavior.
Unfortunately, we don't have the line numbers from the warnings. To find the problematic docstring, we probably just need to do a text search for something like
Int64Index(
Expected Behavior
No warnings.
Installed Versions
Replace this line with the output of pd.show_versions()
The text was updated successfully, but these errors were encountered: