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

Separate turn time and turn weight #2921

Closed
emiltin opened this issue Sep 15, 2016 · 3 comments
Closed

Separate turn time and turn weight #2921

emiltin opened this issue Sep 15, 2016 · 3 comments
Labels
Milestone

Comments

@emiltin
Copy link
Contributor

emiltin commented Sep 15, 2016

Just like speed and weight should be separated, it might be good to separate turn time and turn weight.

In my experience (with bike routing), people prefer a route with few turns, even if it's slightly longer, since It makes the route easier to remember and to follow. As width speeds/weights, if you only have the time penalty to tweak and not a separate weight, there's no way to achieve this without distorting the travel time.

If additional data becomes available to the turn function, you might want to avoid turning to/from/across specific types of roads or lanes based on this data, without affecting the travel time.

Related to #2918.

@MoKob MoKob added the Refactor label Sep 16, 2016
@MoKob
Copy link

MoKob commented Sep 16, 2016

@emiltin from my understanding, this should fall out of #77 if implemented correctly. We might need to adjust the turn function to return both a time and a penalty, but apart from that we should be fine. Timing-penalties need to be added to the time, weight penalties to the weight part and the rest should be handled transparently.

@MoKob MoKob added this to the 6.0 milestone Sep 16, 2016
@emiltin
Copy link
Contributor Author

emiltin commented Sep 16, 2016

sounds good!

@daniel-j-h
Copy link
Member

Closing this in favor of speed / weight split done in #2399.

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

No branches or pull requests

3 participants