Opened 7 years ago

Closed 22 months ago

#27550 closed New feature (fixed)

add version of GEOSGeometry.normalize() that returns new geometry

Reported by: Sergey Fedoseev Owned by: Marcelo Galigniana
Component: GIS Version: dev
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 (last modified by Sergey Fedoseev)

GEOSGeometry.normalize() normalizes geometry in-place. In some cases it would be more convenient to have a cloned normalized geometry for the code conciseness.
We could add clone kwarg to this method similarly to GEOSGeometry.transform() or we could add the normalized property.

Change History (11)

comment:1 by Sergey Fedoseev, 7 years ago

Owner: changed from nobody to Sergey Fedoseev
Status: newassigned

comment:2 by Sergey Fedoseev, 7 years ago

Description: modified (diff)

comment:3 by Claude Paroz, 7 years ago

I'm not sure. Could you audit other methods and tell us if the use case could be applied to other methods? Then we might better recommend using the clone() method before normalizing the geometry.

comment:4 by Tim Graham, 7 years ago

Triage Stage: UnreviewedAccepted

Accepting the problem, even if the solution is undecided.

comment:5 by Mariusz Felisiak, 2 years ago

Owner: Sergey Fedoseev removed
Status: assignednew

comment:6 by Marcelo Galigniana, 23 months ago

Could I go for the "add clone kwarg to this method similarly to GEOSGeometry.transform()" approach?

Or should I post in the developer email group to define the best solution?

Last edited 23 months ago by Marcelo Galigniana (previous) (diff)

comment:7 by Claude Paroz, 23 months ago

Yes, I think having a patch with the clone argument should be welcome.

comment:8 by Marcelo Galigniana, 23 months ago

Owner: set to Marcelo Galigniana
Status: newassigned

comment:9 by Marcelo Galigniana, 23 months ago

Has patch: set

comment:10 by Mariusz Felisiak, 22 months ago

Triage Stage: AcceptedReady for checkin

comment:11 by Mariusz Felisiak <felisiak.mariusz@…>, 22 months ago

Resolution: fixed
Status: assignedclosed

In 76af8613:

Fixed #27550 -- Allowed GEOSGeometry.normalize() to return a normalized clone.

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