mapsme / omim

🗺️ MAPS.ME — Offline OpenStreetMap maps for iOS and Android
Apache License 2.0
4.58k stars 1.15k forks source link

[transit][generator] Fix cross mwm features numeration for experimental transit. #14162

Closed tatiana-yan closed 3 years ago

tatiana-yan commented 3 years ago

У нас в transit cross mwm (как и в остальном cross mwm) закреплено соответствие между cross mwm id и feature id, при этом на генерации feature id брался из json (причём в json feature id кросс-мвмного ребра одинаковый внутри разных мвм, чего кажется быть не должно).

В рантайме feature id нумеруются по порядку в графе (с оффсетом чтобы фичи были фейковыми), а в кросс-мвм к считанному идентификатору прибавляют оффсет. И по факту фича на которую "смотрит" кросс-мвм получатся не та.

В этом реквесте сделан тривиальный фикс в генерацию кросс мвм, который вместо featureId из json использует порядковый номер. В этом реквесте не сделано исправление feature id в json или выпиливание его оттуда поскольку я не разобралась в коде который его создаёт.

Пока мне кажется что целесообразнее выпилить feature id из json, поскольку нумерация должна быть консистентна с нумерацией при построении графа ОТ в рантайме и логичнее эту консистентность требовать от кода генератора чем от gtfs_converter.

Построение кросс-мвмных маршрутов на от. Было: креш в А*, если убрать проверку маршрут такой

Снимок экрана 2021-04-21 в 19 55 11

Стало: нет креша в А*, маршрут такой

Снимок экрана 2021-04-21 в 19 54 51