Opened 8 years ago

Closed 3 years 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, 8 years ago

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

comment:2 by Sergey Fedoseev, 8 years ago

Description: modified (diff)

comment:3 by Claude Paroz, 8 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, 8 years ago

Triage Stage: Unreviewed → Accepted

Accepting the problem, even if the solution is undecided.

comment:5 by Mariusz Felisiak, 3 years ago

Owner: Sergey Fedoseev removed
Status: assigned → new

comment:6 by Marcelo Galigniana, 3 years 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 3 years ago by Marcelo Galigniana (previous) (diff)

comment:7 by Claude Paroz, 3 years ago

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

comment:8 by Marcelo Galigniana, 3 years ago

Owner: set to Marcelo Galigniana
Status: new → assigned

comment:9 by Marcelo Galigniana, 3 years ago

Has patch: set

comment:10 by Mariusz Felisiak, 3 years ago

Triage Stage: Accepted → Ready for checkin

comment:11 by Mariusz Felisiak <felisiak.mariusz@…>, 3 years ago

Resolution: → fixed
Status: assigned → closed

In 76af8613:

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

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