Skip to content

Datetime units not listed in pd.to_timedelta docstring #23248

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
JoshuaC3 opened this issue Oct 20, 2018 · 4 comments · Fixed by #23259
Closed

Datetime units not listed in pd.to_timedelta docstring #23248

JoshuaC3 opened this issue Oct 20, 2018 · 4 comments · Fixed by #23259
Labels
Docs Timedelta Timedelta data type
Milestone

Comments

@JoshuaC3
Copy link

JoshuaC3 commented Oct 20, 2018

Problem description

I wanted to do a month timedelta but upon checking the pd.to_timedelta docstring I gave up on this as an option because there was no 'M' option. I then tried many other round-the-houses methods until someone answered this question on stackoverflow.

This is not updated in the most recent docs but would be good to include, I believe.

Expected Output

To see full list of (or reference to) all units that can be used.

Output of pd.show_versions()

23.4

@ryankarlos
Copy link
Contributor

Hi, I can try adding this in

@ryankarlos
Copy link
Contributor

Ive just made a PR, any feedback would be welcome from reviewers. Thanks !

@JoshuaC3
Copy link
Author

Satisfies my this issue so I am happy. Cheers.

@mroeschke
Copy link
Member

'M' in particular is slated for depreciation; however, expanding all the other units that can be used with to_timedelta in the docstring would be a good addition.

xref #16344

@mroeschke mroeschke added Docs Timedelta Timedelta data type labels Oct 21, 2018
@jreback jreback added this to the 0.24.0 milestone Nov 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docs Timedelta Timedelta data type
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants