Skip to content

Unify .osrm.turn_penalties_index dump same with turn_weight_penalties/turn_duration_penalties#5868

Merged
akashihi merged 3 commits intoProject-OSRM:masterfrom
wangyoucao577:feature/unify-turn-index-dump
Nov 10, 2020
Merged

Unify .osrm.turn_penalties_index dump same with turn_weight_penalties/turn_duration_penalties#5868
akashihi merged 3 commits intoProject-OSRM:masterfrom
wangyoucao577:feature/unify-turn-index-dump

Conversation

@wangyoucao577
Copy link
Copy Markdown
Contributor

@wangyoucao577 wangyoucao577 commented Oct 23, 2020

Issue

What issue is this PR targeting? If there is no issue that addresses the problem, please open a corresponding issue and link it here.
Closes #5862

Please read our documentation on release and version management.
If your PR is still work in progress please attach the relevant label.

Tasklist

Requirements / Relations

Link any requirements here. Other pull requests this PR is based on?

Copy link
Copy Markdown
Contributor

@akashihi akashihi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks much cleaner now!

@wangyoucao577
Copy link
Copy Markdown
Contributor Author

@akashihi Could you please also help to merge it? I don't have write access. :)

@akashihi
Copy link
Copy Markdown
Contributor

Of course, i just wanted to give other maintainers to check that PR :)

@akashihi akashihi merged commit 98fd175 into Project-OSRM:master Nov 10, 2020
@wangyoucao577 wangyoucao577 deleted the feature/unify-turn-index-dump branch November 11, 2020 00:58
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.

Inconsistent implementation between turn_indexes_write_buffer and turn_weight_penalties/turn_duration_penalties

2 participants