graphhopper / jsprit

jsprit is a java based, open source toolkit for solving rich vehicle routing problems
https://www.graphhopper.com/open-source/
Apache License 2.0
1.64k stars 605 forks source link

In AdditionalAccessEgressCalculator class, egress cost doesn't consider completely. #458

Closed baiyyang closed 5 years ago

baiyyang commented 5 years ago

Hi, I think in the AdditionalAccessEgressCalculator class that the calculate method about Egress costs has some trouble. If the new vehicle need not come back depot, and the old vehicle need, then the Egress cost is the new vehicle egress cost 0 minus the old vehicle egress cost. But now, your idea isn't consider the egress cost if the new vehicle need not come back depot.

thanks.

baiyyang commented 5 years ago

I know reason why you don't consider the newVehicle don't need come back depot.... And old vehicle also don't need come back depot...

UpdateEndLocationIfRouteIsOpen, awesome~