Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revamp badge measurement #27

Open
idg10 opened this issue Jan 9, 2024 · 0 comments
Open

Revamp badge measurement #27

idg10 opened this issue Jan 9, 2024 · 0 comments

Comments

@idg10
Copy link
Contributor

idg10 commented Jan 9, 2024

There are some problems with the badge measurement code:

  1. because it uses the system font support, it ends up loading the entire font directory just to look at the metrics for 1 font, which is clobbering our cold start time
  2. I think this gives us a System.Drawing dependency which will stop us from running on Linux in the long run
  3. the imaging library we were using to read this stuff might not longer be available in open source form

The best way to fix the cold start problems would be to extract the metrics for the font we use at build time and code-gen them into an embedded ReadOnlySpan<...> type array. This would absolutely minimize the startup costs. But there may be simpler ways that get us most of the way there.

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

No branches or pull requests

1 participant