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

Example of a Vehicle Routing Problem with Uninterruptible Breaks #595

Closed
wants to merge 7 commits into from
Closed

Example of a Vehicle Routing Problem with Uninterruptible Breaks #595

wants to merge 7 commits into from

Conversation

pmateusz
Copy link

An example of the Vehicle Routing Problem with Time Windows and Breaks. All breaks are mandatory and must be performed without interruptions. Since breaks without interruptions are not available off-the-shelf a custom constraint is developed. A warning message is printed every time vehicle's break overlaps with a visit.

Workaround for the issue: #569

@googlebot
Copy link

Thanks for your pull request. t looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers
  • Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the project maintainer to go/cla#troubleshoot.
  • The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
  • The email used to register you as an authorized contributor must also be attached to your GitHub account.

@pmateusz
Copy link
Author

I have signed it!

@googlebot
Copy link

CLAs look good, thanks!

@Mizux Mizux requested review from orwant and Mizux March 7, 2018 13:54
@Mizux Mizux added Feature Request Missing Feature/Wrapper OS: Windows Windows OS OS: Linux GNU/Linux OS OS: Mac MacOS labels Mar 7, 2018
@Mizux Mizux modified the milestones: v6.X, Backlog, v6.8 Mar 7, 2018
@Mizux Mizux requested review from lperron and furnon and removed request for orwant March 13, 2018 10:46
@Mizux Mizux self-assigned this Apr 23, 2018
@Mizux Mizux modified the milestones: v6.8, v6.9 May 2, 2018
@Mizux
Copy link
Collaborator

Mizux commented Jun 26, 2018

Hi,

It seems "break" feature has been fixed internally, so I hope for v6.9 to bump routing library to our current internal code...

BTW I still keep this PR "open" until break is updated and this PR become obsolete.

@pmateusz
Copy link
Author

So it was a bug indeed. Thank you for confirmation! I am pleased that the work I did helped a little. It is of course always better to fix the root cause rather than rely on the workaround my PR provides. Feel free to close the PR once it is best.

@pmateusz pmateusz closed this Jun 26, 2018
@tomika
Copy link

tomika commented Aug 30, 2018

Dear Mateusz,
I would like to know if it is really necessary to add

solver()->AddConstraint(solver()->MakeIntervalVarRelation(current_interval, Solver::STARTS_AFTER_END, last_interval));

constraints?
Is it not a basic, already existing constraint in the whole routing calculation?
Thanks for your answer in advance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request Missing Feature/Wrapper OS: Linux GNU/Linux OS OS: Mac MacOS OS: Windows Windows OS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants