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

Max travel time not accounted for with vehicle steps. #954

Closed
jcoupey opened this issue Jul 20, 2023 · 0 comments · Fixed by #961
Closed

Max travel time not accounted for with vehicle steps. #954

jcoupey opened this issue Jul 20, 2023 · 0 comments · Fixed by #961
Labels
Milestone

Comments

@jcoupey
Copy link
Collaborator

jcoupey commented Jul 20, 2023

While looking up #946 I came across another problem when using input steps in solving mode. Take for example this simplified version of our example_1 file:

{
  "vehicles": [
    {
      "id": 1,
      "start": [
        2.35044,
        48.71764
      ],
      "end": [
        2.35044,
        48.71764
      ],
      "steps": [
        {
          "type": "job",
          "id": 1
        }
      ],
      "max_travel_time": 1
    }
  ],
  "jobs": [
    {
      "id": 1,
      "location": [
        1.98935,
        48.701
      ]
    }
  ]
}

The max_travel_time parameter is not accounted for when trying to set the initial solution with the vehicle.steps array. In a release mode, this produces a solution breaking the max travel time constraint. In debug mode, this crashes upon formatting the solution:

vroom: ./utils/helpers.h:540: vroom::Solution vroom::utils::format_solution(const vroom::Input&, const RawSolution&): Assertion `v.ok_for_travel_time(eval_sum.duration)' failed.
Aborted (core dumped)
@jcoupey jcoupey added the bug label Jul 20, 2023
@jcoupey jcoupey added this to the v1.14.0 milestone Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant