Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

topoint function #48

Closed
pnorman opened this issue Nov 18, 2016 · 2 comments
Closed

topoint function #48

pnorman opened this issue Nov 18, 2016 · 2 comments

Comments

@pnorman
Copy link
Collaborator

pnorman commented Nov 18, 2016

housenumber_centroid.sql contains

UPDATE osm_housenumber_point SET geometry=topoint(geometry)
WHERE ST_GeometryType(geometry) <> 'ST_Point';

topoint is not defined anywhere.

@ImreSamu
Copy link
Collaborator

topoint is not defined anywhere.

as I know it is used by

  • in docker-compose run --rm import-sql step
  • source code : [ https://github.com/openmaptiles/import-sql ]

the topoint() = https://github.com/mapbox/postgis-vt-util#topoint
( based on https://github.com/openmaptiles/import-sql/blob/master/Dockerfile )

side note:

  • we need to create a good developer documentations ...
  • the new quickstart.sh will contain a little information about this, ( not in the master branch yet )
echo "-------------------------------------------------------------------------------------"
echo "====> : Start SQL postprocessing:  ./build/tileset.sql -> PostgreSQL "
echo "      : Source code: https://github.com/openmaptiles/import-sql "
docker-compose run --rm import-sql

probably I can extend with this information ...

@lukasmartinelli
Copy link
Contributor

as I know it is used by

Yes but in this use case it would probably more clear to just use ST_PointOnSurface. Other functions from vt-util are LineLabel and z.

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

No branches or pull requests

4 participants