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

AddPickupAndDelivery behavior shift #1607

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

braktar
Copy link
Contributor

@braktar braktar commented Sep 30, 2019

The behavior of AddPickupAndDelivery has changed between version 7.0 and 7.3+

It was until now pretty efficient to add this indicator to the model in order to ease the solve with node relations of the following type:

  • same vehicle
  • order (same route, nodes must be in the given order, but other nodes can be inserted between)
  • sequence (same route, nodes must be in the given order and other nodes cannot be inserted between).

But now, it generates SegFaults... Is this behavior change expected?
Btw, here is a sample test case defining the order relation to keep track of it.

@Mizux Mizux self-requested a review January 16, 2020 12:58
@Mizux Mizux added Bug Lang: C++ Native implementation issue Solver: Routing Uses the Routing library and the original CP solver labels Jan 16, 2020
@Mizux Mizux added this to In progress in ToDo via automation Jan 16, 2020
@Mizux Mizux added this to the v7.6 milestone Jan 16, 2020
@Mizux Mizux moved this from In progress to To do in ToDo Jan 21, 2020
@Mizux Mizux modified the milestones: v7.6, v7.7 Mar 3, 2020
@Mizux Mizux modified the milestones: v7.7, v7.8 Jun 12, 2020
@Mizux Mizux modified the milestones: v7.8, v7.9 Jul 21, 2020
@Mizux Mizux modified the milestones: v8.0, v8.1 Sep 9, 2020
@Mizux Mizux removed this from To do in ToDo Sep 9, 2020
@Mizux Mizux modified the milestones: v8.1, v8.2 Nov 23, 2020
@Mizux Mizux modified the milestones: v8.2, v8.3 Feb 10, 2021
@Mizux Mizux modified the milestones: v9.0, v9.1 Mar 22, 2021
@Mizux Mizux modified the milestones: v9.1, v9.2 Sep 13, 2021
@Mizux Mizux added this to In progress in ToDo via automation Sep 13, 2021
@Mizux Mizux removed this from In progress in ToDo Oct 5, 2021
@Mizux Mizux modified the milestones: v9.2, v9.3 Oct 5, 2021
@Mizux Mizux modified the milestones: v9.3, v10.0 Feb 11, 2022
@Mizux Mizux modified the milestones: v10.0, Backlog Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Lang: C++ Native implementation issue Solver: Routing Uses the Routing library and the original CP solver
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants