[routing] Remove unnecessary "keep left/right" directions from the route #2397

RedAuburn · 2022-04-14T13:55:43Z

AntonM030481 commented

2022-04-14 13:55:43 +00:00

(Migrated from github.com)

Fixes: #2262

Signed-off-by: Anton Makouski anton.makouski@gmail.com

Fixes: #2262 Signed-off-by: Anton Makouski anton.makouski@gmail.com

❤️ 2 🎉 2

vng (Migrated from github.com) approved these changes 2022-04-14 15:26:06 +00:00

biodranik (Migrated from github.com) reviewed 2022-04-14 15:58:14 +00:00

routing/turns_generator.cpp

					
				@ -190,7 +191,64 @@ bool KeepTurnByHighwayClass(TurnCandidates const & possibleTurns, TurnInfo const

				      (diff >= kMaxHighwayClassDiffForService && turnHighwayClasses.m_biggestPossibleTurnRoadClass == ftypes::HighwayClass::Service))

				  {

				    // The turn shall be removed if the route goes near small roads.

				    return true;

biodranik (Migrated from github.com) commented

2022-04-14 15:47:30 +00:00

Незавершённое предложение?

biodranik (Migrated from github.com) commented

2022-04-14 15:47:45 +00:00

Можно выровнять после скобки.

biodranik (Migrated from github.com) commented

2022-04-14 15:52:30 +00:00

    // @note The bigger is the road, the lesser is HighwayClass value.

```suggestion // @note The bigger is the road, the lesser is HighwayClass value. ```

biodranik (Migrated from github.com) commented

2022-04-14 15:53:42 +00:00

      // Any alternative turn to a bigger road keeps the turn direction.

```suggestion // Any alternative turn to a bigger road keeps the turn direction. ```

biodranik (Migrated from github.com) commented

2022-04-14 15:54:53 +00:00

      // Any alternative turn to a similar road keeps the turn direction if the turn's angle is less than
      // kMaxAbsAngleSameRoadClass degrees (wider than SlightTurn).

```suggestion // Any alternative turn to a similar road keeps the turn direction if the turn's angle is less than // kMaxAbsAngleSameRoadClass degrees (wider than SlightTurn). ```

biodranik (Migrated from github.com) commented

2022-04-14 15:56:01 +00:00

      // Any alternative turn to a smaller road keeps the turn direction if it's GoStraight or SlightTurn.

```suggestion // Any alternative turn to a smaller road keeps the turn direction if it's GoStraight or SlightTurn. ```

routing/turns_generator.cpp

biodranik (Migrated from github.com) commented

2022-04-14 15:49:49 +00:00

@vng ну хорошо же было бы с std::optional.

routing/turns_generator.cpp

					
				@ -1147,3 +1205,2 @@

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1 &&

				      !KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn))

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1)

				  {

biodranik (Migrated from github.com) commented

2022-04-14 15:58:06 +00:00

Для чего проверка nodes.candidates.size() != 1 ?
candidates.size() не может быть 0?

1. Для чего проверка `nodes.candidates.size() != 1` ? 2. candidates.size() не может быть 0?

AntonM030481 (Migrated from github.com) reviewed 2022-04-14 16:13:33 +00:00

routing/turns_generator.cpp

					
				@ -190,7 +191,64 @@ bool KeepTurnByHighwayClass(TurnCandidates const & possibleTurns, TurnInfo const

				      (diff >= kMaxHighwayClassDiffForService && turnHighwayClasses.m_biggestPossibleTurnRoadClass == ftypes::HighwayClass::Service))

				  {

				    // The turn shall be removed if the route goes near small roads.

				    return true;

AntonM030481 (Migrated from github.com) commented

2022-04-14 16:13:33 +00:00

да. завершил.

AntonM030481 (Migrated from github.com) reviewed 2022-04-14 16:16:01 +00:00

routing/turns_generator.cpp

					
				@ -190,7 +191,64 @@ bool KeepTurnByHighwayClass(TurnCandidates const & possibleTurns, TurnInfo const

				      (diff >= kMaxHighwayClassDiffForService && turnHighwayClasses.m_biggestPossibleTurnRoadClass == ftypes::HighwayClass::Service))

				  {

				    // The turn shall be removed if the route goes near small roads.

				    return true;

AntonM030481 (Migrated from github.com) commented

2022-04-14 16:16:00 +00:00

ок. поправил еще в 2 местах откуда скопировал

AntonM030481 (Migrated from github.com) reviewed 2022-04-14 16:18:26 +00:00

routing/turns_generator.cpp

					
				@ -190,7 +191,64 @@ bool KeepTurnByHighwayClass(TurnCandidates const & possibleTurns, TurnInfo const

				      (diff >= kMaxHighwayClassDiffForService && turnHighwayClasses.m_biggestPossibleTurnRoadClass == ftypes::HighwayClass::Service))

				  {

				    // The turn shall be removed if the route goes near small roads.

				    return true;

AntonM030481 (Migrated from github.com) commented

2022-04-14 16:18:26 +00:00

ок

AntonM030481 (Migrated from github.com) reviewed 2022-04-14 16:19:41 +00:00

routing/turns_generator.cpp

					
				@ -190,7 +191,64 @@ bool KeepTurnByHighwayClass(TurnCandidates const & possibleTurns, TurnInfo const

				      (diff >= kMaxHighwayClassDiffForService && turnHighwayClasses.m_biggestPossibleTurnRoadClass == ftypes::HighwayClass::Service))

				  {

				    // The turn shall be removed if the route goes near small roads.

				    return true;

AntonM030481 (Migrated from github.com) commented

2022-04-14 16:19:41 +00:00

ок

AntonM030481 (Migrated from github.com) reviewed 2022-04-14 16:20:45 +00:00

routing/turns_generator.cpp

					
				@ -190,7 +191,64 @@ bool KeepTurnByHighwayClass(TurnCandidates const & possibleTurns, TurnInfo const

				      (diff >= kMaxHighwayClassDiffForService && turnHighwayClasses.m_biggestPossibleTurnRoadClass == ftypes::HighwayClass::Service))

				  {

				    // The turn shall be removed if the route goes near small roads.

				    return true;

AntonM030481 (Migrated from github.com) commented

2022-04-14 16:20:44 +00:00

ок

AntonM030481 (Migrated from github.com) reviewed 2022-04-14 16:26:50 +00:00

routing/turns_generator.cpp

					
				@ -190,7 +191,64 @@ bool KeepTurnByHighwayClass(TurnCandidates const & possibleTurns, TurnInfo const

				      (diff >= kMaxHighwayClassDiffForService && turnHighwayClasses.m_biggestPossibleTurnRoadClass == ftypes::HighwayClass::Service))

				  {

				    // The turn shall be removed if the route goes near small roads.

				    return true;

AntonM030481 (Migrated from github.com) commented

2022-04-14 16:26:50 +00:00

непонятно

AntonM030481 (Migrated from github.com) reviewed 2022-04-14 16:41:51 +00:00

routing/turns_generator.cpp

					
				@ -1147,3 +1205,2 @@

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1 &&

				      !KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn))

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1)

				  {

AntonM030481 (Migrated from github.com) commented

2022-04-14 16:41:51 +00:00

Хороший вопрос.
Просто скопировал как в KeepTurnByHighwayClass.
Я не вижу проблем с nodes.candidates.size() == 1.
Предположительно, не может.
@vng

1. Хороший вопрос. Просто скопировал как в KeepTurnByHighwayClass. Я не вижу проблем с `nodes.candidates.size() == 1`. 2. Предположительно, не может. @vng

vng (Migrated from github.com) reviewed 2022-04-14 19:47:51 +00:00

routing/turns_generator.cpp

					
				@ -1147,3 +1205,2 @@

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1 &&

				      !KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn))

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1)

				  {

vng (Migrated from github.com) commented

2022-04-14 19:47:50 +00:00

Не может, тут все хорошо и да по аналогии с предидущим сравнением.
Эквивалентно nodes.candidates.size() > 1, проверяем когда больше одной альтернативы

Не может, тут все хорошо и да по аналогии с предидущим сравнением. Эквивалентно nodes.candidates.size() > 1, проверяем когда больше одной альтернативы

vng (Migrated from github.com) reviewed 2022-04-14 19:48:58 +00:00

routing/turns_generator.cpp

					
				@ -190,7 +191,64 @@ bool KeepTurnByHighwayClass(TurnCandidates const & possibleTurns, TurnInfo const

				      (diff >= kMaxHighwayClassDiffForService && turnHighwayClasses.m_biggestPossibleTurnRoadClass == ftypes::HighwayClass::Service))

				  {

				    // The turn shall be removed if the route goes near small roads.

				    return true;

vng (Migrated from github.com) commented

2022-04-14 19:48:58 +00:00

foo(int x,
    int y)

``` foo(int x, int y) ```

vng (Migrated from github.com) reviewed 2022-04-14 19:50:13 +00:00

routing/turns_generator.cpp

vng (Migrated from github.com) commented

2022-04-14 19:50:13 +00:00

тут наверное да

biodranik (Migrated from github.com) reviewed 2022-04-14 20:05:42 +00:00

routing/turns_generator.cpp

					
				@ -1147,3 +1205,2 @@

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1 &&

				      !KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn))

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1)

				  {

biodranik (Migrated from github.com) commented

2022-04-14 20:05:41 +00:00

Тогда надо не копипастить, а сделать так:

  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1)
  {
    if (!KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn) ||
        !KeepTurnByAlignedAlternatives(nodes, turnInfo, numMwmIds))
    {
      turn.m_turn = CarDirection::None;
      return;
    }
  }

