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

vector_layers for raster tilesets #67

Open
pnorman opened this issue Aug 23, 2022 · 1 comment
Open

vector_layers for raster tilesets #67

pnorman opened this issue Aug 23, 2022 · 1 comment

Comments

@pnorman
Copy link
Contributor

pnorman commented Aug 23, 2022

The current text of 3.0.0 has a couple of problems for raster layers

Note: When describinng a set of raster tiles or other tile format that does not have a "layers" concept (i.e. "format": "jpeg"), the vector_layers key is not required.

There's a few issues that should be changed for 3.0.1

  1. Typo in "describinng"
  2. use of "i.e." In a case where there is an example, it should be e.g., not i.e., as the latter says jpeg is the only raster tile format
  3. Referencing format. Previous versions of the spec had a format key, but that is no longer defined in the spec, so it's unclear to users what "format": "jpeg" means
  4. Making it optional. The text earlier in the section says vector_layers is required, but this changes that. Instead of sorting that out, I recommend making it optional. For vector tiles with a layer concept, it's not always necessary to know what all the layers are in order to make use of the tiles.
@pnorman
Copy link
Contributor Author

pnorman commented Aug 23, 2022

"implemenntation" is another typo, which makes me think someone had a sticky "n" key.

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

1 participant