Skip to content

TST: Fixed test_log_margins test - #7453

Merged
efiring merged 1 commit into
matplotlib:masterfrom
Kojoley:fix-test_log_margins
Nov 13, 2016
Merged

efiring merged 1 commit into
matplotlib:masterfrom
Kojoley:fix-test_log_margins

Conversation

@Kojoley

@Kojoley Kojoley commented Nov 13, 2016

Copy link
Copy Markdown
Member

Closes #7452

@Kojoley Kojoley added this to the 2.0.1 (next bug fix release) milestone Nov 13, 2016

@dopplershift dopplershift left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@efiring

efiring commented Nov 13, 2016

Copy link
Copy Markdown
Member

@Kojoley, peripheral note: the Travis failure was the pytest run, after 25 minutes; I've seen this failure more often than not. Is there anything that can be done about it?

@efiring
efiring merged commit 8515322 into matplotlib:master Nov 13, 2016
efiring added a commit that referenced this pull request Nov 13, 2016
TST: Fixed `test_log_margins` test
@efiring

efiring commented Nov 13, 2016

Copy link
Copy Markdown
Member

back-ported to v2.x as e6f2222

@Kojoley

Kojoley commented Nov 14, 2016 •

Copy link
Copy Markdown
Member Author

@efiring I've seen this failure on both nose and pytest previously. Today I have run test_mixedsubplots about 500 times plus the whole test_mplot3d.py about 1500 times and it did not fail a single time. Unfortunately I do not have any idea what it could be and how to reproduce this failure. We have to grab a diff image from travis/appveyor at least to understand what is going on there,

@QuLogic QuLogic modified the milestones: 2.0.1 (next bug fix release), 2.0 (style change major release) Dec 7, 2016
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.

4 participants