mapbox / mapbox/mapbox-navigation-android

Make route parsing options dynamic in viewport data source

オープン
#4,366 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

feature jira-sync-complete UI
主要言語
Kotlin
スター
651
フォーク
321
PR マージ指標
30日以内にマージされた PR はありません

説明

Some of the options in the viewport data source are "static" in a sense that they only take effect when a route changes (MapboxNavigationViewportDataSource#onRouteChanged). Namely:

  • options.followingFrameOptions.intersectionDensityCalculation.minimumDistanceBetweenIntersections
  • options.followingFrameOptions.frameGeometryAfterManeuver.distanceToCoalesceCompoundManeuvers
  • options.followingFrameOptions.frameGeometryAfterManeuver.distanceToFrameAfterManeuver

Now, the route geometry is parsed once when it's provided and resulting geometries for all steps are cached. We could explore a lazy loading approach where we generate and cache only the route geometries for the upcoming step (as long as the options don't change) to make the above options also respected "dynamically".

cc @MaximAlien

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

MapboxNavigationViewportDataSource#onRouteChanged から始め、すべてのステップのルートジオメトリがどのように解析され、キャッシュされるかを追跡します。オプションが変更されていない間は次のステップのジオメトリをキャッシュする lazy なアプローチを評価し、その後、ルートを変更せずに変更された場合でも、一覧にある3つのオプションが反映されることを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
kotlin
領域
mobile
issue の種類
機能追加
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
30/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。