-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DOC: Add reference to frequency strings #13632
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 reference to frequency strings #13632
Conversation
sahildua2305
commented
Jul 12, 2016
•
edited
Loading
edited
- closes The documentation around frequency strings is unclear #13160
@@ -312,6 +312,9 @@ class Window(_Window): | |||
* ``gaussian`` (needs std) | |||
* ``general_gaussian`` (needs power, width) | |||
* ``slepian`` (needs width). | |||
|
|||
To know more about the frequency strings (offset aliases), please visit this | |||
link - http://pandas.pydata.org/pandas-docs/stable/timeseries.html#offset-aliases. |
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.
see how links are done, look at core/frame.py/to_latex
.
To learn more about .....
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.
Thanks! :)
lots of places where this can be added. asfreq, resample, Period, do a search. Once we decide on the text you can update. |
into an instance of pandas ``DateOffset``, which represents a regular | ||
frequency increment. Specific offset logic like "month", "business day", or | ||
"one hour" is represented in its various subclasses. | ||
frequencies by passing in :ref: `frequency strings <timeseries.offset_aliases>` |
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.
no space after the :ref:
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.
Thanks!
Current coverage is 84.38%@@ master #13632 diff @@
==========================================
Files 138 142 +4
Lines 51095 51223 +128
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
+ Hits 43093 43224 +131
+ Misses 8002 7999 -3
Partials 0 0
|
@jreback please confirm the text once so that I can find and add the same at other places as well. |
@@ -271,6 +271,9 @@ def rolling_count(arg, window, **kwargs): | |||
The `freq` keyword is used to conform time series data to a specified | |||
frequency by resampling the data. This is done with the default parameters | |||
of :meth:`~pandas.Series.resample` (i.e. using the `mean`). | |||
|
|||
To learn more about the frequency strings, please see `this link |
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.
these are all deprecated, but ok I guess.
@jorisvandenbossche lgtm. |
@sahildua2305 Thanks a lot! |