Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fixed simplify to not get stuck in an infinite loop on certain geometries #2830

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

smallsaucepan
Copy link
Member

simplify was getting stuck in an infinite loop when it couldn't reduce the tolerance far enough to generate a valid polygon. This often happened with very small rings e.g. couple of centimetres to a side.

Resolves #1788

Please provide the following when creating a PR:

  • Meaningful title, including the name of the package being modified.
  • Summary of the changes.
  • Heads up if this is a breaking change.
  • Any issues this resolves.
  • Inclusion of your details in the contributors field of package.json - you've earned it! 👏
  • Confirmation you've read the steps for preparing a pull request.

…uce the tolerance far enough to generate a valid polygon. Tidied up docs.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

@turf/simplify gets stuck in an infinite loop on certain geometries
1 participant