Opened 5 years ago

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

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

comment:2 by Mariusz Felisiak, 5 years ago

Triage Stage: Unreviewed → Accepted

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, 2 years ago

Owner: Sergey Fedoseev removed
Status: assigned → new

comment:6 by Ondřej Böhm, 2 years ago

Owner: set to Ondřej Böhm
Status: new → assigned

comment:8 by Mariusz Felisiak, 22 months ago

Owner: Ondřej Böhm removed
Status: assigned → new

comment:9 by Claude Paroz, 22 months ago

Needs someone with Oracle knowledge to finish the patch.

in reply to:  9 comment:10 by Mariusz Felisiak, 22 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, 22 months ago

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

comment:12 by Mariusz Felisiak, 22 months ago

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

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

comment:13 by Mariusz Felisiak, 22 months ago

Owner: set to Mariusz Felisiak
Patch needs improvement: unset
Status: new → assigned

​PR

comment:14 by Carlton Gibson, 22 months ago

Triage Stage: Accepted → Ready for checkin

comment:15 by GitHub <noreply@…>, 22 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, 22 months ago

Has patch: unset
Triage Stage: Ready for checkin → Accepted

Support for srid is missing.

comment:17 by Mariusz Felisiak, 8 months ago

Owner: Mariusz Felisiak removed
Status: assigned → new

comment:18 by Claude Paroz, 8 months ago

Has patch: set

​PR

comment:19 by Mariusz Felisiak, 8 months ago

Needs documentation: set
Owner: set to Claude Paroz
Patch needs improvement: set
Status: new → assigned

comment:20 by Claude Paroz, 8 months ago

Needs documentation: unset
Patch needs improvement: unset

comment:21 by Mariusz Felisiak, 8 months ago

Triage Stage: Accepted → Ready for checkin

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

In 10b31ee:

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

comment:23 by Mariusz Felisiak, 8 months ago

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