mapbox / mapbox/mapbox-navigation-android

Make route parsing options dynamic in viewport data source

Abierto
#4,366 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

feature jira-sync-complete UI
Lenguaje dominante
Kotlin
Estrellas
651
Forks
321
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza con MapboxNavigationViewportDataSource#onRouteChanged y rastrea cómo se analiza y almacena en caché la geometría de la ruta para todos los pasos. Evalúa un enfoque lazy que almacene en caché la geometría del próximo paso mientras las opciones permanezcan sin cambios y, después, verifica que las tres opciones indicadas se respeten tras cambiar, sin que cambie la ruta.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
kotlin
Área
mobile
Tipo de issue
Nueva funcionalidad
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
30/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.