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

TST : fix font-priority test #3073

Closed
wants to merge 1 commit into from

Conversation

tacaswell
Copy link
Member

For reasons I don't understand wrapping the test in a cleanup decorator
seems to make it pass consistently.

Closes #3065

For reasons I don't understand wrapping the test in a cleanup decorator
seems to make it pass consistently.

Closes matplotlib#3065
@efiring
Copy link
Member

efiring commented May 19, 2014

@tacaswell, it looks like we are running into a 10000-line log file limit in Travis.

@tacaswell
Copy link
Member Author

If you hit the raw log button in the upper right it will give you a text file (which loads faster anyway). And this does not actually work. It built correctly twice, but it is still failing.

@efiring
Copy link
Member

efiring commented May 19, 2014

I wonder whether the problem with the test is that the font lookup is cached, so setting the rcParam doesn't change anything unless the cache is deleted first to force a fresh lookup. This caching is normally on a per mpl session basis, but it looks like if anything forces rebuilding of the overall FontManager cache pickle file after lookups have been executed, the cached result of those lookups will also persist.
It seems like a mechanism is needed for clearing the lookup caches whenever a font-related rcParam is changed. Or, include the relevant rcParam settings in the lookup hash.

@tacaswell tacaswell closed this May 19, 2014
@tacaswell tacaswell deleted the font_priority branch September 6, 2014 03:05
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.

font priority bug
2 participants