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

Include turn restrictions? #22

Closed
maning opened this issue Jan 9, 2017 · 5 comments
Closed

Include turn restrictions? #22

maning opened this issue Jan 9, 2017 · 5 comments

Comments

@maning
Copy link

maning commented Jan 9, 2017

Great tool! For modelling routes, does this module incorporate turn_restrictions? In OSM, it is modeled as relations.

@gboeing
Copy link
Owner

gboeing commented Jan 10, 2017

At the moment it does not, but I'm looking into adding this into an upcoming release.

@gboeing
Copy link
Owner

gboeing commented Jan 30, 2017

While I like the idea of this in theory, it looks like it's beyond the scope of OSMnx. A robust routing package built on top of OSMnx could perhaps be a better future home for this functionality. Directional turn restrictions provide a nontrivial algorithmic challenge. OSMnx uses NetworkX's path calculation algorithms, and they don't account for spatial networks with turn restrictions. Using something like OSRM or SimpleOsmRouter or PyRoute might work for your application. But at this point, adding this functionality is beyond the project scope.

@gboeing gboeing closed this as completed Jan 30, 2017
@higee
Copy link

higee commented Nov 16, 2017

@gboeing Any news for this issue?

I wonder if you by any chance have started working on this issue. I found out that pgRouting, for instance, offers TRSP (turn restriction shortest path). But as a big fan of osmnx, I want to stay in this platform.

If not, could you provide me with any kind of hint so that I could try customizing osmnx and probably networkx by myself? I'm fully aware of the fact that it might become a huge project as you've mentioned couple month ago.

I'm also curious whether @maning found a way to this issue.

@gboeing
Copy link
Owner

gboeing commented Nov 17, 2017

@higee I haven't worked on it further for the reasons I mentioned previously. But I'm open to it if it's feasible to incorporate TRSP with a python package. Keep me posted if your explorations uncover something useful.

@higee
Copy link

higee commented Nov 21, 2017

@gboeing Could you share your email address? I wrote down some issues and my approaches with images inside pdf file. I tried uploading here but somehow it doesn't go up.

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

3 participants