Skip overlapping contour check for variable fonts

[HinTak]

Skip overlapping contour check for variable fonts

[HinTak]

Thanks for the investigation. Yes, I filed this issue because overlapping contours are common/expected of variable fonts. I'll get to it one day, when I finish the other more urgent/important issues (like getting building with visual studio to work again, after Microsoft removed it at the beginning...)

[thundernixon]

When I run a FontBakery test on a font I'm mastering, I get several errors that I am confused by, and which I can't find documentation for:

• ℹ️ INFO MS-FonVal: Unable to perform test due to previously detected errors DETAILS:
  • Glyph index 1 Test: ValidateSimpContMisor
  • Glyph index 10 Test: ValidateSimpContMisor
  • Glyph index 26 Test: ValidateSimpContMisor
  • Glyph index 38 Test: ValidateSimpContMisor
  • Glyph index 44 Test: ValidateSimpContMisor
  • Glyph index 69 Test: ValidateSimpContMisor
  • Glyph index 76 Test: ValidateSimpContMisor
  • Glyph index 91 Test: ValidateSimpContMisor
  • Glyph index 105 Test: ValidateSimpContMisor
  • Glyph index 107 Test: ValidateSimpContMisor
  • NOTE: 37 other similar results were hidden!

The naming of this error (and what I can understand in the fontval code) indicates that the message is about Contour Misorientation. However, the glyphs listed have similar contour directions to other glyphs.

However, each of the glyphs listed here has overlaps, while those excluded tend to be individual shapes. Is it safe to assume that this stems from the same issue, or is this something separate?

If helpful, here's the font triggering this check, the TTX xml, and the Font Bakery report:
https://github.com/thundernixon/Libre-Caslon/tree/2622969efd19f148fa17536345b085441fb723c9/dist/2018-11-07-15_29

[HinTak]

I don't know - I think you should run FontVal bare and read the report. At least messages are in the right order. I am not sure whether test cannot continue because of contour mis-orientation, or that the mis-orientation test was not performed because of previous (other) problems. I do not want to look at FB code to find out.

I'll have a look at the font itself at some point in time, but not right away.

[thundernixon]

Thanks for your comments and the recommendation, @HinTak. I ran it in the Mac GUI from Georg Seifert, and found that some of the outlines did have incorrect path directions.

Once I fixed that, it appeared that anything with overlap triggered this error/info combo:

| E1111 | Intersecting contours |
| I1111 | Unable to perform test due to previously detected errors |

So, I just wanted to report this as something else that probably could be upgraded when Variable Font updates are supported.