googlefonts / fontc

Where in we pursue oxidizing (context: https://github.com/googlefonts/oxidize) fontmake.
Apache License 2.0
62 stars 10 forks source link

Match hb-draw more precisely #784

Closed rsheeter closed 2 months ago

rsheeter commented 2 months ago

As per https://github.com/googlefonts/fontations/pull/880#pullrequestreview-2029561181, "we should make sure we actually offer HB compatibility beyond 2 decimal places if we're going to promise it."

There are a couple of adjustments needed to match HB more precisely. For context #880 doesn't do the floating point math identically so while it matches at low precision (e.g. 2 decimal places for icons) there will be differences in output when the full floating point values are compared. Adjust the following:

For advance and deltas contrast with the sleipnir code in https://github.com/googlefonts/sleipnir/pull/15/commits/cae1b6320734a78bc3c4f115ec2a8569cd2d2ebf.

Deferring to this issue as #880 appears to be sufficient for icon use where a minute floating point difference doesn't really matter.

rsheeter commented 2 months ago

off by one ... repo. Moving to fontations.