-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DOC: add hidden IntervalIndex.str to prevent doc build warnings #16050
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: add hidden IntervalIndex.str to prevent doc build warnings #16050
Conversation
For the constructor, we could add it as It is just that adding the full IntervalIndex add another > 100 pages to the API docs (making the doc build longer and the docs heavier), which are mostly not needed |
Codecov Report
@@ Coverage Diff @@
## master #16050 +/- ##
=======================================
Coverage 90.79% 90.79%
=======================================
Files 156 156
Lines 50534 50534
=======================================
Hits 45883 45883
Misses 4651 4651
Continue to review full report at Codecov.
|
yes certainly fine to remove interval from docs |
what if we take off all of the index autogenerated ones (but leave the specifically named method), eg. remove the first second but leave the 2nd? @jorisvandenbossche @TomAugspurger
|
@jreback yes, I think that is a good idea. |
@jorisvandenbossche do you want me to take a look at this before the release? Just so I understand correctly.
There is an |
What I was thinking was to make a separate class template for those classes for which we don't want all the members listed. But didn't think it fully through yet. Another option is for now just add a manual autodoc page for IntervalIndex that is not autogenerated and does not included the overview of all methods/attributes (but not sure whether this prevents it being overwritten by the automatic generation step) If you have time to look at this, certainly feel free! |
@jorisvandenbossche I tried a few things to not autodoc everything on Index subclasses without success. Maybe just merge this as is for now. It'll remove the warnings at the least. |
@jorisvandenbossche newer versions of numpydoc have a |
I am not sure we can use it in our case in general, as eg for DataFrame and Series we want all ones included, and not those that are inherited from NDFrame (but didn't try it, so don't know whether it makes sense). |
Oh yeah, of course. I was only thinking of public things that inherit from other public things :/ I'll see how hard it is to extend numpydoc to give us more control over what goes into autosummary. Not before the release though. Merge this in the meantime? |
Closed by alternative PR #16221 |
@TomAugspurger @jreback This (should) fixes the warning related to the IntervalIndex.str, but, I was thinking we should maybe remove the full
IntervalIndex
class from the API docs?I mean the class docstring, not the docstrings for the specific methods that are unique to the IntervalIndex