Skip to content
This repository has been archived by the owner on Aug 9, 2023. It is now read-only.

font.info.unitsPerEm seems to be required #49

Open
typemytype opened this issue Sep 20, 2016 · 3 comments
Open

font.info.unitsPerEm seems to be required #49

typemytype opened this issue Sep 20, 2016 · 3 comments

Comments

@typemytype
Copy link

https://github.com/googlei18n/cu2qu/blob/master/Lib/cu2qu/ufo.py#L224

would be nice to have some fallback value when the value is not set

@anthrotype
Copy link
Member

What should the default unitsPerEm value be?
I guess 1000.
Maybe the UFO spec should say what the fallback value is when the value is not set.

@typemytype
Copy link
Author

The default 'DEFAULT_MAX_ERR' makes it confusing...

It should not be calculated from UPM if it is not set

On Wednesday, 21 September 2016, Cosimo Lupo notifications@github.com
wrote:

What should the default unitsPerEm value be?
I guess 1000.
Maybe the UFO spec should say what the fallback value is when the value is
not set.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#49 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABIp1uFBLWYbTqDNQmZJhFmgeAStfhj4ks5qsFdjgaJpZM4KCI8q
.

gr Frederik
www.typemytype.com

@anthrotype
Copy link
Member

Yeah, that's a bit confusing.

Also, I think we should disallow using a relative approximation error to convert compatibly fonts having different UPEMs, since OpenType Variations must have the same UPEM (if I am not mistaken).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants