googlefonts / OswaldFont

repo for the Oswald font family
SIL Open Font License 1.1
71 stars 22 forks source link

Added build and generated VFs #12

Closed m4rc1e closed 5 years ago

m4rc1e commented 5 years ago

cc @davelab6

m4rc1e commented 5 years ago

FB report for the new VF font.

Fontbakery report

Fontbakery version: 0.6.4

[6] Oswald-VF.ttf
:fire: FAIL: Font has old ttfautohint applied? * [com.google.fonts/check/056](https://github.com/googlefonts/fontbakery/search?q={checkid}) * :fire: **FAIL** Failed to parse ttfautohint version values: installed = '1.8.2'; used_in_font = '1.8.1.43-b0c9' [code: parse-error]
:warning: WARN: Checking OS/2 achVendID. * [com.google.fonts/check/018](https://github.com/googlefonts/fontbakery/search?q={checkid}) * :warning: **WARN** OS/2 VendorID value 'newt' is not a known registered id. You should set it to your own 4 character code, and register that code with Microsoft at https://www.microsoft.com/typography/links/vendorlist.aspx [code: unknown]
:warning: WARN: Stricter unitsPerEm criteria for Google Fonts. * [com.google.fonts/check/116](https://github.com/googlefonts/fontbakery/search?q={checkid}) * :warning: **WARN** Even though unitsPerEm (1000) in this font is reasonable. It is strongly advised to consider changing it to 2000, since it will liely improve the quality of Variable Fonts by avoiding excessive rounding of coordinates on interpolations.
:warning: WARN: Check if each glyph has the recommended amount of contours. * [com.google.fonts/check/153](https://github.com/googlefonts/fontbakery/search?q={checkid}) * :warning: **WARN** This check inspects the glyph outlines and detects the total number of contours in each of them. The expected values are infered from the typical ammounts of contours observed in a large collection of reference font families. The divergences listed below may simply indicate a significantly different design on some of your glyphs. On the other hand, some of these may flag actual bugs in the font such as glyphs mapped to an incorrect codepoint. Please consider reviewing the design and codepoint assignment of these to make sure they are correct. The following glyphs do not have the recommended number of contours: Glyph name: g Contours detected: 4 Expected: 2 or 3 Glyph name: lira Contours detected: 3 Expected: 1 Glyph name: uni20AD Contours detected: 3 Expected: 1 Glyph name: uni20AE Contours detected: 3 Expected: 1 Glyph name: uni20B5 Contours detected: 3 Expected: 1 or 2 Glyph name: uni20B4 Contours detected: 3 Expected: 1 or 2 Glyph name: aogonek Contours detected: 3 Expected: 2 Glyph name: eogonek Contours detected: 3 Expected: 2 Glyph name: gcircumflex Contours detected: 5 Expected: 3 or 4 Glyph name: gbreve Contours detected: 5 Expected: 3 or 4 Glyph name: gdotaccent Contours detected: 5 Expected: 3 or 4 Glyph name: uni0123 Contours detected: 5 Expected: 3 or 4 Glyph name: Tbar Contours detected: 2 Expected: 1 Glyph name: tbar Contours detected: 2 Expected: 1 Glyph name: Uogonek Contours detected: 2 Expected: 1 Glyph name: uogonek Contours detected: 2 Expected: 1 Glyph name: ohorn Contours detected: 3 Expected: 2 Glyph name: Uhorn Contours detected: 2 Expected: 1 Glyph name: uhorn Contours detected: 2 Expected: 1 Glyph name: uni01E4 Contours detected: 2 Expected: 1 Glyph name: uni01E5 Contours detected: 4 Expected: 2 Glyph name: gcaron Contours detected: 5 Expected: 3 or 4 Glyph name: uni01EA Contours detected: 3 Expected: 2 Glyph name: uni01EB Contours detected: 3 Expected: 2 Glyph name: uni01F5 Contours detected: 5 Expected: 3 Glyph name: uni0452 Contours detected: 2 Expected: 1 Glyph name: uni045B Contours detected: 2 Expected: 1 Glyph name: uni0462 Contours detected: 3 Expected: 2 Glyph name: uni0463 Contours detected: 3 Expected: 2 Glyph name: uni046A Contours detected: 3 Expected: 2 Glyph name: uni046B Contours detected: 3 Expected: 2 Glyph name: uni048C Contours detected: 3 Expected: 2 Glyph name: uni048D Contours detected: 3 Expected: 2 Glyph name: uni048E Contours detected: 3 Expected: 1 or 2 Glyph name: uni048F Contours detected: 3 Expected: 2 Glyph name: uni0492 Contours detected: 2 Expected: 1 Glyph name: uni0493 Contours detected: 2 Expected: 1 Glyph name: uni0496 Contours detected: 3 Expected: 1 or 2 Glyph name: uni049A Contours detected: 3 Expected: 1 or 2 Glyph name: uni049C Contours detected: 2 Expected: 1 Glyph name: uni049D Contours detected: 2 Expected: 1 Glyph name: uni049E Contours detected: 2 Expected: 1 Glyph name: uni04A0 Contours detected: 2 Expected: 1 Glyph name: uni04A1 Contours detected: 2 Expected: 1 Glyph name: uni04A2 Contours detected: 3 Expected: 1 or 2 Glyph name: uni04A4 Contours detected: 2 Expected: 1 Glyph name: uni04A8 Contours detected: 1 Expected: 2 Glyph name: uni04A9 Contours detected: 1 Expected: 2 Glyph name: uni04AC Contours detected: 3 Expected: 1 or 2 Glyph name: Ustraitstrokecy Contours detected: 2 Expected: 1 Glyph name: ustraitstrokecy Contours detected: 2 Expected: 1 Glyph name: uni04B2 Contours detected: 3 Expected: 1 or 2 Glyph name: uni04B6 Contours detected: 3 Expected: 1 or 2 Glyph name: uni04B8 Contours detected: 2 Expected: 1 Glyph name: uni04B9 Contours detected: 2 Expected: 1 Glyph name: uni04BC Contours detected: 3 Expected: 2 Glyph name: uni04BD Contours detected: 3 Expected: 2 Glyph name: uni04BE Contours detected: 6 Expected: 2 or 3 Glyph name: uni04BF Contours detected: 6 Expected: 2 or 3 Glyph name: uni04CB Contours detected: 3 Expected: 1 or 2 Glyph name: uni04F6 Contours detected: 3 Expected: 1 or 2 Glyph name: uni04FE Contours detected: 2 Expected: 1 Glyph name: uni04FF Contours detected: 2 Expected: 1 Glyph name: uni0524 Contours detected: 3 Expected: 1 Glyph name: uni0525 Contours detected: 2 Expected: 1 Glyph name: uni0526 Contours detected: 3 Expected: 1 Glyph name: uni0527 Contours detected: 2 Expected: 1 Glyph name: uni052E Contours detected: 3 Expected: 1 Glyph name: uni052F Contours detected: 2 Expected: 1 Glyph name: uni2113 Contours detected: 1 Expected: 2 Glyph name: uni1EDB Contours detected: 4 Expected: 3 Glyph name: uni1EDD Contours detected: 4 Expected: 3 Glyph name: uni1EDF Contours detected: 4 Expected: 3 Glyph name: uni1EE1 Contours detected: 4 Expected: 3 Glyph name: uni1EE3 Contours detected: 4 Expected: 3 Glyph name: uni1EE8 Contours detected: 3 Expected: 2 Glyph name: uni1EE9 Contours detected: 3 Expected: 2 Glyph name: uni1EEA Contours detected: 3 Expected: 2 Glyph name: uni1EEB Contours detected: 3 Expected: 2 Glyph name: uni1EEC Contours detected: 3 Expected: 2 Glyph name: uni1EED Contours detected: 3 Expected: 2 Glyph name: uni1EEE Contours detected: 3 Expected: 2 Glyph name: uni1EEF Contours detected: 3 Expected: 2 Glyph name: uni1EF0 Contours detected: 3 Expected: 2 Glyph name: uni1EF1 Contours detected: 3 Expected: 2 Glyph name: uni20B9 Contours detected: 2 Expected: 1 Glyph name: uni20BA Contours detected: 3 Expected: 1 Glyph name: uni20BC Contours detected: 2 Expected: 1 Glyph name: uni20BD Contours detected: 3 Expected: 2
:warning: WARN: Is there kerning info for non-ligated sequences? * [com.google.fonts/check/065](https://github.com/googlefonts/fontbakery/search?q={checkid}) * :warning: **WARN** GPOS table lacks kerning info for the following non-ligated sequences: - f + f - f + i - i + f - f + l - l + f - i + l [code: lacks-kern-info]
:warning: WARN: Are there caret positions declared for every ligature? * [com.google.fonts/check/064](https://github.com/googlefonts/fontbakery/search?q={checkid}) * :warning: **WARN** This font lacks caret position values for ligature glyphs on its GDEF table. [code: lacks-caret-pos]

Summary

:broken_heart: ERROR :fire: FAIL :warning: WARN :zzz: SKIP :information_source: INFO :bread: PASS
0 1 5 63 7 62
0% 1% 4% 46% 5% 45%

Note: The following loglevels were omitted in this report:

I'll get you diff images,

davelab6 commented 5 years ago

lmk when to merge

m4rc1e commented 5 years ago

All good sir you can merge, images in https://github.com/google/fonts/pull/1814#issuecomment-454800876.

Diff images are around 60mb. Shall I google drive these?

davelab6 commented 5 years ago

Diff images are around 60mb. Shall I google drive these?

Yes