a-b-street / a-b-street/ltn

An auto-generated boundary misses the main road on one side

Abierto
#378 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Rust
Estrellas
21
Forks
7
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

Around `17/57.158421/-2.097337` in Aberdeen, the generated boundary misses all of the main road King Street bordering on the east side:

![Image](https://github.com/user-attachments/assets/236deb55-339e-4cf6-86b3-fb8359be2473)
(The lack of border arrows shows the problem.) But then, if I click "Adjust boundary", make no changes, and accept, then it works:

![Image](https://github.com/user-attachments/assets/f2ef9cc9-98cd-4266-b2d8-0e33b3dfa061)

The reason is https://github.com/a-b-street/ltn/blob/69e389d44d6f7f96f7dd77f87562786d7eb118a8/backend/src/neighbourhood.rs#L93 -- we simplify the boundary a bit before generating waypoints, and that gets lucky and nudges us close enough to pick up the main road. Seems like the initial boundary / thresholds should capture this, though

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

El problema está en backend/src/neighbourhood.rs, línea 93, donde la simplificación del límite antes de generar waypoints afecta a la inclusión de carreteras. Empieza por entender la lógica de generación y simplificación del límite. Ejecuta el código alrededor de las coordenadas indicadas para reproducir la carretera que falta y, después, sigue el rastro de por qué el límite/los umbrales iniciales no consiguen capturarla. «Hecho» significa que el límite generado automáticamente incluye correctamente King Street sin ajustes manuales.

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

Evaluación

Stack tecnológico
rust
Área
backend
Tipo de issue
Error
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.