-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DOC: add PeriodIndex.asfreq docstring #10051
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
Conversation
Convert the PeriodIndex to the specified frequency `freq`. | ||
|
||
Parameters | ||
========== |
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.
Can you use ---
instead of ===
for the lines? (to be consistent with the other docstrings)
@jorisvandenbossche thanks. Should be good now I think |
Returns | ||
------- | ||
|
||
new: PeriodIndex with the new frequency |
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 small one :-)
here also a space between new and :
@TomAugspurger added some more .. :-) |
@TomAugspurger can you update? |
@jreback done. Merge when you're ready. |
@TomAugspurger I think you forgot to add your changes |
Seems to be good now. |
I realize after we merge index-repr prob need to fix s bunch of doc strings :( |
e.g. will be.
|
Indeed. A search for |
DOC: add PeriodIndex.asfreq docstring
Part of #6384
I guess PeriodIndex don't show up in the API docs at all. Not sure if I'll get to any more.