Do no rely on fields missing from the ControllerEnvironment

N-Wouda / Euro-NeurIPS-2022

OptiML's contribution to the EURO meets NeurIPS 2022 vehicle routing competition.

Other

16 stars 2 forks source link

Do no rely on fields missing from the ControllerEnvironment #109

Closed N-Wouda closed 2 years ago

N-Wouda commented 2 years ago

This PR reverts the use of env.is_done, env.final_costs, and env.final_solutions (#102). Those attributes are not available when running with the ControllerEnvironment, so submissions would fail.

N-Wouda commented 2 years ago

Does not work with DQN but we're removing that one anyway, so I'm OK with this minimal fix.