Amélioration des suggestions de recherche par importance géographique #2083

younes_mstr commented 2026年04月27日 01:28:01 +02:00

Ce PR fait suite au ticket #1769. Photon retournait les résultats sans biais géographique quand "Chercher ici" était décoché ce qui pouvait faire remonter des résultats moins pertinents que voulu (un village qui s'appelle Barcelone avant la ville de Barcelone). J'ai donc ajouté un re-classement basé sur la hiérarchie OSM proposé par Photon : les grandes villes remontent avant les villages, les pays avant les régions, etc. Le re-classement est ignoré quand "Chercher ici" est coché ou quand la requête ressemble à une adresse (rue, avenue ...). Seul le fichier fetchPhoton.ts a été modifié.

Ce PR fait suite au ticket #1769. Photon retournait les résultats sans biais géographique quand "Chercher ici" était décoché ce qui pouvait faire remonter des résultats moins pertinents que voulu (un village qui s'appelle Barcelone avant la ville de Barcelone). J'ai donc ajouté un re-classement basé sur la hiérarchie OSM proposé par Photon : les grandes villes remontent avant les villages, les pays avant les régions, etc. Le re-classement est ignoré quand "Chercher ici" est coché ou quand la requête ressemble à une adresse (rue, avenue ...). Seul le fichier fetchPhoton.ts a été modifié.

laem commented 2026年04月27日 07:39:43 +02:00

Salut @younes_mstr, merci pour cette PR ! Peux-tu nous présenter une liste de 3 ou 4 requêtes qui montrent les différences entre la version actuelle et la nouvelle ?

En commentaire ici.

Salut @younes_mstr, merci pour cette PR ! Peux-tu nous présenter une liste de 3 ou 4 requêtes qui montrent les différences entre la version actuelle et la nouvelle ? En commentaire ici.

younes_mstr commented 2026年04月27日 12:26:46 +02:00

Salut @laem, oui bien sur !

AVANT	APRÈS
boulogne avant	boulogne apres
asnieres avant	asvnieres apres
aulnay avant	aulnay apres
aix avant	aix apres

On peut voir sur le tableau comparatif que la PR remonte les villes au détriment des villages et même quelques fois, fait apparaître des villes qu'on ne voyait même pas (le cas par exemple d'Asnières et d'Aulnay).

Les scores selon le type du résultat (ville, pays, village, ...) sont dans un record et donc peuvent être affiné selon vous (j'ai mis ce qui me paraissaient le plus évident et j'ai un peu affiné après mais, c'est selon moi, rien ne vous empêche d'affiner par la suite).

Ce que je n'ai pas réussi à penser c'est pour les adresses (c'est pour ça que je les ai exclus pour le moment) : "rue de la paix" va faire sortir des rues de Laval et de Gentilly. Peut-être essayer de trouver un moyen pour avoir un score de popularité ou quelque chose du genre (population ou tout type d'indicateur pouvant hiérarchiser convenablement les rues et adresses).

Salut @laem, oui bien sur ! | AVANT | APRÈS | |---------|---------| |  |  | |  |  | |  |  | |  |  | On peut voir sur le tableau comparatif que la PR remonte les villes au détriment des villages et même quelques fois, fait apparaître des villes qu'on ne voyait même pas (le cas par exemple d'Asnières et d'Aulnay). Les scores selon le type du résultat (ville, pays, village, ...) sont dans un record et donc peuvent être affiné selon vous (j'ai mis ce qui me paraissaient le plus évident et j'ai un peu affiné après mais, c'est selon moi, rien ne vous empêche d'affiner par la suite). Ce que je n'ai pas réussi à penser c'est pour les adresses (c'est pour ça que je les ai exclus pour le moment) : "rue de la paix" va faire sortir des rues de Laval et de Gentilly. Peut-être essayer de trouver un moyen pour avoir un score de popularité ou quelque chose du genre (population ou tout type d'indicateur pouvant hiérarchiser convenablement les rues et adresses).

laem commented 2026年05月04日 18:33:56 +02:00

Merci pour la PR ! C'est cool, les résultats sont prometteurs et le code est simple.

J'ai une interrogation cependant : ton code écrase complètement pour les "place" l'ordre par défaut retourné par Photon. Mais je crois que c'est pas grave, car en testant https://photon.komoot.io, hors cartes.app donc, je me rend compte que sans la recherche locale (appelée location bias chez Photon), l'ordre est vraiment tout pété et constant, non dépendant de l'utilisateur.

Donc oui, très bien d'ordonner par défaut :)

Merci pour la PR ! C'est cool, les résultats sont prometteurs et le code est simple. J'ai une interrogation cependant : ton code écrase complètement pour les "place" l'ordre par défaut retourné par Photon. Mais je crois que c'est pas grave, car en testant https://photon.komoot.io, hors cartes.app donc, je me rend compte que sans la recherche locale (appelée location bias chez Photon), l'ordre est vraiment tout pété et constant, non dépendant de l'utilisateur. Donc oui, très bien d'ordonner par défaut :)