Тогда надо не копипастить, а сделать так: ```C++ if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1) { if (!KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn) || !KeepTurnByAlignedAlternatives(nodes, turnInfo, numMwmIds)) { turn.m_turn = CarDirection::None; return; } } ```

AntonM030481 (Migrated from github.com) reviewed 2022-04-15 08:49:49 +00:00

routing/turns_generator.cpp

					
				@ -1147,3 +1205,2 @@

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1 &&

				      !KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn))

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1)

				  {

AntonM030481 (Migrated from github.com) commented

2022-04-15 08:49:49 +00:00

вопросы к @vng :

Note 2. If there's only one exit from this junction (nodes.candidates.size() != 1)
  // this turn should be kept.

если выезд только один, то nodes.candidates.size() == 1, а не наоборот?
если выезд только один - то почему именно в этом случае нужно оставить поворот? Если этого условия не будет, то оба KeepTurnBy... превратят SlightTurn в None. По идее, он и так уже должен быть таким к этому моменту. А если например кандидата 2: вперед и назад - то почему поворот не обязательно оставлять?

вопросы к @vng : ``` Note 2. If there's only one exit from this junction (nodes.candidates.size() != 1) // this turn should be kept. ``` - если выезд только один, то nodes.candidates.size() == 1, а не наоборот? - если выезд только один - то почему именно в этом случае нужно оставить поворот? Если этого условия не будет, то оба KeepTurnBy... превратят SlightTurn в None. По идее, он и так уже должен быть таким к этому моменту. А если например кандидата 2: вперед и назад - то почему поворот не обязательно оставлять?

biodranik (Migrated from github.com) reviewed 2022-04-15 10:18:03 +00:00

routing/turns_generator.cpp

					
				@ -190,7 +191,64 @@ bool KeepTurnByHighwayClass(TurnCandidates const & possibleTurns, TurnInfo const

				      (diff >= kMaxHighwayClassDiffForService && turnHighwayClasses.m_biggestPossibleTurnRoadClass == ftypes::HighwayClass::Service))

				  {

				    // The turn shall be removed if the route goes near small roads.

				    return true;

biodranik (Migrated from github.com) commented

2022-04-15 10:18:03 +00:00

bool KeepTurnByAlignedAlternatives(TurnCandidates const & possibleTurns, TurnInfo const & turnInfo,
                                   NumMwmIds const & numMwmIds)

```suggestion bool KeepTurnByAlignedAlternatives(TurnCandidates const & possibleTurns, TurnInfo const & turnInfo, NumMwmIds const & numMwmIds) ```

vng (Migrated from github.com) reviewed 2022-04-15 13:28:50 +00:00

routing/turns_generator.cpp

					
				@ -1147,3 +1205,2 @@

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1 &&

				      !KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn))

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1)

				  {

vng (Migrated from github.com) commented

2022-04-15 13:28:50 +00:00

Я эти комменты не писал :) но предполагаю что:

