Skip to content
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

Filter warnings in rcparams test (and others) #3244

Merged
merged 11 commits into from
Aug 26, 2014

Conversation

jenshnielsen
Copy link
Member

These are warnings from matplotlib which should be there so we just filter them.

Update:
This branch filters various warnings raised in different parts of the test suite. They are all warnings that are genuine and should be raised. We thus only filter them at the stage of the tests to get a clean test output and make it easier to spot real problematic warnings.

@tacaswell tacaswell added this to the v1.4.x milestone Jul 12, 2014
@jenshnielsen
Copy link
Member Author

Not sure why we are getting out of memory errors.

@tacaswell
Copy link
Member

The out of memory error is (usually) a transient.

These are genuine too so we filter them
These are warnings that are raised because tightlayout does not work
fully automatic on gridspec. However, the point of the test is doing
this with some manual help so this works as expected.
@jenshnielsen jenshnielsen changed the title Filter warnings in rcparams test. Filter warnings in rcparams test (and others) Jul 26, 2014
tacaswell added a commit that referenced this pull request Aug 26, 2014
Filter warnings in rcparams test (and others)
@tacaswell tacaswell merged commit 82ac087 into matplotlib:master Aug 26, 2014
@tacaswell
Copy link
Member

cherry-picked to 1.4.x as 1cadc90

tacaswell added a commit that referenced this pull request Aug 26, 2014
Filter warnings in rcparams test (and others)
@jenshnielsen jenshnielsen deleted the rcwarnings branch August 28, 2014 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants