10
105
Fork
You've already forked web
30

les éléments OSM supprimés provoquent des crashs #954

Open
opened 2025年05月21日 23:20:15 +02:00 by etienneJr · 7 comments

cet élément OSM a été supprimé hier : https://www.openstreetmap.org/node/3152633780
Il s'agit de ce bar à Toulouse :
https://cartes.app/#21/43.598237/1.4326015

Image

Comme Overpass est à jour instantanément mais pas les tuiles, on le voit sur le fond de carte, mais si on clique dessus, on se retrouve avec un chargement infini :

Image

C'est encore pire si on arrive par un lien de partage : ça fait un écran bleu
https://cartes.app/?allez=bar+supprimé|n3152633780

Il faudrait plutôt attrapper l'erreur et afficher lieu introuvable dans OSM, il est probable qu'il n'existe plus

cet élément OSM a été supprimé hier : https://www.openstreetmap.org/node/3152633780 Il s'agit de ce bar à Toulouse : https://cartes.app/#21/43.598237/1.4326015 ![Image](https://github.com/user-attachments/assets/e0e6b1ee-f687-4ed0-bb2c-9689b3d88314) Comme Overpass est à jour instantanément mais pas les tuiles, on le voit sur le fond de carte, mais si on clique dessus, on se retrouve avec un chargement infini : ![Image](https://github.com/user-attachments/assets/7fde647c-df03-4349-9e55-c5baf424daa5) C'est encore pire si on arrive par un lien de partage : ça fait un écran bleu https://cartes.app/?allez=bar+supprimé|n3152633780 Il faudrait plutôt attrapper l'erreur et afficher `lieu introuvable dans OSM, il est probable qu'il n'existe plus`
Contributor
Copy link

J'ai eu la même erreur sur un noeud qui a été transformé en polygone (https://www.openstreetmap.org/way/71349481). Pour ce cas là, est-ce que ça serait possible de chercher si le noeud qui n'existe plus est englobé dans un polygone avec les mêmes attributs ?

J'ai eu la même erreur sur un noeud qui a été transformé en polygone (https://www.openstreetmap.org/way/71349481). Pour ce cas là, est-ce que ça serait possible de chercher si le noeud qui n'existe plus est englobé dans un polygone avec les mêmes attributs ?
Author
Member
Copy link

Bizarre que le bug persiste alors que les tuiles ont été recalculées depuis ta modification. Le noeud ne devrait pas être présent dans les tuiles. Ça voudrait dire que les tuiles ont été calculées à partir d'un planet.osm de plus de 8j ?!? @laem

Et non, comme le noeud a été supprimé, on ne peut pas connaître ses coordonnées ni ses tags en interrogeant l'instance overpass d'osm fr vu qu'elle n'a pas l'historique.

On peut trouver les coordonnées vu l'endroit du clic mais ça ne gère pas le cas où on arrive par un lien périmé.

Bizarre que le bug persiste alors que les tuiles ont été recalculées depuis ta modification. Le noeud ne devrait pas être présent dans les tuiles. Ça voudrait dire que les tuiles ont été calculées à partir d'un planet.osm de plus de 8j ?!? @laem Et non, comme le noeud a été supprimé, on ne peut pas connaître ses coordonnées ni ses tags en interrogeant l'instance overpass d'osm fr vu qu'elle n'a pas l'historique. On peut trouver les coordonnées vu l'endroit du clic mais ça ne gère pas le cas où on arrive par un lien périmé.
Contributor
Copy link

@etienneJr wrote in #954 (comment):

Bizarre que le bug persiste alors que les tuiles ont été recalculées depuis ta modification. Le noeud ne devrait pas être présent dans les tuiles. Ça voudrait dire que les tuiles ont été calculées à partir d'un planet.osm de plus de 8j ?!? @laem

Oui, j'ai aussi regardé d'autres lieux que j'ai modifié récemment et les tuiles n'ont pas été mises à jour depuis un moment.

@etienneJr wrote in https://codeberg.org/cartes/web/issues/954#issuecomment-7394437: > Bizarre que le bug persiste alors que les tuiles ont été recalculées depuis ta modification. Le noeud ne devrait pas être présent dans les tuiles. Ça voudrait dire que les tuiles ont été calculées à partir d'un planet.osm de plus de 8j ?!? @laem Oui, j'ai aussi regardé d'autres lieux que j'ai modifié récemment et les tuiles n'ont pas été mises à jour depuis un moment.
Author
Member
Copy link

@esmenard wrote in #954 (comment):

les tuiles n'ont pas été mises à jour depuis un moment.

Pourtant si, elles ont été recalculées il y a 2 jours pour intégrer la 3D complète des batiments, cf ce message. C'est pour ça que mon hypothèse est qu'elles aient été calculées il y a 2 jours à partir de données OSM plus anciennes. Comme si le fichier source téléchargé chez geofabrik n'était pas à jour.

@esmenard wrote in https://codeberg.org/cartes/web/issues/954#issuecomment-7396198: > les tuiles n'ont pas été mises à jour depuis un moment. Pourtant si, elles ont été recalculées il y a 2 jours pour intégrer la 3D complète des batiments, cf [ce message](https://forum.openstreetmap.fr/t/discussions-autour-des-fonds-carte-de-cartes-app/28339/44). C'est pour ça que mon hypothèse est qu'elles aient été calculées il y a 2 jours à partir de données OSM plus anciennes. Comme si le fichier source téléchargé chez geofabrik n'était pas à jour.
Author
Member
Copy link

