Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

renvoyer la tuile de rendu si le point est en bordure, pas seulement celle théorique #316

Merged

Conversation

IGNFBourcier
Copy link
Contributor

@IGNFBourcier IGNFBourcier commented Oct 27, 2023

Context

Lors de la composition des bal un calcul de la tuile (xyz) correspondant aux positions ponctuelles des adresses et voie en vue d'un rendu et service tuilé.
Actuellement le service mvt renvoie bien l'ensemble des adresses présente dans la tuile , mais du fait de l'encodage certaines adresses en bordure sont considérées hors étendu et ne sont pas rendu par des clients (mapbox/maplibre).

il a environ 6k adresses dite fantômes (pour le niveau 14 dernier niveau servi par le serveur)
#324

Cette PR propose d'inclure la tuile de rendu dans la liste des tuiles pour position si le point est en bordure.

Effet

  • affichage des positions où actuellement il y a default,
  • les 2 tuiles renverront la donnée du point en bordure. (un identifiant est déjà présent et permet donc de distinguer les doublons)

Exemple

l'adresse actuelle de 74096_0114_00228 , à la position

  "lon": 6.108398,
    "lat": 46.034484,
    "x": 940355.25,
    "y": 6553034.78,
    "tiles": [
        "12/2117/1456",
        "13/4234/2913",
        "14/8469/5826"
    ],

est en bordure de la tuile 13/4234/2913 et 14/8469/5826.
Sur adresse.data.gouv elle est actuellement invisible, Avec un client openlayer ou QGIS, le point de l'adresse est affiché par moitié de point
carte sous openlayer

avec ce patch et un recalcul, le lookup de l'adresse 228 Grand'rue Cruseilles 74096_0114_00228 aura 2 tuiles en plus 13/4235/2913 et 14/8469/5826 et l'explorer adresse présentera le point 228 graphiquement.

Le recalcul peut s'obtenir par demande de composition de la commune.

@mmortier
Copy link
Contributor

fix #324

@mmortier mmortier merged commit 73d78b6 into master Nov 16, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants