-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
VIS: register datetime64 in matplotlib units registry (GH8614) #8693
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
VIS: register datetime64 in matplotlib units registry (GH8614) #8693
Conversation
to plot such values as datetimes. This is activated once pandas is imported. In | ||
previous versions, plotting an array of ``datetime64`` values will have resulted | ||
in plotted integer values. To keep the previous behaviour, you can do | ||
``matplotlib.units.registry[np.datetime64] = None`` (:issue:`8614`). |
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.
del matplotlib.units.registry[np.datetime64]
would imho be slightly neater.
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.
yep indeed
0146152
to
932afd8
Compare
Rebased, and will merge when travis is green |
932afd8
to
a87d6ea
Compare
VIS: register datetime64 in matplotlib units registry (GH8614)
@@ -283,8 +293,15 @@ Bug Fixes | |||
|
|||
|
|||
|
|||
<<<<<<< HEAD |
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.
I don't think you meant to merge the conflict markers.
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.
hmm, no not really ..
should have gone something wrong with rebasing
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 for noticing!
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.
Closes #8614