Skip to content

Trying to partial-instance CFF2 #4710

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Trying to partial-instance CFF2 #4710

wants to merge 1 commit into from

Conversation

behdad
Copy link
Member

@behdad behdad commented May 11, 2024

Honestly I'm lost. But I'll find my way.

@behdad behdad force-pushed the cff2-instancing-try2 branch from 669d4b5 to 0a2feeb Compare May 11, 2024 14:51
@behdad
Copy link
Member Author

behdad commented May 11, 2024

What needs to be done from what I currently understand:

  • The private-dict always processes blends. We need a specialization for the subset case that stores them, like the cs-interpretter does.

  • Both need to gain support for partial instancing. Not sure were to put all the region expensions. Perhaps the CFF2 subset accel.

  • The varstore needs to be pruned of unused regions.

@behdad
Copy link
Member Author

behdad commented May 13, 2024

Humm. Maybe easier to do in fonttools first...

@behdad
Copy link
Member Author

behdad commented May 14, 2024

I actually implemented this in FontTools. Wasn't too bad there. I hope we can use some of the same design here.

@qxliu76 is this something you can take on?

@garretrieger

@behdad
Copy link
Member Author

behdad commented May 23, 2024

I actually implemented this in FontTools. Wasn't too bad there. I hope we can use some of the same design here.

fonttools/fonttools#3506

@behdad behdad force-pushed the cff2-instancing-try2 branch from 0a2feeb to 1ce4f2b Compare March 10, 2025 09:18
@behdad
Copy link
Member Author

behdad commented Mar 10, 2025

I rebased this but don't think I'd have time any time soon to get back to this. :(

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.

1 participant