Thanks to visit codestin.com
Credit goes to github.com

Skip to content

improve docstring of Axes.plot_date #10246

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 3 commits into from
Jan 15, 2018

Conversation

timhoffm
Copy link
Member

PR Summary

As part of #10148: Docstring update for Axes.plot_date and part of matplotlib.dates.

@@ -20,24 +41,22 @@
732403, whereas using the Gregorian calendar via the datetime
module we find::

In [31]:date(2006,4,1).toordinal() - date(1,1,1).toordinal()
Out[31]:732401
In [1]: date(2006,4,1).toordinal() - date(1,1,1).toordinal()
Copy link
Contributor

Choose a reason for hiding this comment

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

may as well make this pep8-compliant (with spaces after commas)?

Copy link
Member

@dstansby dstansby left a comment

Choose a reason for hiding this comment

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

👍 feel free to merge if the note can't live in the notes section.

:class:`matplotlib.dates.AutoDateFormatter` (if the tick
formatter is not already set to a
:class:`matplotlib.dates.DateFormatter` instance).
Note: If you are using custom date tickers and formatters, it may be
Copy link
Member

Choose a reason for hiding this comment

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

Can this go in the notes section below?

@dstansby dstansby added this to the v2.1.2 milestone Jan 14, 2018
@jklymak
Copy link
Member

jklymak commented Jan 15, 2018

Push committed the minor change, though @timhoffm let me know if that ruins your workflow, and I won't do again. Sorry for not asking first - I saw "dstansby" and my mind got confused.

@jklymak jklymak merged commit 1dfa196 into matplotlib:master Jan 15, 2018
@lumberbot-app
Copy link

lumberbot-app bot commented Jan 15, 2018

There seem to be a conflict, please backport manually

tacaswell pushed a commit that referenced this pull request Jan 17, 2018
improve docstring of Axes.plot_date

Conflicts:
	lib/matplotlib/dates.py
          - both branches changed docstring, kept master version
@tacaswell
Copy link
Member

backported to v2.1.x as as 6d9eab7

@timhoffm Sorry, I missed that you had opened a PR to do this as well!

tacaswell added a commit that referenced this pull request Jan 18, 2018
Backport of PR #10246: Improve docstring of Axes.plot_date
@timhoffm timhoffm deleted the axes-doc-plot_date branch January 19, 2018 03:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants