Fix issue with PGF backend not falling back to default LaTeX fonts. #22111
Conversation
|
The main question is if it should fall back to LaTeX or matplotlibfonts in general. Now it will always fall back to LaTeX rather than matplotlib (which doesn't seem to work properly on my Windows machine anyway...). |
|
I updated it, but now (I do not recall the previous status), we instead get a different warning message (and no plot shown): |
|
Good catch, the error makes sense, we need something like diff --git i/lib/matplotlib/backends/backend_pgf.py w/lib/matplotlib/backends/backend_pgf.py
index 255714a7ea..a8f2587b9e 100644
--- i/lib/matplotlib/backends/backend_pgf.py
+++ w/lib/matplotlib/backends/backend_pgf.py
@@ -151,16 +151,27 @@ def _escape_and_apply_props(s, prop):
"""
commands = []
- families = {"serif": r"\rmfamily", "sans": r"\sffamily",
- "sans-serif": r"\sffamily", "monospace": r"\ttfamily"}
+ standard_families = {
+ "serif": r"\rmfamily",
+ "sans": r"\sffamily", # Doesn't match FontManager, perhaps deprecate?
+ "sans-serif": r"\sffamily",
+ "sans serif": r"\sffamily",
+ "monospace": r"\ttfamily",
+ }
family = prop.get_family()[0]
- if family in families:
- commands.append(families[family])
- elif (any(font.name == family for font in fm.fontManager.ttflist)
- and mpl.rcParams["pgf.texsystem"] != "pdflatex"):
- commands.append(r"\setmainfont{%s}\rmfamily" % family)
+ if family in standard_families:
+ commands.append(standard_families[family])
else:
- _log.warning("Ignoring unknown font: %s", family)
+ if family in fm.font_family_aliases: # Only cursive & fantasy remain.
+ path = fm.findfont(family)
+ names = [fe.name for fe in fm.fontManager.ttflist
+ if fe.fname == path]
+ family = names[0]
+ if (any(font.name == family for font in fm.fontManager.ttflist)
+ and mpl.rcParams["pgf.texsystem"] != "pdflatex"):
+ commands.append(r"\setmainfont{%s}\rmfamily" % family)
+ else:
+ _log.warning("Ignoring unknown font: %s", family)
size = prop.get_size_in_points()
commands.append(r"\fontsize{%f}{%f}" % (size, size * 1.2))(Perhaps nicer would be to have a variant of findfont which directly returns the correct FontEntry rather than just the path, as this avoids having to try to get back to the FontEntry from the path which may fail if we don't have a bijection. Also once we have both the family and the path, we can additionally explicitly set the font path for tex similarly to how it's already done in _get_preamble.) |
| # with backslashes. | ||
| # 2) The dirname needs to include a separator. | ||
| path = pathlib.Path(fm.findfont(family, | ||
| fallback_to_default=False)) |
There was a problem hiding this comment.
minor nit: only the findfont call should be wrapped in the try... except; this makes it clearer that you're catching failures of findfont and not e.g. findfont returning an invalid value for Path(). (Plus this avoids the awkward linebreak.)
PR Summary
Fixes #20850
Closes #20945
Seems like a minor fix, so not "What's new" required?
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).