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

Implement Mapbox.Geometry #18

Open
wilhelmberg opened this issue Nov 16, 2016 · 5 comments
Open

Implement Mapbox.Geometry #18

wilhelmberg opened this issue Nov 16, 2016 · 5 comments
Assignees

Comments

@wilhelmberg
Copy link
Contributor

wilhelmberg commented Nov 16, 2016

/cc @mapbox/games

Carrying over @springmeyer's comments from #7 (comment)

I've started the C# counterpart of geometry.hpp here: https://github.com/mapbox/GeometryCs

But we won't follow the C++ lead until geometry.hpp has fully found its way into vector_tile.hpp

@wilhelmberg wilhelmberg self-assigned this Nov 16, 2016
@tmpsantos
Copy link

👍

@david-rhodes
Copy link

@brnkhy Did you see this?

Pass only a scaling value to decode geometries into tile coordinates

This sounds relevant to your recent optimizations.

@BergWerkGIS Was this specific piece implemented? Is this similar to what @brnkhy is doing here: https://github.com/mapbox/mapbox-unity-sdk/tree/vector2d?

@brnkhy
Copy link

brnkhy commented Mar 30, 2017

@david-rhodes that's actually what I'm doing as well (scaling parameter thing). Vector tile coordinate system is a little different, I initially thought we have the commands to calculate the geometry which would have taken much longer. Then realized that's actually already done and we have vector tile coordinates which I can simply scale to our tiles etc.

@wilhelmberg
Copy link
Contributor Author

@david-rhodes @brnkhy
The scale parameter exists in the method signature but currently it's not implemented.

@wilhelmberg
Copy link
Contributor Author

wilhelmberg commented Mar 31, 2017

REMOVED comment!
created a new issue dedicated to discussion about scale: #30

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

4 participants