-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
TST : force re-building of font-cache #3074
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
Conversation
- there should be a better way to do this, this will get the tests passing again.
@efiring @mdboom Can one of you take a look at this soon? I would like to get this merged so travis is useful again. This snippet illustrates the problem:
There probably should be a check on |
Thanks for all the investigation on this -- ideally, changing font params should not require rebuilding of the cache -- the cache should be purely informational. I'll have a look to see if there's a way around that, and this additional information is really helpful in honing in on what might be going wrong. |
@tacaswell: I've restarted the Travis build here (and will probably do it a few more times) just to make sure your test fix does the trick... If it does, that really diagnoses the problem quite well and paves the way forward for a more complete solution. Honestly, I've spent a long time on this, but this is the closest to a diagnosis we've come, so thanks a lot. |
TST : force re-building of font-cache
I've merged this, as you say, to get tests passing. What is needed as a real fix, though, does not have to be triggering a rebuild of the font manager upon rcParams update, but merely clearing of the lookup caches inside the font manager. |
tests passing again.
as suggested by @efiring in #3073
closes #3065