Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[transit] Validate edge weight in gtfs_converter. #13448

Merged
merged 1 commit into from Jul 24, 2020

Conversation

mesozoic-drones
Copy link
Contributor

@mesozoic-drones mesozoic-drones commented Jul 21, 2020

Класс WorldFeed нужен, чтобы конвертировать фиды из формата GTFS и сохранять в специального вида line-by-line json. Этот класс используется тулзой gtfs_converter. Сохраненные с ее помощью json'ы затем отдаются генератору карт (generator_tool) для заполнения новой версии транзитной секции. Более подробное описание WorldFeed есть в реквесте по его добавлению: #12946

В этом реквесте в WorldFeed внесены улучшения:

  • Добавлена валидация весов ребер (edges), соединяющих остановки в граф для роутинга. В GTFS может попадаться оторванное от нашего пространства-времени расписание, а также удивительные координаты остановок. Например, время в пути между двумя автобусными остановками, разнесенными на 60км, может составлять 40 секунд. Линии маршрута с такими аномалиями мы не сохраняем в json. И если в маршруте присутствуют только аномальные линии, удаляем и его. Аналогично поступаем с операторами (network).
  • В связи с более жесткой валидацией ребер скорректирован алгоритм проецирования остановок на полилинию маршрута.
  • Нам не нужно, чтобы айдишники всех сущностей транзита были в интервале FakeFeatureIds::IsTransitFeature(). В этом интервале должны быть только айди ребер, а они заполняются на этапе generator_tool. Поэтому минимальное значение айди сброшено в 0.
  • Совсем мелкие фиксы типа "на сколько элементов в векторе делать reserve" и опечаток.

@mesozoic-drones mesozoic-drones force-pushed the world-feed-edge-validation branch 3 times, most recently from fa2d725 to 29a7c6e Compare July 23, 2020 07:58
@mesozoic-drones
Copy link
Contributor Author

JTALL

@mesozoic-drones
Copy link
Contributor Author

JTANDROID

@mesozoic-drones
Copy link
Contributor Author

JTMAC

1 similar comment
@mesozoic-drones
Copy link
Contributor Author

JTMAC

@mesozoic-drones
Copy link
Contributor Author

JTLINUX

1 similar comment
@ldo2
Copy link
Contributor

ldo2 commented Jul 23, 2020

JTLINUX

@mesozoic-drones
Copy link
Contributor Author

Linux - не проходит тест, не связанный с реквестом: maps_generator.generator.exceptions.FailedTest: Test test_booking_data is failed.

@ldo2
Copy link
Contributor

ldo2 commented Jul 23, 2020

JTALL

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants