Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#25728 closed Bug (fixed)

description of `GEOSGeometry.contains` is incorrect

Reported by: Sergey Fedoseev Owned by: Sergey Fedoseev
Component: Documentation Version: 1.8
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Documentation says that GEOSGeometry.contains

Returns True if GEOSGeometry.within() is False.

this is incorrect:

In [23]: pol1 = GEOSGeometry('POLYGON((0 0, 0 4, 4 4, 4 0, 0 0))')

In [24]: pol2 = GEOSGeometry('POLYGON((6 6, 6 9, 9 9, 9 6, 6 6))')

In [25]: pol1.within(pol2)
Out[25]: False

In [26]: pol1.contains(pol2)
Out[26]: False

Change History (6)

comment:1 by Sergey Fedoseev, 8 years ago

Owner: changed from nobody to Sergey Fedoseev
Status: newassigned

comment:2 by Sergey Fedoseev, 8 years ago

Has patch: set

comment:3 by Tim Graham, 8 years ago

Triage Stage: UnreviewedReady for checkin

comment:4 by Tim Graham <timograham@…>, 8 years ago

Resolution: fixed
Status: assignedclosed

In 34669ea6:

Fixed #25728 -- Fixed description of GEOSGeometry.contains().

comment:5 by Tim Graham <timograham@…>, 8 years ago

In 8d87447:

[1.9.x] Fixed #25728 -- Fixed description of GEOSGeometry.contains().

Backport of 34669ea6b88cefb6e6f7176c71b7947234fa50b4 from master

comment:6 by Tim Graham <timograham@…>, 8 years ago

In 1deaafe:

[1.8.x] Fixed #25728 -- Fixed description of GEOSGeometry.contains().

Backport of 34669ea6b88cefb6e6f7176c71b7947234fa50b4 from master

Note: See TracTickets for help on using tickets.
Back to Top