Validate every tile - Githubissues

mapbox / mapbox-upload-validate

Validate that files can be uploaded to mapbox.com

ISC License

5 stars 7 forks source link

Validate every tile #58

Closed mapsam closed 8 years ago

mapsam commented 8 years ago

This PR aims to add complete tile validation for serialtiles and mbtiles in preparation for v2 vector tiles. The goal is to:

validate ALL mbtiles and serialtiles tiles via validationstream.js using mapnik.VectorTile.info() and the information it returns (such as version and errors)
make validationstream.js useable outside of mapbox upload validate, specifically for validation in mapbox-tile-copy

cc/ @jakepruitt @flippmoke @springmeyer @GretaCB @rclark @yhahn

jakepruitt commented 8 years ago

Can we close this PR now @mapsam? https://github.com/mapbox/mapbox-tile-copy/pull/72 is performing validation during the copy operation - although I'm not familiar enough with the other use-cases of mapbox-upload-validate to know if we should do this.

mapsam commented 8 years ago

We should definitely close this PR @jakepruitt. That aside, though, do we need to make any updates to this repo? Now that we are upgrading mbtiles to v2, I presume the answer is no.