locationtech / locationtech/jts
Expose GeometryOverlay `isOverlayNG` flag
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 2.2k
- Forks
- 475
- Avg merge
- 14d 10h
- Merged PRs (30d)
- 1
Description
The GeometryOverlay class has a flag for switching overlay engine between the original algorithm and OverlayNG.
However this flag is only configurable via the system property "jts.overlay" , which is rather awkward since one cannot change it during runtime and it won't even work if the class happens to be loaded before the user wants to set it (I've had to resort to reflection since System.setProperty("jts-overlay", "ng") is not working).
I think it should be exposed in some way, making the class public and either the setOverlayImpl() method public or the isOverlayNG flag public for instance.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the GeometryOverlay class and reading how the jts.overlay system property selects the overlay engine. Determine which public API best supports changing the engine at runtime, then verify that the selected implementation changes as expected without relying on reflection.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- computer-graphics
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100