Opened 4 years ago

Closed 11 days ago

#31014 closed New feature (fixed)

add FromWKB and FromWKT GIS functions

Reported by: Sergey Fedoseev Owned by: Claude Paroz
Component: GIS Version: dev
Severity: Normal Keywords: wkt, wkb
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


Change History (23)

comment:1 by Sergey Fedoseev, 4 years ago

Owner: changed from nobody to Sergey Fedoseev
Status: newassigned

comment:2 by Mariusz Felisiak, 4 years ago

Triage Stage: UnreviewedAccepted

comment:3 by Nick Pope, 4 years ago

Has patch: set
Keywords: wkt wkb added

PR

I also note that some backends support ST_GeomFromEWKB and ST_GeomFromEWKT. Perhaps it is worth checking out whether it is worth adding these too?

comment:4 by Mariusz Felisiak, 4 years ago

Patch needs improvement: set

comment:5 by Sergey Fedoseev, 22 months ago

Owner: Sergey Fedoseev removed
Status: assignednew

comment:6 by Ondřej Böhm, 18 months ago

Owner: set to Ondřej Böhm
Status: newassigned

comment:8 by Mariusz Felisiak, 15 months ago

Owner: Ondřej Böhm removed
Status: assignednew

comment:9 by Claude Paroz, 15 months ago

Needs someone with Oracle knowledge to finish the patch.

in reply to:  9 comment:10 by Mariusz Felisiak, 15 months ago

Replying to Claude Paroz:

Needs someone with Oracle knowledge to finish the patch.

We can always mark them as unsupported on Oracle for now, and leave the ticket open.

comment:11 by Mariusz Felisiak, 15 months ago

There is also an open question about the support of the srid argument.

comment:12 by Mariusz Felisiak, 15 months ago

IMO, there is no need to support the srid argument. I can prepare a new PR.

Last edited 15 months ago by Mariusz Felisiak (previous) (diff)

comment:13 by Mariusz Felisiak, 15 months ago

Owner: set to Mariusz Felisiak
Patch needs improvement: unset
Status: newassigned

comment:14 by Carlton Gibson, 15 months ago

Triage Stage: AcceptedReady for checkin

comment:15 by GitHub <noreply@…>, 15 months ago

In 552384fa:

Refs #31014 -- Added FromWKB and FromWKT GIS database functions.

Co-authored-by: Ondřej Böhm <ondrej.bohm@…>
Co-authored-by: Mariusz Felisiak <felisiak.mariusz@…>
Co-authored-by: Sergey Fedoseev <fedoseev.sergey@…>

comment:16 by Mariusz Felisiak, 15 months ago

Has patch: unset
Triage Stage: Ready for checkinAccepted

Support for srid is missing.

comment:17 by Mariusz Felisiak, 3 weeks ago

Owner: Mariusz Felisiak removed
Status: assignednew

comment:18 by Claude Paroz, 3 weeks ago

Has patch: set

comment:19 by Mariusz Felisiak, 3 weeks ago

Needs documentation: set
Owner: set to Claude Paroz
Patch needs improvement: set
Status: newassigned

comment:20 by Claude Paroz, 13 days ago

Needs documentation: unset
Patch needs improvement: unset

comment:21 by Mariusz Felisiak, 11 days ago

Triage Stage: AcceptedReady for checkin

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

In 10b31ee:

Refs #31014 -- Added srid argument to FromWKB/FromWKT() GIS functions.

comment:23 by Mariusz Felisiak, 11 days ago

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.
Back to Top