Skip to content

Feature: Terrain Preview and simple Elevation Query#1425

Merged
acalcutt merged 14 commits intomaptiler:masterfrom
okimiko:terrain-and-elevation
Jan 2, 2025
Merged

Feature: Terrain Preview and simple Elevation Query#1425
acalcutt merged 14 commits intomaptiler:masterfrom
okimiko:terrain-and-elevation

Conversation

@okimiko
Copy link
Copy Markdown
Contributor

@okimiko okimiko commented Dec 17, 2024

This PR will add

  • a preview for terrain data (in addition to the current raw data preview)
    • highlight color is based on the encoding
  • a simple elevation api based on served terrain data
    • includes a link on the index page
    • for details, see endpoints.rst

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.

okimiko and others added 2 commits December 21, 2024 21:39
Co-authored-by: Andrew Calcutt <acalcutt@techidiots.net>
okimiko and others added 3 commits December 22, 2024 08:07
@okimiko
Copy link
Copy Markdown
Contributor Author

okimiko commented Dec 22, 2024

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 :)

@acalcutt acalcutt merged commit a2bc9f0 into maptiler:master Jan 2, 2025
@acalcutt
Copy link
Copy Markdown
Collaborator

I was noticing how long some of the decimal point values are. I wonder if we should add a limit to the number of decimal places, maybe something like 7

@okimiko
Copy link
Copy Markdown
Contributor Author

okimiko commented Jan 27, 2025

@acalcutt: That should be no problem, but at least for lat/long (float-url) the values are the same as the input, which are taken from leaflet click event. For x/y the values are calculated. Or should we only limit the display in the preview?

@acalcutt
Copy link
Copy Markdown
Collaborator

acalcutt commented Jan 31, 2025

I mean more when it was displayed, like
image

But possibly the output should have a limit also

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants