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

/collections/{collectionId}/tiles instead of /collections/{collectionId}/coverage/tiles ? #174

Open
jerstlouis opened this issue Sep 14, 2023 · 4 comments

Comments

@jerstlouis
Copy link
Member

jerstlouis commented Sep 14, 2023

Having second thoughts about the decision to use /collections/{collectionId}/coverage/tiles as the recommended or required path for the Coverage Tiles requirements class in light of:

  • Several people had expressed some preference for the shorter / simpler path which would also be consistent with "Vector Tiles" in terms of providing raw data tiles
  • We recently got rid of the other /coverage/* sub-resources (domainset and rangetype) in favor of information in /collections/{collectionId} and /collections/{collectionId}/schema
  • Part of the original reasoning for this was decision to distinguish from vector tiles and allow for both 'vector tiles' and 'coverage tiles' for the same collection. The common case where data is both vector and coverage are point clouds. However, I think vector tiles and coverage tiles in this context are mostly equivalent, and this may possibly also apply in other contexts where data is both vector and coverage. The distinction could then possibly be the media type used to negotiate for example LAS vs. a GeoJSON multi-point feature collection. GeoJSON and CoverageJSON also have their own media type that can distinguish them from application/json for CIS JSON.

One potential reason to stick to the current path is that the published OGC API - Tiles does include a mention of /collections/{collectionId}/coverage/tiles in Table 8 of Section 6.5. However, there is a very clear note below that table saying that these exact paths are ONLY examples and are NOT required by this Standard. Potentially, we could do a minor corrigendum to update this.

Thoughts / opinions?

cc @joanma747

@joanma747
Copy link

If there is no longer "coverage" in the domainset and rangetype, I agree with removing the "/coverage/" also with coverage tiles to converge in a single query model.

@cnreediii
Copy link
Contributor

@jerstlouis Sorry to sound like an old record, but "coverage tiles"??? Also, as a standards organization we should not use the term "vector tiles" unless we are specifically referring to MapBox vector tiles. In the latter case, to be very clear to the reader we should say "MapBox vector tiles (MVT)" to be explicitly clear. AS you know, "tile" is an abstract concept. When tiling is physically implemented, then content is tiled against that tiling scheme. The content could be any data type that has spatial context. Could be Tweets! So would be say "tweet tiles"? Semantically (and conceptually), the OGC should be consistently using terms such as tiled geometry, tiled vector data, tiled gridded data, and so on. Note that in CDB 2.0, this is the approach that I have (hopefully) been consistently using.

Obviously there is common usage (vector tiles) but in a standards organization we should be using standards speak (tiled vector data).

Just had to speak up :-)

@jerstlouis
Copy link
Member Author

2024-01-24 SWG: We will go ahead and make the change from /coverage/tiles to simply /tiles

@m-mohr
Copy link

m-mohr commented Nov 12, 2024

Just use links to expose the endpoints, then you don't need to worry about API endpoint paths.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

4 participants