Je me suis renseigné sur le wiki : le fichier planet.osm n'est mis à jour que 1 fois par semaine. Le calcul commence le lundi et se termine dans la nuit du vendredi au samedi (c'est variable, mais globalement ça se décale petit à petit au fur et à mesure que la base grossit).

Donc voici ce qu'il s'est passé :

  • @laem a lancé le calcul des tuiles jeudi 25/09
  • à partir du fichier planet.osm publié le 20/09
  • qui avait commencé à être calculé le 15/09

C'est donc vraiment dommage mais "normal" que ta modification du 18/09 n'apparaisse pas dans les tuiles.

Le mieux qu'on puisse faire, c'est lancer le recalcul des tuiles juste après la publication du planet.osm (par ex le samedi après midi), on aura alors :

  • 2 à 6 jours de retard le dimanche juste après le calcul des tuiles
  • 8 à 12 jours de retard le vendredi suivant, juste avant le recalcul

Si dans le futur on veut des mises à jour plus rapides, on n'aura pas d'autre choix que de passer à tilekiln. Ca aura du sens quand le contenu des tuiles sera figé, car, en contrepartie de la mise à jour minute, tilekiln est beaucoup moins souple que tilemaker pour modifier le contenu des tuiles.

Je me suis renseigné sur [le wiki](https://wiki.openstreetmap.org/wiki/Planet.osm#Update_frequency) : le fichier planet.osm n'est mis à jour que 1 fois par semaine. Le calcul commence le lundi et se termine dans la nuit du vendredi au samedi (c'est variable, mais globalement ça se décale petit à petit au fur et à mesure que la base grossit). Donc voici ce qu'il s'est passé : - @laem a lancé le calcul des tuiles jeudi 25/09 - à partir du fichier planet.osm publié le 20/09 - qui avait commencé à être calculé le 15/09 C'est donc vraiment dommage mais "normal" que ta modification du 18/09 n'apparaisse pas dans les tuiles. Le mieux qu'on puisse faire, c'est lancer le recalcul des tuiles juste après la publication du planet.osm (par ex le samedi après midi), on aura alors : - 2 à 6 jours de retard le dimanche juste après le calcul des tuiles - 8 à 12 jours de retard le vendredi suivant, juste avant le recalcul Si dans le futur on veut des mises à jour plus rapides, on n'aura pas d'autre choix que de passer à [tilekiln](https://github.com/pnorman/tilekiln). Ca aura du sens quand le contenu des tuiles sera figé, car, en contrepartie de la mise à jour minute, tilekiln est beaucoup moins souple que tilemaker pour modifier le contenu des tuiles.
Owner
Copy link

Merci beaucoup pour ton analyse Étienne c'est très clair. Je savais qu'il y avait un délai, mais pas autant !

Pour cartes.app, il faut garder en tête qu'on a la chance d'avoir l'instance overpass. Seul la création, suppression, déplacement ou changement de type principal pose problème, les attributs eux étant récupérés via ovrrpass, à jour.

Elle nous permet d'avoir des lieux à jour lors des recherches par catégorie, mais elle nous permettrait aussi d'extraire une couche à jour des lieux nouvellement crées chaque jour. Pour qu'un commerce ne soit pas frustré de ne pas le voir sur le fond de carte. Mais ça demande du dev.

À noter que le moteur de recherche Photon a le même type de délais.

Merci beaucoup pour ton analyse Étienne c'est très clair. Je savais qu'il y avait un délai, mais pas autant ! Pour cartes.app, il faut garder en tête qu'on a la chance d'avoir l'instance overpass. Seul la création, suppression, déplacement ou changement de type principal pose problème, les attributs eux étant récupérés via ovrrpass, à jour. Elle nous permet d'avoir des lieux à jour lors des recherches par catégorie, mais elle nous permettrait aussi d'extraire une couche à jour des lieux nouvellement crées chaque jour. Pour qu'un commerce ne soit pas frustré de ne pas le voir sur le fond de carte. Mais ça demande du dev. À noter que le moteur de recherche Photon a le même type de délais.
Owner
Copy link

Note que sur Google Maps, l'ajout de lieux prend du temps également. Le délai d'une semaine me semble problématique, mais si on arrivait à faire mieux que 24h, on serait SOTA.

Il faudrait qu'on regarde si un planet journalier n'est pas trouvable quelque part quand même...

Édit : il y a des diffs ! Faudrait voir le temps que ça prend de patcher un planet.osb.pbf tous les n jours avant de relancer le calcul des tuiles. https://wiki.openstreetmap.org/wiki/Planet.osm/diffs

En cela c'est une prouesse le côté instantané de OSM. Juste dommage que ce soit si peu grand public.

Note que sur Google Maps, l'ajout de lieux prend du temps également. Le délai d'une semaine me semble problématique, mais si on arrivait à faire mieux que 24h, on serait SOTA. Il faudrait qu'on regarde si un planet journalier n'est pas trouvable quelque part quand même... Édit : il y a des diffs ! Faudrait voir le temps que ça prend de patcher un planet.osb.pbf tous les n jours avant de relancer le calcul des tuiles. https://wiki.openstreetmap.org/wiki/Planet.osm/diffs En cela c'est une prouesse le côté instantané de OSM. Juste dommage que ce soit si peu grand public.
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
3 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#954
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?