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

Configuration option to force routing to select Turn Cost sensitive algo for routing #1220

Closed
takb opened this issue Oct 5, 2022 · 1 comment · Fixed by #1221
Closed
Assignees

Comments

@takb
Copy link
Contributor

takb commented Oct 5, 2022

A customer wants to have the option to have a profile have CH enabled (for matrix) but still have routing respect turn costs. Algorithm selection needs to default to FLEX_PREPROCESSED mode instead of FLEX_STATIC.

Easiest place to put is encoder_options of the respective profile, where we also enable turn costs, e.g.

  "profile-car": {
    "profiles": "driving-car",
    "parameters": {
      "encoder_options": "turn_costs=true|force_turn_costs=true|...",
      ...

We decided to have a patch release based off of the last release, although this breaks our version semantics a bit.

@takb takb self-assigned this Oct 5, 2022
@takb
Copy link
Contributor Author

takb commented Oct 7, 2022

Semantically it makes more sense to place directly as a profile parameter, e.g.

  "profile-car": {
    "profiles": "driving-car",
    "parameters": {
      "force_turn_costs": true,
      ... 

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 a pull request may close this issue.

1 participant