Skip to content

Lazy-load fonts for pdf creation.#556

Merged
bohning merged 3 commits intomainfrom
lazy-load-fonts
Feb 25, 2026
Merged

Lazy-load fonts for pdf creation.#556
bohning merged 3 commits intomainfrom
lazy-load-fonts

Conversation

@bohning
Copy link
Owner

@bohning bohning commented Feb 24, 2026

Like this?

@randompersona1
Copy link
Collaborator

The healthcheck should be modified to load the fonts - that way, we verify that the font files are present in both wheels and bundles.

@bohning
Copy link
Owner Author

bohning commented Feb 25, 2026

The healthcheck should be modified to load the fonts - that way, we verify that the font files are present in both wheels and bundles.

You mean just add

from usdb_syncer.gui.pdf import _ensure_fonts_registered
_ensure_fonts_registered()

to _run_healthcheck()?

Copy link
Collaborator

@RumovZ RumovZ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick implementation!

@bohning bohning merged commit 0b0a753 into main Feb 25, 2026
4 checks passed
@bohning bohning deleted the lazy-load-fonts branch February 25, 2026 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants