-
Notifications
You must be signed in to change notification settings - Fork 646
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
Feature: Terrain Preview and simple Elevation Query #1425
base: master
Are you sure you want to change the base?
Conversation
Co-authored-by: Andrew Calcutt <[email protected]>
docs/config.rst
Outdated
}, | ||
"terrain": { | ||
"url": "mbtiles://terrain.mbtiles", | ||
"encoding": "mapbox" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we are going to need to find a good spot to put this documentation since it doesn't just apply to mbtiles. maybe a new section that notes this and the information right below. ir might be good to note it only works with pmtiles or mbtiles
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would like the second approach, in general the tileserver handles "data" even if we add some special terrain handling here, I think there is no need to show that in an own section. (This may change, if there will be more features in this section ;-)).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I miss understood your comment (I read it as a change of the code regarding configuration).
I updated the documentation with a dedicated section nested in "data".
Co-authored-by: Andrew Calcutt <[email protected]>
Thanks for all suggestions so far. I did a local build and converted my terrain mbtile source to a pmtile and both are working so far :) |
This PR will add
Both require configured encoding (see config.rst).
In general this adds just some eye candy and an "inspect" function for terrain like the geojson endpoint.