Merge pull request #33 from wiseodd/type-42

Avoid pdf fonttype 3 in the generated files

Author: wiseodd

.github/workflows/pytest.yml

@@ -13,6 +13,8 @@ jobs:
         run: uv python install 3.11
       - name: Install project dependencies (including dev deps)
         run: uv sync
+      - name: Install poppler-utils (for pdffonts)
+        run: sudo apt-get install -y poppler-utils
       - name: Run tests
         run: uv run pytest tests --cov --cov-report=xml
       - name: Upload results to Codecov

README.md

@@ -1,4 +1,4 @@
-# Agustinus' _Very_ Opiniated Publication-Ready Plotting Library
+# A _Very_ Opiniated Publication-Ready Plotting Library
 
 <div align="center">
 
@@ -18,7 +18,7 @@ advocate for the latter ([proof](https://agustinus.kristia.de/blog/plotting/)).
 
 > [!IMPORTANT]
 > Here's what I use now for all my publication needs. This library is designed to be
-> _*very*_ opiniated. Beauty is in the eye of the beholder. Also, it is _very_ simple,
+> **very** opiniated. Beauty is in the eye of the beholder. Also, it is _very_ simple,
 > just a single file and that's it.
 
 > [!NOTE]

makefile

@@ -11,3 +11,9 @@ mypy:
 lint:
 	make ruff
 	make mypy
+
+build:
+	uv build
+
+publish:
+	uv publish

pub_ready_plots/pub_ready_plots.py

@@ -133,6 +133,7 @@ def get_mpl_rcParams(
         "ytick.major.size": format.tick_size,
         "xtick.major.width": format.tick_width,
         "ytick.major.width": format.tick_width,
+        "pdf.fonttype": 42,
     }
 
     w = width_frac * (format.col_width if single_col else format.text_width)

tests/test_core_func.py

@@ -1,6 +1,9 @@
+import subprocess
+from pathlib import Path
 from typing import Sequence
 
 import matplotlib as plt
+import numpy as np
 import pytest
 
 import pub_ready_plots as prp
@@ -75,3 +78,23 @@ def test_double_column(layout: prp.Layout) -> None:
             _ = prp.get_mpl_rcParams(
                 width_frac=1, height_frac=0.1, layout=layout, single_col=True
             )
+
+
+@pytest.mark.parametrize("layout", LAYOUTS)
+def test_no_type3_fonts(layout: prp.Layout, tmp_path: Path) -> None:
+    FNAME = "temp.pdf"
+
+    with prp.get_context(
+        width_frac=0.5,
+        height_frac=0.15,
+        layout=layout,
+    ) as (fig, ax):
+        x = np.linspace(-1, 1)
+        fx = np.sin(x)
+        ax.plot(x, fx)
+        fig.savefig(FNAME)
+
+    result = subprocess.run(
+        ["pdffonts", str(FNAME)], capture_output=True, text=True, check=True
+    )
+    assert "Type 3" not in result.stdout