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

Add directions API redundancy with mapbox #2

Merged
merged 1 commit into from Aug 24, 2017

Conversation

norydev
Copy link
Owner

@norydev norydev commented Aug 24, 2017

OSRM has been down, triggered this redundancy implementation.

the :mapbox symbol passed to API should probably be more dynamic so that wrapper can be changed without modifying the code

I'm not very happy with mapbox, it's not free nor open source. On the positive side, they probably use OSRM in the back, as it's the same API, so few code change needed

EDIT: see comment in thread

the :mapbox symbol passed to API should probably be more dynamic so that wrapper can be changed without modifying the code

I'm not very happy with mapbox, it's not free nor open source. On the positive side, they probably use OSRM in the back, as it's the same API, so few code change needed
@norydev norydev merged commit 4994d01 into master Aug 24, 2017
@norydev norydev deleted the mapbox-alternative-to-osrm branch August 24, 2017 13:02
@simison
Copy link

simison commented Aug 24, 2017

Mapbox actually is pretty cool company; they publish a lot of their code open source and they even pay for OSRM development currently.

https://wiki.openstreetmap.org/wiki/Open_Source_Routing_Machine#History
https://wiki.openstreetmap.org/wiki/Mapbox

@simison
Copy link

simison commented Aug 24, 2017

Lemme know if you want to use MapBox APIs, you can piggyback on Hitchwiki's account.

@norydev
Copy link
Owner Author

norydev commented Aug 24, 2017

@simison great, I did not know!

I made myself an account on Mapbox anyway, so I'm fine for now.

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

Successfully merging this pull request may close these issues.

None yet

2 participants