gpesquero/organicmaps
1
Fork 0
forked from organicmaps/organicmaps
Code Pull requests Activity

[routing] Fix inadequate manoeuvres near the mwm border

Browse source
This commit is contained in:
Добрый Ээх 2016-12-09 15:55:26 +03:00
parent 9d39725228
commit 7caba7823e
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
routing/car_router.cpp
View file

@ -466,7 +466,12 @@ CarRouter::ResultCode CarRouter::CalculateRoute(m2::PointD const & startPoint,
}
INTERRUPT_WHEN_CANCELLED(delegate);
if (crossCode == NoError && crossDistanceM < route.GetTotalDistanceMeters())
// Cross mwm and single mwm routes are built by different algorithms.
// It is not correct to compare distances from different algorithms.
// Add some penalty to cross mwm route to avoid inadequate manoeuvres near the mwm borders.
// TODO: remove this hack together with OSRM routing.
double constexpr kCrossMwmPenalty = 0.8;
if (crossCode == NoError && crossDistanceM < kCrossMwmPenalty * route.GetTotalDistanceMeters())
{
LOG(LINFO, ("Cross mwm path is shorter than single mwm path. Cross distance:",
crossDistanceM, "single distance:", route.GetTotalDistanceMeters()));