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

update matrix demo to use sources_to_targets #206

Open
kevinkreiser opened this issue Aug 3, 2017 · 0 comments
Open

update matrix demo to use sources_to_targets #206

kevinkreiser opened this issue Aug 3, 2017 · 0 comments

Comments

@kevinkreiser
Copy link
Member

kevinkreiser commented Aug 3, 2017

all the matrix end points are special cases of sources_to_targets. we should update the demo to just use sources_to_targets as, its easier to understand and the preferred method from within the service code as well. this will require some ui changes to the demo as we need to some how choose whether we are click points for sources or for targets but otherwise thats about the only change that needs to be made save for a small request format change to accommodate two lists of locations.

this was suggested in valhalla/valhalla#870

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant