You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It would make sense, but the tricky part are the default methods of Bounds:
contains: feature.contains(otherFeature) is ambiguous -- it isn't clear that this check only applies to the bboxes of the features
intersects: feature.intersects(otherFeature) is also ambiguous, it returns true if only the bboxes intersect, but user might expect a geometry intersection test
Maybe keep Bounds simple and implement these as static methods in Box?
The text was updated successfully, but these errors were encountered:
It would make sense, but the tricky part are the default methods of
Bounds
:contains
:feature.contains(otherFeature)
is ambiguous -- it isn't clear that this check only applies to the bboxes of the featuresintersects
:feature.intersects(otherFeature)
is also ambiguous, it returnstrue
if only the bboxes intersect, but user might expect a geometry intersection testMaybe keep
Bounds
simple and implement these as static methods inBox
?The text was updated successfully, but these errors were encountered: