Fix issue with tex-encoding on non-Unicode platforms#23033
Fix issue with tex-encoding on non-Unicode platforms#23033oscargus merged 1 commit intomatplotlib:mainfrom
Conversation
618e86d to
3ad3dd8
Compare
|
Yes, turned out that I didn't check the error message well enough. It raises a |
|
I'm a bit confused; it should ultimately raise a RuntimeError, not a CalledProcessError, if you do a fig.canvas.draw(), doesn't it? |
|
Here is the error message: so it seems like the test sees (Somehow one would really like to test that there is not a |
|
The output above is what I get in Spyder (that automatically does drawing). Calling fig.canvas.draw() in an ipython console gives the same error message. |
|
No, the actually thrown exception is the lower one? ("CalledProcessError was the direct cause of the following exception...") Try e.g. wrapping the whole thing in a try... except... to check what exception type you can catch. |
3ad3dd8 to
dbc8b7a
Compare
|
Ahh, you are right. I was confused by the test failure and thought that it raised a different exception, rather than no exception. |
dbc8b7a to
8c13417
Compare
8c13417 to
23a60c0
Compare
|
Rebased. |
|
Depending on the merge order of this and #23045 I will update the one that is not merged first. |
| import pytest | ||
|
|
||
|
|
||
| needs_usetex = pytest.mark.skipif( |
There was a problem hiding this comment.
@oscargus I think you can now import this from matplotlib.testing._markers.
Feel free to self-merge after fixing this.
23a60c0 to
0edf0f1
Compare
0edf0f1 to
b81546b
Compare
PR Summary
Closes #23019
Thanks @anntzer for the rapid fix.
PR Checklist
Tests and Styling
pytestpasses).flake8-docstringsand runflake8 --docstring-convention=all).Documentation
doc/users/next_whats_new/(follow instructions in README.rst there).doc/api/next_api_changes/(follow instructions in README.rst there).