Skip to content

Conversation

@cookiedan42
Copy link
Contributor

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

Removed necessary enumerate and check in the LineString.contains(Coord) implementation

benched using boundary of point in simple polygon bench

point in simple linestring time:   [2.9182 ns 2.9279 ns 2.9396 ns]
                        change: [-28.259% -27.484% -26.754%] (p = 0.00 < 0.05)
                        Performance has improved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant