[ttLib] removeOverlaps pathops fails with glyphSet argument
See original GitHub issueI’m trying to make instances from a variable TTF and it works fine without --remove-overlaps
, but with it
File "/fontTools/ttLib/removeOverlaps.py", line 33, in skPathFromGlyph
pathPen = path.getPen(glyphSet=glyphSet)
File "src/python/pathops/_pathops.pyx", line 153, in pathops._pathops.Path.getPen
TypeError: getPen() got an unexpected keyword argument 'glyphSet'
Looks like there are no tests in removeOverlaps_test.py for getPen with glyphSet
Issue Analytics
- State:
- Created 10 months ago
- Comments:6 (2 by maintainers)
Top Results From Across the Web
fonttools/removeOverlaps.py at main
Simplify TrueType glyphs by merging overlapping contours/components. Requires https://github.com/fonttools/skia-pathops. """ import itertools.
Read more >fonttools [python-library]
[ttGlyphPen] Fixed typing annotation of TTGlyphPen glyphSet parameter (#2315). ... [removeOverlaps] Retry pathops.simplify after rounding path coordinates ...
Read more >FontTools
Lib/fontTools/ttLib/removeOverlaps.py ... [ttGlyphPen] Fixed typing annotation of TTGlyphPen glyphSet parameter ... It requires the skia-pathops module.
Read more >FontTools fonttools Issues
[ttLib] removeOverlaps pathops fails with glyphSet argument · `pyftsubset` gives missing glyphs exception when using single hyphens for CLI args · [ttx] Non-ascii ......
Read more >fonttools from fonttools
Lib/fontTools/ttLib/removeOverlaps.py. Simplify TrueType glyphs by merging overlapping contours and components. skia-pathops: Python bindings for the Skia ...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
Sorry, I forgot to mention you also need to install pathops. pip install skia-pathops
Ah, probably this venv is just old. I need to trash a few of these and make new venvs.