Skip to content

DOC: Add context to pd.to_timedelta examples #11789

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

Merged
merged 1 commit into from
Dec 9, 2015
Merged

DOC: Add context to pd.to_timedelta examples #11789

merged 1 commit into from
Dec 9, 2015

Conversation

proinsias
Copy link
Contributor

Update the reStructuredText tutorial on timedeltas to include
context, similar to edits suggested for #11785.

Also, add a space after various commas (PEP8), and periods/colons
at the end of sentences for consistency.

@jreback jreback added the Docs label Dec 7, 2015
@jreback jreback added this to the 0.18.0 milestone Dec 7, 2015
to_timedelta(np.arange(5),unit='s')
to_timedelta(np.arange(5),unit='d')

# Parsing a list or array of strings
Copy link
Contributor

Choose a reason for hiding this comment

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

I actually like having these as separate ipython blocks, with the text in-between (as opposed to a comment).

Update the reStructuredText tutorial on timedeltas to include
context, similar to edits suggested for #11785.

Also, add a space after various commas (PEP8), and periods/colons
at the end of sentences for consistency.
@proinsias
Copy link
Contributor Author

I've made these changes, and updated the commit.

jreback added a commit that referenced this pull request Dec 9, 2015
DOC: Add context to pd.to_timedelta examples
@jreback jreback merged commit b1caf01 into pandas-dev:master Dec 9, 2015
@jreback
Copy link
Contributor

jreback commented Dec 9, 2015

@proinsias thanks!

@proinsias proinsias deleted the doc_timedeltas branch December 15, 2015 04:49
@proinsias proinsias restored the doc_timedeltas branch December 15, 2015 04:50
@proinsias proinsias deleted the doc_timedeltas branch December 15, 2015 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants