← Retour aux issues

pb photon : rue non trouvée si housenumber pas dans OSM

publié le , mis à jour
Avatar Codeberg de etienneJretienneJr

ticket ouvert suite à ce message sur mastodon

plus précisément : rue non trouvée (ou éventuellement en 4e position avec le biais local) si la requête photon contient un numéro de maison qui n'existe pas dans OSM

j'ai posé la question sur le repo photon : https://github.com/komoot/photon/discussions/968

  1. Avatar Codeberg de etienneJretienneJr

    dans #1231 une 1ère amélioration de photon résout le bug du message mastodon, mais c'est pas totalement satisfaisant, il y a encore du boulot image

  2. Avatar Codeberg de laemlaem

    @etienneJr yes I'm afraid the PR could lead to a lot of false positives what about adding a request ? we would then remove duplicates and show 3 "adress" results with an explicit label in the list to explain what we did to the user, and 3 general results ?

  3. Avatar Codeberg de etienneJretienneJr

    Yes good idea. But when I read the code, I didn't know how to modify it to fetch 2 request in parallel. Do you have an example?

  4. Avatar Codeberg de etienneJretienneJr

    @pmiossec est-ce que ça t'intéresserait de bosser là dessus ? Il y a aussi le ticket #1261 en lien, et les PR associées #1231 et #1267

    J'aurais aimé le faire, mais mes compétences en react sont limitées, je ne sais pas par quel bout le prendre.

    Je pense que le pb vient du fait que quand photon ne trouve pas "11 avenue de la sérane", alors il sort tous les "11 avenue de" plutôt que de sortir "avenue de la sérane".

    Une option serait de se pencher sur photon pour comprendre comment il marche et modifier ses poids internes...

    Une autre option côté cartes.app, un peu comme ce que tu as fait pour les pluscode :

    • quand on détecte le vocabulaire rue, avenue, ... on déclenche en parallèle 1 ou 2 autres requêtes photon avec layer=street et layer=house
      • celle parmi les rues devrait toujours fonctionner
      • pour celle parmi les house, on pourrait essayer de virer le mot rue, avenue, ... de la query, donc chercher "11 de la sérane", pour voir si ça évite de récupérer tous les "11 avenue de"
    • sur l'app, on affiche "adresse détectée" avec les résultats de cette/ces recherche(s).

    Dernière option : utiliser une api tierce spécifique des adresses quand on détecte rue, avenue, ... : la poste ? ign ? addock ? ban ? (je n'y ai pas réfléchi) mais avec le défaut que ça ne marche qu'en France.

  5. Avatar Codeberg de etienneJretienneJr
  6. Avatar Codeberg de etienneJretienneJr

    photon#1041 a corrigé le pb, ça marche sur https://photon.komoot.io/ mais pas sur cartes.app @laem comment on met à jour photon vers la version 1.0.1 ?

  7. Avatar Codeberg de laemlaem

    Es-tu sûr @etienneJr que la v 1.0.1 résoud ce bug ? Publiée le 9 mars, j'ai l'impression que la nouvelle est toujours en rodage, non publiée sur github ?

  8. Avatar Codeberg de etienneJretienneJr

    Ah oui pardon tu as raison, j'avais confondu la liste des commits réalisés depuis la release, et celle des commits déjà intégrés dans la release.

  9. Avatar Codeberg de laemlaem

    D'ailleurs tu as vu, une PR a été faite hier https://github.com/komoot/photon/pull/1051

✏️ Participer à la discussion