Skip to content

Geometry cast of pcpoint with optional srid, z and m values - #139

Closed
mbredif wants to merge 1 commit into
pgpointcloud:masterfrom
mbredif:xyzm_point
Closed

Geometry cast of pcpoint with optional srid, z and m values#139
mbredif wants to merge 1 commit into
pgpointcloud:masterfrom
mbredif:xyzm_point

Conversation

@mbredif

Copy link
Copy Markdown
Contributor

This is a first PR to better support use cases that are not only 2D with XY dimensions but also 3 or 4D (Z and/or T/M), replacing the bloated #104 PR.

  • position caching in schema of X and Y dimensions is extended to Z and M positions, looking up for Z/H/Height and M/T/Time/GPSTime dimension names.

  • the setting of x/y_positions with the xydim code was redundant with pc_schema_check_xy

  • pc_point_to_geometry_wkb now returns z and m values if available.

@mbredif

Copy link
Copy Markdown
ContributorAuthor

@strk, @pramsey, what do you think about this PR ?

@elemoine

Copy link
Copy Markdown
Contributor

FWIW, I am +1 on this patch.

@strk

strk commented Feb 16, 2017 via email

Copy link
Copy Markdown
Member

@mbredif

Copy link
Copy Markdown
ContributorAuthor

merged by 165d9ea.
Thanks @elemoine and @strk !

@mbredifmbredif closed this Feb 16, 2017
@mbredif
mbredif deleted the xyzm_point branch February 16, 2017 09:29
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@mbredif@elemoine@strk