Opened 14 years ago

Closed 9 years ago

#15165 closed Bug (fixed)

GeoQuerySet.perimeter returns invalid results on geometry fields with geodetic coordinate systems

Reported by: jbronn Owned by: Tim Graham <timograham@…>
Component: GIS Version: 1.2
Severity: Normal Keywords: perimeter
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

The method GeoQuerySet.perimeter returns Distance objects with erroneous values when called on a geometry field that has a geodetic coordinate system (e.g., srid=4326). The values in the returned distance objects are really perimeter measurements in degrees of latitude/longitude, rather than meters.

Here's an example using the models and data from the GeoDjango tutorial:

>>> from world.models import WorldBorders
>>> us = WorldBorders.objects.perimeter().get(name='United States')
>>> print us.perimeter
771.493229562 m

Obviously, the perimeter of the United States is greater than a kilometer. To fix, the helper method GeoQuerySet._distance_attribute needs to be modified to raise an exception or return a raw numeric value when attempting to calculate perimeters on geodetic coordinate systems.

Change History (8)

comment:1 by jbronn, 14 years ago

Owner: changed from nobody to jbronn
Status: newassigned

comment:2 by Russell Keith-Magee, 14 years ago

Triage Stage: UnreviewedAccepted

comment:3 by Łukasz Rekucki, 13 years ago

Severity: Normal
Type: Bug

comment:4 by Aymeric Augustin, 13 years ago

UI/UX: unset

Change UI/UX from NULL to False.

comment:5 by Aymeric Augustin, 13 years ago

Easy pickings: unset

Change Easy pickings from NULL to False.

comment:6 by Claude Paroz, 9 years ago

Has patch: set
Owner: jbronn removed
Status: assignednew

comment:7 by Tim Graham, 9 years ago

Patch needs improvement: set

A resolution for the test failure on Oracle is needed.

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

Owner: set to Tim Graham <timograham@…>
Resolution: fixed
Status: newclosed

In 00cb9e13:

Fixed #15165 -- Prevented wrong results with perimeter on geodetic fields.

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