Для одного exit когда nodes.candidates.size() == 1, мы НЕ заходим в KeepTurnByHighwayClass и turn точно сохраняется. Можете переписать этот коммент на понятный :)
Вероятно он остается, потому что вычисляется по какому-то другому жесткому условию раньше (например это может быть пересадка на паром), тут не скажу точно. Пусть так и остается, пока не увидим явный косяк

Я эти комменты не писал :) но предполагаю что: - Для одного exit когда nodes.candidates.size() == 1, мы НЕ заходим в KeepTurnByHighwayClass и turn точно сохраняется. Можете переписать этот коммент на понятный :) - Вероятно он остается, потому что вычисляется по какому-то другому жесткому условию раньше (например это может быть пересадка на паром), тут не скажу точно. Пусть так и остается, пока не увидим явный косяк

AntonM030481 (Migrated from github.com) reviewed 2022-04-15 13:41:16 +00:00

routing/turns_generator.cpp

					
				@ -1147,3 +1205,2 @@

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1 &&

				      !KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn))

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1)

				  {

AntonM030481 (Migrated from github.com) commented

2022-04-15 13:41:16 +00:00

Меняю, если нет возражений, на

  // Note 2. If there's only one exit from this junction (nodes.candidates.size() == 1)
  // this turn should be kept (because if was kept by previous logic as an exception).

Меняю, если нет возражений, на ``` // Note 2. If there's only one exit from this junction (nodes.candidates.size() == 1) // this turn should be kept (because if was kept by previous logic as an exception). ```

AntonM030481 (Migrated from github.com) reviewed 2022-04-15 13:51:39 +00:00

routing/turns_generator.cpp

					
				@ -1147,3 +1205,2 @@

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1 &&

				      !KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn))

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1)

				  {

AntonM030481 (Migrated from github.com) commented

2022-04-15 13:51:39 +00:00

После перехода на

if (!KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn) ||
        !KeepTurnByAlignedAlternatives(nodes, turnInfo, numMwmIds))

