8000 Fix matplotlib / pandas 0.21 bug in examples by Cadair · Pull Request #2336 · sunpy/sunpy · GitHub
[go: up one dir, main page]

Skip to content

Fix matplotlib / pandas 0.21 bug in examples #2336

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
Nov 9, 2017

Conversation

Cadair
Copy link
Member
@Cadair Cadair commented Nov 8, 2017

Here we manually register the pandas matplotlib converters so people doing manual plotting with pandas works under pandas 0.21

Here we manually register the pandas matplotlib converters so people
doing manual plotting with pandas works under pandas 0.21
@Cadair Cadair requested review from wafels and nabobalis November 8, 2017 17:47
@pep8speaks
Copy link
pep8speaks commented Nov 8, 2017

Hello @Cadair! Thanks for updating the PR.

Cheers ! There are no PEP8 issues in this Pull Request. 🍻

Comment last updated on November 08, 2017 at 22:04 Hours UTC

@Cadair Cadair added 0.8.x Affects Release An issue/bug that affects a released version (use a version label too) labels Nov 8, 2017
@wafels
Copy link
Member
wafels commented Nov 8, 2017

There is also

pandas-dev/pandas#18153

and the official matplotlib advice

https://matplotlib.org/faq/howto_faq.html#plot-numpy-datetime64-values

Copy link
Member
@wafels wafels left a comment

Choose a reason for hiding this comment

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

@wafels
Copy link
Member
wafels commented Nov 8, 2017

Example fail again: https://github.com/sunpy/sunpy/blob/master/examples/AIA_limb_STEREO.py#L63

Why hasn't this been caught before?

@Cadair
Copy link
Member Author
Cadair commented Nov 8, 2017

@wafels that error is a result of a VSO download error, I have restarted it.

@wafels
Copy link
Member
wafels commented Nov 8, 2017

Try block gets a +1

@Cadair Cadair merged commit 794c38f into sunpy:master Nov 9, 2017
@Cadair
Copy link
Member Author
Cadair commented Nov 9, 2017

@nabobalis this too needs backporting.

@Cadair Cadair deleted the pandas_plotting_fix branch November 9, 2017 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Affects Release An issue/bug that affects a released version (use a version label too)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0