9
102
Fork
You've already forked web
30

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

Open
opened 2025年11月06日 14:05:42 +01:00 by etienneJr · 9 comments

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

ticket ouvert suite à [ce message sur mastodon](https://piaille.fr/@chicraote/115474386901086952) 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
Author
Member
Copy link

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

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](/attachments/9e310231-4d76-47f2-9b2d-6ab0c6d8fb9b)
Owner
Copy link

@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 ?

@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 ?
Author
Member
Copy link

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?

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?
Author
Member
Copy link

@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.

@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.
Author
Member
Copy link
À propos d'addock https://forum.geocommuns.fr/t/addok-1-3-est-disponible/2910
Author
Member
Copy link

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 ?

[photon#1041](https://github.com/komoot/photon/pull/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 ?
Owner
Copy link

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 ?

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 ?
Author
Member
Copy link

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.

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.
Owner
Copy link

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

D'ailleurs tu as vu, une PR a été faite hier https://github.com/komoot/photon/pull/1051
Sign in to join this conversation.
No Branch/Tag specified
master
abostats
SynchroM-master
gps
embeddings
lysios-patch-5
lysios-patch-4
lysios-patch-3
lysios-patch-2
lysios-patch-1
abo-serveur2
clim
zone-en-favori
categories_toujours_visibles
rechercher_ici_avant_resultats
test-2365
panoramax-contribution
add-categories-validations
gel-bug-prod
osm-note-2
broken
wikidata_booking
fuse7.3
lysios-patrimoine-3
gps2
atproto-fix-2
styles-secondaires-mais-maison
atproto-scopes
itineraire-printemps-2026
modal-sheet-5.6
nlnet-cost-climate-perf
pb-icones-religieux
couvoiture
etiennejr-patch-2
meteo-fainéante
logo-static-url
poc-page-vitrine
remove-compass
test-reproduce-1795
display-cycleway-lanes-2
poc-image
add-media-condition
mapterhorn
test-indoor-equal
outdoors-VS-indoorEqual
hiking-colors
fix-blog-typing
poc-next-international
fix-map-style-modal-not-closing
i18n
hide-odd-zoom-levels
filter_categories
resume_wikipedia
deplacement_bouttons
fix-1095
mobile_permettre_selection
position-carte-initiale
dégradé-icônes
better_wikipedia
affichage_donnees_wikidata
mobile-permettre-selection
avis-details
perf-seo
modal-sheet-5
plans-novembre
display-cycleway-lanes
fix-1227
fix-1234
etiennejr-patch-1
update-1240
fix-1236
fix-1131
photon-search-street
fix-1221
article-avis
net-haut-zoom
baies-moins-visibles
belles-fiches-lieu
fix-1157
bug-page-vie-privée
use-watername
fix-1176
carte-du-onde
fix-1166
page-news
tchou
add-icon-alias
vitesse
photon-opensearch
private-in-italic
bâtiments-3D
merge-categories
handle-private-highway
improve-findcategory
injection-pmtiles
fix-1128
check-icons
update-readme
category-rugby
bug-position-carte-vs-lieu
limites-vitesse
improve-find-category-from-tags-v2
integrer
document-osmfeature
translate-document
velo-fin-aout-25
velo-ete-2025
moins-de-maptiler
planet-pmtiles-tilemaker
fix-1048
panoramax-leger
plume
horaires-i18n
planet
maj-deps-juillet-2025
improve-find-category-from-tags
bug-encart
interactions-encart-mobile
clic-on-station
fix-1028
fix-1014
edit
add-big-rivers
add-shortbread-tiles
handle-deleted-osm-elements-v2
codeberg
improve-useEffect
immoloc
update-dashboard
analyze
terradraw
sombre2
calendar
poi-sprites-using-svg
style-sombre
sprites
annuaire
motis-gtfs-attributes
next-styled-components
dev
cool2
cool
panoramax-angle
env-static-urls
borders-only
panoramax2
intelligence
debug-iles
nouveau-serveur
france-protomaps
turbo
fouss/cartes-#338
blog2
legislatives
presentation
pas-de-geoloc-si-hash-different
protomaps
vaul
carte-trains
citiz
maj-paquets
icones
ui2
bblog
voyagevoyage
test-image-origine
bugimagefin
searchParams
voyage
avion
perf
studio
piscine
journée
année
journée-transition
score-dès-acceuil
fix-space-problem
rebase-5août
correspondance
règles-séparées
indeps-acre
demo
indeps
indeps-cachés
indication-temps-plein
indeps-refactor-simulations
indeps-vue-comparaison
indeps-inversion
barème-continu
indeps-démo
textes-branches-sécu
engine-enum
2019
transport
publicodes
as
infrance-nouvelle-landing
better-translation
co2
simplifATMP
controle-SMIC
webi18n
bug-avantages-en-nature
revert-352-infrance-navigation-v2
laem-patch-1
robots.txt
improve-perf
iframe-bugfix
quickfixes
lib
irsimple
hiringinfrance.com
dégradé
refstate
johan
zrr
logo-interactif
revert-216-improve-tracking
redesign2
zrr2
fixtrad
correction-salaire-brut
petit-state
alien-consolidate
translate-ui
npm
alien-brut-net
translatable
différent
ir18
fix-typo
add-tracking
fix-collect-missing
défauts-partout
fix-double-unfold-three
fix-double-unfold
versement-transport
blocker-questions
v3
v2.1
v2
v1
v0.6.4
v0.6.3
v0.6.2
v0.6.1
v0.6.0
0.7.0
v0.5.1
v0.5.0
v0.4.2
v0.4.1
v0.4.0
v0.3.3
v0.3.2
v0.3.1
v0.3
v0.2
v0.1-cdd-alpha
v0-démo-tag-nav
Labels
Clear labels
🎨 Design
À propos de l'interface et de l'UX
💡 Idée
Nouvelle feature à intégrer
question
Further information is requested
🌍 OSM
À propos de l'affichage et l'édition des éléments OSM
🌱 Amélioration
Amélioration/fix de l'existant
🐞 Bug
Something isn't working
📊 SEO
📋 Catégories
À propos des catégories de recherche et de l'annuaire
📖 Wiki
📝Avis
A propos de la fonction avis et de bluesky
📢 Oyez
📷 Photos
A propos des photos (sources, affichage)
🔍Recherche
A propos de la barre de recherche et de photon
🔥 Actuel
🔧 tech
🖌 Styles
Styles de la carte
🧝i18n
🧪 R&D
🚍 Transports
🚙 Voiture
🚨Prioritaire
Ticket à traiter rapidement
🚲Vélo
Cartes
climat
Communication
demo
Cette branche doit être déployée comme une démo
dependencies
Pull requests that update a dependency file
javascript
Pull requests that update javascript code
Pas urgent
Plein-air
Tuiles
Verso
Vieux
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
cartes/web#1230
Reference in a new issue
cartes/web
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?