Matt Amos

Thanks!

The scaling factor used to [normalise the normals](https://github.com/mapzen/joerd/blob/master/joerd/output/normal.py#L149-L157) needs to be fixed for mercator projection. The reason it's there in the first place is so that the halving of the...

Hi! > Why this choice of value (5)? It used to be possible for the code which made the geometry valid to insert points at non-integer coordinates. This meant that...

I did a highly unscientific single test case, but I think it's fairly representative of the whole; take NED13 `n32w090`, which is native `Float32` data type: - 359MB - Original...

@nvkelso, what do you think about the precision / file size trade-off? We could, of course, make a separate format for `Int16` and `Float32` GeoTiffs?

I think it's worth noting that the *tests* pass OK, and the segfault happens when it tries to run the tests a second time for *coverage*. That being said, I...

Hi! Thanks for getting in touch. Looks like we're still stuck with test failures, although I'm not seeing the segfault any more, we've now got a protobuf-related problem (from https://github.com/tilezen/mapbox-vector-tile/pull/108#issue-233208179):...

Good idea! I tried that and [pushed it up to a branch](https://github.com/tilezen/mapbox-vector-tile/compare/zerebubuth/no-relative-imports), but unfortunately it's [still failing](https://travis-ci.org/tilezen/mapbox-vector-tile/builds/505671412). Please take a look at the branch - perhaps there's something I missed...

Yup, sorry about that! The decoder outputs a data structure which is [pretty close to the internal structure of an MVT tile](https://github.com/tilezen/mapbox-vector-tile/tree/46629e478935954fc2dcbd8d17d005546b8d31a0#decoding), and wouldn't be usable directly as GeoJSON. I...

Is that just a matter of inserting: ``` python tif_ds.SetMetadataItem('AREA_OR_POINT', 'Area') ``` after the [datasource is created](https://github.com/mapzen/joerd/blob/master/joerd/output/tiff.py#L57)?