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

Feature Request: Minify attribute names #638

Closed
stdmn opened this issue Aug 31, 2018 · 3 comments
Closed

Feature Request: Minify attribute names #638

stdmn opened this issue Aug 31, 2018 · 3 comments

Comments

@stdmn
Copy link

stdmn commented Aug 31, 2018

@ericfischer It would be great to have a simple way of minifying attribute names and then including their full names in the metadata file.

The general idea would be similar to the -k, --keys attribute of this repo: https://github.com/TNOCS/minify-geojson#readme which turns attribute names like "SQFOOT" to "S" and then provides a map to map them back to their original values.

@e-n-f
Copy link
Contributor

e-n-f commented Aug 31, 2018

I don't think this will ultimately save very much space in the tiles because each attribute name is only stored once per tile. Are you seeing significant savings from shorter names?

@stdmn
Copy link
Author

stdmn commented Sep 1, 2018

Sorry, I didn't realize that. I guess there could be a way to do a similar thing for values where there is a lot of repetition. Not sure if that is already included in the encoding...

@e-n-f
Copy link
Contributor

e-n-f commented Sep 1, 2018

Yes, duplicate values are also encoded as references into a pool of constants. (See the comments in mapbox/vector-tile-spec#121 for the current efforts to make them compress even better.)

@stdmn stdmn closed this as completed Sep 1, 2018
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

2 participants