Opened 8 years ago

Closed 8 years ago

#25722 closed New feature (fixed)

add support for `GEOSCovers`

Reported by: Sergey Fedoseev Owned by: Sergey Fedoseev
Component: GIS Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Support for GEOSCoveredBy also could be added, but GEOSCoveredBy(g1, g2) is the same as GEOSCovers(g2, g1), so it seems that there is no point in adding it.

Change History (6)

comment:1 by Sergey Fedoseev, 8 years ago

Owner: changed from nobody to Sergey Fedoseev
Status: new → assigned

comment:2 by Claude Paroz, 8 years ago

Triage Stage: Unreviewed → Accepted
Version: 1.8 → master

comment:3 by Sergey Fedoseev, 8 years ago

Has patch: set

PR -- ​https://github.com/django/django/pull/5627

comment:4 by Tim Graham, 8 years ago

Patch needs improvement: set

Left some minor comments for improvement.

comment:5 by Sergey Fedoseev, 8 years ago

Patch needs improvement: unset

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

Resolution: → fixed
Status: assigned → closed

In ccc8f67b:

Fixed #25722 -- Added the GEOSGeometry.covers() method.

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