Стала понятна немного некорректная логика в названиях функций.
Исходя из названия, должно быть так

if (! (KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn) ||
        KeepTurnByAlignedAlternatives(nodes, turnInfo, numMwmIds)))

те если ни одной причины нет, чтобы сохранить поворот - то мы его удаляем.
а у нас получается, что если причины 1 нет - удаляем, если причины 2 нет - удаляем.

Предлагаю инвертировать функции на DiscardTurnByHighwayClass и DiscardTurnByNoAlignedAlternatives.
Тогда получится:

if (DiscardTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn) ||
    DiscardTurnByNoAlignedAlternatives(nodes, turnInfo, numMwmIds))

После перехода на ``` if (!KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn) || !KeepTurnByAlignedAlternatives(nodes, turnInfo, numMwmIds)) ``` Стала понятна немного некорректная логика в названиях функций. Исходя из названия, должно быть так ``` if (! (KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn) || KeepTurnByAlignedAlternatives(nodes, turnInfo, numMwmIds))) ``` те если ни одной причины нет, чтобы сохранить поворот - то мы его удаляем. а у нас получается, что если причины 1 нет - удаляем, если причины 2 нет - удаляем. Предлагаю инвертировать функции на DiscardTurnByHighwayClass и DiscardTurnByNoAlignedAlternatives. Тогда получится: ``` if (DiscardTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn) || DiscardTurnByNoAlignedAlternatives(nodes, turnInfo, numMwmIds)) ```

AntonM030481 (Migrated from github.com) reviewed 2022-04-15 13:57:58 +00:00

