Skip to content
This repository has been archived by the owner on Dec 10, 2024. It is now read-only.

Fontlayout: Speedup, show progress per polygon #31

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

iopapamanoglou
Copy link
Contributor

Fontlayout: Speedup, show progress per polygon

Description

Reopened from faebryk/faebryk#213

  • Speedup by introducing a convergence threshold and a maximum number of points per polygon
  • Add option for showing progress per polygon
  • Add bezier support
  • Add support for multiline strings in string_to_polygons()

Checklist

Please read and execute the following:

  • My code follows the coding guidelines of this project
  • My PR title is following the contribution guidelines
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (if applicable)
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works (if applicable)
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • I ran Black to format my code

Code of Conduct

By submitting this issue, you agree to follow our Code of Conduct:

Sorry, something went wrong.

IoannisP-ITENG and others added 8 commits July 6, 2024 15:50
Speedup the finding of distributed points in a polygon by introducing a
higher convergence threshold and an absolute limit on the amount of points
per polygon
Also add options for showing progress
…ts_in_polygon

Lowers the default theshold to get more accurate positioning
- Fix moving point inside polygon when COM is outside
Moves the new point inside the polygon again when the calculated
center-of-mass of the point is outside the polygon.
- Fix exception when no points have travelled
- Add check for empty polygon to return empty array
- Fix some tasks not updating
Add support for left-aligned multiline strings
Add get_point_on_bezier_curve()
Adds a resolution parameter to string_to_polygons() which specifies in
how many points a bezier curve in a font is approximated.
Improve placement of characters that have disjoint polygons by iterating
until threshold, then forcing the points to be inside the polygon and
iterating again instead of always forcing the points to be inside the
polygon.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants