geo icon indicating copy to clipboard operation
geo copied to clipboard

Adds Within trait and fixes more Contains edge cases

Open michaelkirk opened this issue 2 years ago • 1 comments

  • [x] I agree to follow the project's code of conduct.
  • [x] I added an entry to CHANGES.md if knowledge of this change could be valuable to users.

~~Based on #880, so review that first.~~ Merged! This is ready for review!

The new Within trait is implemented in terms of Contains.

Running the "within" tests from the JTS suite exposed some further inconsistencies with how we're handling line string edge cases.

michaelkirk avatar Jul 28 '22 19:07 michaelkirk

Now that https://github.com/georust/geo/pull/880 has been merged, this is ready for review!

michaelkirk avatar Aug 01 '22 22:08 michaelkirk

bors r=frewsxcv

michaelkirk avatar Aug 13 '22 02:08 michaelkirk

Build succeeded:

bors[bot] avatar Aug 13 '22 03:08 bors[bot]