Closed jieyibi closed 5 months ago
It seems there was a mistake here - I noticed that the "tasks" are a bit hardcoded, i.e. the "size" and "capacity" (I guess for TSP and CVRP, right?)
Speaking of testing: you can make sure things work on your device before committing by using pytest tests
Routine check, how is progress going? I think the multi-distribution generators in particular should be included, since they are part of MDPOMO
, right?
The updated commit supports training on multiple mixed distributions by changing the argument loc_distribution
to "mix_distribution" in the generator_params
of the environment, e.g.
env = CVRPEnv(generator_params={'loc_distribution': "mix_distribution"})
The rest of the arguments remain the same.
Note that this mixed distribution setting follows the # setting in Bi et al., 2022.
Great! Should the code be merged?
Yes. I think it is ready to be merged. The newly reproduced performance (by changing some key parameters) seems similar to that in the literature.
Great!
How about the generalization experiments, i.e., MDPOMO? Do you have yaml
configuration for them like this, or a training script?
Yeah, already added to the main branch:)
Awesome! Then we can go ahead and merge :)
Description
Add a new training framework based on meta learning. Details refer to Zhou et al. 2023.
Motivation and Context
To address the generalization issue.
Types of changes
What types of changes does your code introduce? Remove all that do not apply:
Checklist
Go over all the following points, and put an
x
in all the boxes that apply. If you are unsure about any of these, don't hesitate to ask. We are here to help!