mapbox / mapbox/mapbox-navigation-android
Make route parsing options dynamic in viewport data source
Nadie ha tomado este issue todavía.
- 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.minimumDistanceBetweenIntersectionsoptions.followingFrameOptions.frameGeometryAfterManeuver.distanceToCoalesceCompoundManeuversoptions.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
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- 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