chairemobilite / transition

Transition is a modern new approach to transit planning. It's a web application to model, simulate and plan public transit and alternative transportation.
http://transition.city
MIT License
20 stars 13 forks source link

GTFS: Use postgis to get existing nodes when aggregation radius is 0 #931

Closed tahini closed 2 months ago

tahini commented 2 months ago

fixes #908

Since KDBush sometimes does not return a node which is at the exact location as another one, the main code path sometimes created new nodes at the exact same location when the aggregation radius was set to 0. postgis, on the other hand, does not do that, so we get the existing node from the database in that case.

Also

tahini commented 2 months ago

@greenscientist cette PR va faire des conflits avec ta branche actuelle

greenscientist commented 2 months ago

@greenscientist cette PR va faire des conflits avec ta branche actuelle

Ouais, expected! Mais comme je sais pas ça va prendre combien de temps régler la mienne, c'est bon.

tahini commented 2 months ago

@greenscientist Le dernier commit ajoute des validations de la présence des transferable nodes dans la cache.