routing/turns_generator.cpp

					
				@ -190,7 +191,64 @@ bool KeepTurnByHighwayClass(TurnCandidates const & possibleTurns, TurnInfo const

				      (diff >= kMaxHighwayClassDiffForService && turnHighwayClasses.m_biggestPossibleTurnRoadClass == ftypes::HighwayClass::Service))

				  {

				    // The turn shall be removed if the route goes near small roads.

				    return true;

AntonM030481 (Migrated from github.com) commented

2022-04-15 13:57:57 +00:00

спасибо. понял.

vng (Migrated from github.com) reviewed 2022-04-15 14:15:22 +00:00

routing/turns_generator.cpp

					
				@ -1147,3 +1205,2 @@

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1 &&

				      !KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn))

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1)

				  {

vng (Migrated from github.com) commented

2022-04-15 14:15:22 +00:00

Согласен на переименовать. Тогда надо аккуратно внутри них инвертировать true/false и проверить актуальность комментов к ним.

AntonM030481 (Migrated from github.com) reviewed 2022-04-15 14:27:55 +00:00

routing/turns_generator.cpp

					
				@ -1147,3 +1205,2 @@

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1 &&

				      !KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn))

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1)

				  {

AntonM030481 (Migrated from github.com) commented

2022-04-15 14:27:55 +00:00

в процессе переименования обратил внимание на

bool KeepTurnByHighwayClass()
  ...
  if (!GetTurnHighwayClasses(possibleTurns, turnInfo, numMwmIds, turnHighwayClasses))
    return false;

в процессе переименования обратил внимание на ``` bool KeepTurnByHighwayClass() ... if (!GetTurnHighwayClasses(possibleTurns, turnInfo, numMwmIds, turnHighwayClasses)) return false; ```

AntonM030481 (Migrated from github.com) reviewed 2022-04-15 14:31:58 +00:00

routing/turns_generator.cpp

					
				@ -1147,3 +1205,2 @@

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1 &&

				      !KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn))

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1)

				  {

AntonM030481 (Migrated from github.com) commented

2022-04-15 14:31:58 +00:00

Исходя из описания GetTurnHighwayClasses()
// If it's possible fills |turnHighwayClasses| and returns true, and returns false otherwise.
это выглядит ошибкой: если данные не получили - то вернуть false и удалить поворот.
должно быть наоборот? если данные не получили - то не трогать поворот на всякий случай.

Исходя из описания GetTurnHighwayClasses() // If it's possible fills |turnHighwayClasses| and returns true, and returns false otherwise. это выглядит ошибкой: если данные не получили - то вернуть false и удалить поворот. должно быть наоборот? если данные не получили - то не трогать поворот на всякий случай.

AntonM030481 (Migrated from github.com) reviewed 2022-04-15 14:38:51 +00:00

routing/turns_generator.cpp

					
				@ -1147,3 +1205,2 @@

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1 &&

				      !KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn))

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1)

				  {

AntonM030481 (Migrated from github.com) commented

2022-04-15 14:38:51 +00:00

Есть подозрение, что
GetTurnHighwayClasses()
не совсем соответствует описанию в отношении возвращаемого значения.

Есть подозрение, что GetTurnHighwayClasses() не совсем соответствует описанию в отношении возвращаемого значения.

AntonM030481 (Migrated from github.com) reviewed 2022-04-15 15:14:05 +00:00

routing/turns_generator.cpp

					
				@ -1147,3 +1205,2 @@

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1 &&

				      !KeepTurnByHighwayClass(nodes, turnInfo, numMwmIds, turn.m_turn))

				  if (!turn.m_keepAnyway && IsGoStraightOrSlightTurn(turn.m_turn) && nodes.candidates.size() != 1)

				  {

AntonM030481 (Migrated from github.com) commented

2022-04-15 15:14:05 +00:00

Оставил старую логику (инвертировав true на false), добавив комент:

bool DiscardTurnByHighwayClass(TurnCandidates const & possibleTurns, TurnInfo const & turnInfo,
                               NumMwmIds const & numMwmIds, CarDirection carDirection)
...
  // Looks like a bug. If no info received turn should not be discarded
  if (!GetTurnHighwayClasses(possibleTurns, turnInfo, numMwmIds, turnHighwayClasses))
    return true;

Оставил старую логику (инвертировав true на false), добавив комент: ``` bool DiscardTurnByHighwayClass(TurnCandidates const & possibleTurns, TurnInfo const & turnInfo, NumMwmIds const & numMwmIds, CarDirection carDirection) ... // Looks like a bug. If no info received turn should not be discarded if (!GetTurnHighwayClasses(possibleTurns, turnInfo, numMwmIds, turnHighwayClasses)) return true; ```

AntonM030481 (Migrated from github.com) reviewed 2022-04-15 15:15:07 +00:00

routing/turns_generator.cpp

					
				@ -190,7 +191,64 @@ bool KeepTurnByHighwayClass(TurnCandidates const & possibleTurns, TurnInfo const

				      (diff >= kMaxHighwayClassDiffForService && turnHighwayClasses.m_biggestPossibleTurnRoadClass == ftypes::HighwayClass::Service))

				  {

				    // The turn shall be removed if the route goes near small roads.

				    return true;

AntonM030481 (Migrated from github.com) commented

2022-04-15 15:15:07 +00:00

done

AntonM030481 (Migrated from github.com) reviewed 2022-04-15 15:17:17 +00:00

routing/turns_generator.cpp

AntonM030481 (Migrated from github.com) commented

2022-04-15 15:17:17 +00:00

предлагаю пока оставить как есть.
а потом отрефакторить, и заодно разобраться с возвращаемыми значениями GetTurnHighwayClasses()

предлагаю пока оставить как есть. а потом отрефакторить, и заодно разобраться с возвращаемыми значениями GetTurnHighwayClasses()

vng (Migrated from github.com) approved these changes 2022-04-15 15:38:09 +00:00

AntonM030481 commented

2022-04-15 17:24:38 +00:00

(Migrated from github.com)

Не очень понятно, почему не все тесты не прошли.

biodranik commented

2022-04-15 17:42:09 +00:00

(Migrated from github.com)

Сначала смотрим в конце
FAILED
2022-04-15 16:05:25,582 - base_tests

Потом ищем упавший тест:
Running small_set_test.cpp::SmallMap_Benchmark3
FAILED
base_tests/small_set_test.cpp:262 TEST(t3 < t2) 111 111

Это не релевантно к текущему PR. @vng ?

Сначала смотрим в конце FAILED 2022-04-15 16:05:25,582 - base_tests Потом ищем упавший тест: Running small_set_test.cpp::SmallMap_Benchmark3 FAILED base_tests/small_set_test.cpp:262 TEST(t3 < t2) 111 111 Это не релевантно к текущему PR. @vng ?

vng commented

2022-04-15 17:49:50 +00:00

(Migrated from github.com)

Не релевантно, если @biodranik approve, squash and merge.

biodranik (Migrated from github.com) approved these changes 2022-04-15 17:52:07 +00:00

biodranik (Migrated from github.com) reviewed 2022-04-15 17:53:31 +00:00

routing/turns_generator.cpp

					
				@ -194,0 +219,4 @@

				  Segment inversedLastIngoingSegment;

				  if (!turnInfo.m_ingoing.m_segmentRange.GetLastSegment(numMwmIds, inversedLastIngoingSegment))

				    return false;

biodranik (Migrated from github.com) commented

2022-04-15 17:53:31 +00:00

Момент, если false исправлено на true, то коммент надо дополнить.
// Previous version was returning false and it looked like a bug. If no info received turn should not be discarded.

Момент, если false исправлено на true, то коммент надо дополнить. // Previous version was returning false and it looked like a bug. If no info received turn should not be discarded.

biodranik (Migrated from github.com) reviewed 2022-04-15 17:55:24 +00:00

routing/turns_generator.cpp

biodranik (Migrated from github.com) commented

2022-04-15 17:55:23 +00:00

  // this turn should be kept (because it was kept by previous logic as an exception).

```suggestion // this turn should be kept (because it was kept by previous logic as an exception). ```

AntonM030481 (Migrated from github.com) reviewed 2022-04-15 18:37:35 +00:00

routing/turns_generator.cpp

					
				@ -194,0 +219,4 @@

				  Segment inversedLastIngoingSegment;

				  if (!turnInfo.m_ingoing.m_segmentRange.GetLastSegment(numMwmIds, inversedLastIngoingSegment))

				    return false;