8000 Rename PolygonHullSimplifier by dr-jts · Pull Request #880 · locationtech/jts · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Rename PolygonHullSimplifier #880

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jun 2, 2022
Merged

Rename PolygonHullSimplifier #880

merged 1 commit into from
Jun 2, 2022

Conversation

dr-jts
Copy link
Contributor
@dr-jts dr-jts commented Jun 2, 2022

Rename PolygonHull to PolygonHullSimplifier, and move to simplify package.
Reasons for the change are:

  • avoid confusion with the concave hull functionality
  • better reflects the likely usage of outer and inner hulls
  • the parameters for the Polygon Hull algorithm are more similar to existing simplfication parameters than to the concave hull parameters.

Signed-off-by: Martin Davis mtnclimb@gmail.com

@dr-jts
Rename and move PolygonHullSimplifier
13ec3b8 
Signed-off-by: Martin Davis <mtnclimb@gmail.com>
@dr-jts dr-jts self-assigned this Jun 2, 2022
@dr-jts dr-jts merged commit 21c9f0a into locationtech:master Jun 2, 2022
@dr-jts dr-jts deleted the rename-polygonhullsimplifier branch June 2, 2022 20:02
@dr-jts dr-jts mentioned this pull request Jun 2, 2022
Merged
3 tasks
@jodygarnett jodygarnett added this to the 1.19.0 milestone Jun 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@dr-jts dr-jts

Labels
jts-core type-improvement
Projects
None yet
Milestone
1.19.0
Development

Successfully merging this pull request may close these issues.
2 participants
@dr-jts @jodygarnett
0