Skip to content

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

Closed

Conversation

sahildua2305
Copy link
Contributor

@sahildua2305 sahildua2305 commented Jul 12, 2016

@sahildua2305 sahildua2305 changed the title Add reference to frequency strings; fixes #13160 DOC: Add reference to frequency strings; fixes #13160 Jul 12, 2016
@sahildua2305 sahildua2305 changed the title DOC: Add reference to frequency strings; fixes #13160 DOC: Add reference to frequency strings Jul 12, 2016
@jreback jreback added Docs Datetime Datetime data dtype labels Jul 13, 2016
@@ -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.
Copy link
Contributor

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 .....

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! :)

@jreback
Copy link
Contributor

jreback commented Jul 13, 2016

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>`
Copy link
Member

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:

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@codecov-io
Copy link

codecov-io commented Jul 13, 2016

Current coverage is 84.38%

Merging #13632 into master will increase coverage by 0.04%

@@             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          

Powered by Codecov. Last updated by 7c357d2...b45ae21

@sahildua2305
Copy link
Contributor Author

@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
Copy link
Contributor

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.

@jreback jreback added this to the 0.19.0 milestone Jul 20, 2016
@jreback
Copy link
Contributor

jreback commented Jul 20, 2016

@jorisvandenbossche lgtm.

@jorisvandenbossche
Copy link
Member

@sahildua2305 Thanks a lot!

@sahildua2305 sahildua2305 deleted the frequency-strings-fix branch July 21, 2016 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Datetime Datetime data dtype Docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The documentation around frequency strings is unclear
4 participants