-
Notifications
You must be signed in to change notification settings - Fork 30
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
OSRM profiles removed #70
Comments
I guess you then have somehow named your osrm base url according to the previous osrm "profiles" routing-py was referencing? honestly, I have no idea how that ever worked before #64 . I realized finally that osrm doesn't have any notion of a profile for its endpoints. the endpoint urls for osrm are always postfixed with e.g. so I'm curious how it could've worked for you before #64 and doesn't work now? can you share an example URL? |
ah, I'm just seeing in the api docs what you mean.. for parameter
that's weird though.. the just seeing that I was wrong: it doesn't have to be can it then be any URL? osrm must know which service to use and where the coordinates are, so e.g. you're right, #64 is not doing the right thing either. I'm wondering if we'd need to let the user have full control over the URL part after (sorry, I edited quite heavily after posting initially) |
So the way we are using it is that we have several OSRM instances running, each has it's own profile (according to lua preprocessing script). Then we use nginx to route the requests by simple url matching. For example I think each OSRM instance just discards the Here is an example from public OSRM instance. The main benefit of profiles was that we only needed one instance of OSRM router from routingpy, that would make requests according to vehicle type. The previous version did the job, so I don't think full url customization is needed. |
yeah right, thanks, don't know why I didn't try myself 😅 I find that whole URL architecture super strange.. but alright then, I'm convinced to revert the PR. and put some more comments in the docstrings so people aren't confused that |
Hi, thanks that great. |
Hi guys!
Thanks for the amazing library. I was wondering for the reasons of OSRM profiles being removed from the library by #64.
According to OSRM api docs there is a concept of "profile" and in fact our team was using routing-py with multi profile OSRM. This is usually achieved by using a proxy, which maps different OSRM instances by url. Using profiles in routing-py router was extremely convenient for us.
Is it possible to revert #64 and restore previous API, while setting default version of profile to driving?
The text was updated successfully, but these errors were encountered: