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

Support topojson? #20

Open
josiekre opened this issue Jun 30, 2017 · 2 comments
Open

Support topojson? #20

josiekre opened this issue Jun 30, 2017 · 2 comments

Comments

@josiekre
Copy link

Have you thought about supporting topojson? It would mean using pre-specified topology from topojson. I use topojson a lot, and I'd prefer to calculate the shortest paths based on the topology building algo there.

@perliedman
Copy link
Owner

That does indeed make a lot of sense. I seem to remember looking into TopoJSON when I started the project, but don't remember why I decided against it.

In principle, I don't see any issue with using TopoJSON, all the information GeoJSON Path Finder needs should definitely be there. The main issue I see is with TopoJSON's alternative coordinate format (IIRC it uses some form of compressed coordinates): GeoJSON Path Finder needs to access the edges' coordinates directly in some places, and would need to be adapted to TopoJSON's alternative format.

This is not something I'm likely to take on myself at this point, but I would be happy to help out answering questions if this is something you would be interested in adding.

@josiekre
Copy link
Author

josiekre commented Jul 3, 2017

Thanks for the reply. I'll think about extending your work for topojson if I cannot find a solution already. I've only begun digging around.

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