Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork8.1k
Skip svg tex test if tex is not installed#5770
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
jenshnielsen commentedDec 31, 2015
Unfortunatly this doesn't seem to fix the issue |
mdboom commentedDec 31, 2015
Does the baseline image need to be updated? Beyond that, I wonder if OrdereredDict isn't enough for the issue if things still come in from TeX in a non-deterministic order. We may need to forcibly sort... |
jenshnielsen commentedDec 31, 2015
Yes thats true, Not completely sure what happens but this is only an issue on 2.x so presumably ti requires something which is not backported to 2.x |
Borrowed frommatplotlib#5677 which is not backported to 2.x
jenshnielsen commentedDec 31, 2015
mdboom commentedDec 31, 2015
The exception changed in the rewrite of the C++ extensions last year, but none of that was in the test suite (since we didn't have TeX+SVG tests until this one). I say we just apply diff --git a/lib/matplotlib/textpath.py b/lib/matplotlib/textpath.pyindex c063c3a..b694619 100644--- a/lib/matplotlib/textpath.py+++ b/lib/matplotlib/textpath.py@@ -345,7 +345,7 @@ class TextToPath(object): 1094995778)]: try: font.select_charmap(charmap_code)- except ValueError:+ except (ValueError, RuntimeError): pass else: break here, and not backport all of#5677 -- it turns out there are few issues with it and we may end up reverting it in the long run. |
jenshnielsen commentedDec 31, 2015
mdboom commentedDec 31, 2015
I agree on all points. Not sure if it matters too much how we do it -- we can always make another PR against v1.5.x with16698e8 cherry-picked in it. |
Skip svg tex test if tex is not installed
This should fix failure on the 2.x branch after the merge of#5766
This should be merged asap to make the 2.x branch pass again