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

split-delivery #5

Open
yr18 opened this issue May 2, 2019 · 2 comments
Open

split-delivery #5

yr18 opened this issue May 2, 2019 · 2 comments

Comments

@yr18
Copy link

yr18 commented May 2, 2019

Hi,
In the paper, the author mentioned if the whole delivery constraint is relaxed the result will get better.

However if I omit the constraint( demands.lt(load) the result is always worse. even if I try to retrain the model.

In the paper even for the greedy, split delivery is better than whole delivery.

Do you know what may cause the difference here?

Thank you very much for your help!

@mveres01
Copy link
Owner

mveres01 commented May 2, 2019

Hi!

You're on the right line for split delivery; on line 80, make sure you comment out only the demands.lt() portion and not the whole line, otherwise the mask just won't work

I'd take a look at the hyperparameters a little more first. If you consider split delivery in the context of purely learning to route, it feels like a bit of a specialized behaviour in the sense that the salesman is now given the choice of ignoring the city unless it can meet demand, or taking a detour to fulfill part of it. Try either increasing the batch size and increasing LR, or decreasing batch size and decreasing LR. I've had some luck with e.g.:

python trainer.py --task=vrp --nodes=10 --batch_size=512 --actor_lr=1e-3 --critic_lr=1e-3

Edit: fixed broken link

@yr18
Copy link
Author

yr18 commented May 3, 2019

My case is a VRP problem. I only comment out *demands.lt() part. I only increased the batch_size before, however it gave me worse results. I will try your method.

Thank you so much for your reply!

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

No branches or pull requests

2 participants