Skip to content

Add geospatial function line_interpolate_point#13612

Merged
mbasmanova merged 1 commit intoprestodb:masterfrom
jagill:point-on-line
Nov 1, 2019
Merged

Add geospatial function line_interpolate_point#13612
mbasmanova merged 1 commit intoprestodb:masterfrom
jagill:point-on-line

Conversation

@jagill
Copy link
Contributor

@jagill jagill commented Oct 27, 2019

One of our most requested functions. Given a linestring and a double
between 0 and 1, give the point at that fractional distance along the
line.

== RELEASE NOTES ==

General Changes
-----------------
* Add geospatial function line_interpolate_point

Copy link
Contributor

@mbasmanova mbasmanova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider including function name in the error message. Otherwise, it might be challenging for the user to figure out which part of a complex query is causing this error.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: consider introducing a helper function assertLineInterpolatePoint("LINESTRING EMPTY", 0.5, "POINT_EMPTY")

@mbasmanova mbasmanova self-assigned this Oct 31, 2019
One of our most requested functions.  Given a linestring and a double
between 0 and 1, give the point at that fractional distance along the
line.
Sign up for free to 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.

4 participants