Jan 5, 2022 · Jul 10, 2022
diff --git a/examples/userdemo/pgf_fonts.py b/examples/userdemo/pgf_fonts.py
 =========
 """

 import matplotlib as mpl
 import matplotlib.pyplot as plt
 mpl.use("pgf")
 plt.rcParams.update({
    "font.family": "serif",
    # Use LaTeX default serif font.
diff --git a/lib/matplotlib/backends/backend_pgf.py b/lib/matplotlib/backends/backend_pgf.py
            families = ["serif", "sans\\-serif", "monospace"]
            commands = ["setmainfont", "setsansfont", "setmonofont"]
            for family, command in zip(families, commands):
                # 1) Forward slashes also work on Windows, so don't mess with
                # backslashes.  2) The dirname needs to include a separator.
                path = pathlib.Path(fm.findfont(family))
                preamble.append(r"\%s{%s}[Path=\detokenize{%s/}]" % (
                    command, path.name, path.parent.as_posix()))
                try:
                    # 1) Forward slashes also work on Windows, so do not mess
                    # with backslashes.
                    # 2) The dirname needs to include a separator.
                    path = pathlib.Path(fm.findfont(family,
                                                    fallback_to_default=False))
                except ValueError:
                    # Use default LaTeX font instead
                    pass
                else:
                    preamble.append(r"\%s{%s}[Path=\detokenize{%s}]" % (
                        command, path.name, path.parent.as_posix() + "/"))
    return "\n".join(preamble)
Original file line number	Diff line number	Diff line change
Expand Up		@@ -4,7 +4,9 @@
		=========
		"""

		import matplotlib as mpl
		import matplotlib.pyplot as plt
		mpl.use("pgf")
oscargus marked this conversation as resolved. Show resolvedHide resolved
		plt.rcParams.update({
		"font.family": "serif",
		# Use LaTeX default serif font.
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -65,11 +65,18 @@ def _get_preamble():
		families = ["serif", "sans\\-serif", "monospace"]
		commands = ["setmainfont", "setsansfont", "setmonofont"]
		for family, command in zip(families, commands):
		# 1) Forward slashes also work on Windows, so don't mess with
		# backslashes. 2) The dirname needs to include a separator.
		path = pathlib.Path(fm.findfont(family))
		preamble.append(r"\%s{%s}[Path=\detokenize{%s/}]" % (
		command, path.name, path.parent.as_posix()))
		try:
		# 1) Forward slashes also work on Windows, so do not mess
		# with backslashes.
		# 2) The dirname needs to include a separator.
		path = pathlib.Path(fm.findfont(family,
		fallback_to_default=False))
Copy link Contributor anntzerJul 10, 2022 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. 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.)
		except ValueError:
		# Use default LaTeX font instead
		pass
		else:
		preamble.append(r"\%s{%s}[Path=\detokenize{%s}]" % (
		command, path.name, path.parent.as_posix() + "/"))
		return "\n".join(preamble)


Expand Down