Conflit entre leisure et sport dans la génération des tuiles

alexandreG

J'ai remarqué que certains POI de type "Sport" ne s'affichaient pas sur la carte, notamment les Piscines, Patinoires, Gymnases.

1. laem

   Ah, peut-être un problème d'icône ? Notre gestion des icône est capricieuse.

2. etienneJr

   Ou alors une incohérence entre le bassin et le bâtiment ? On a un autre ticket ouvert à propos des piscines, mais côté catégories, le #2509

3. gregori

   J'ai noté la même chose, un "Complexe sportif" (centre sportif, Tag:leisure=sports_centre, sport=squash;badminton;table_tennis;10pin;billiards;darts;fitness ) ne s'affiche pas sous forme d'icône (https://cartes.app/?allez=Van+der+Merve%7Cw292639410%7C7.54712%7C47.55955#17.5/47.558887/7.548341). Par contre une piscine (Tag:leisure=sports_centre, sport=swimming) qui ne s'affichait pas quand alexandreG a ouvert le ticket, a l'air réparé (https://cartes.app/?allez=Gartenbad+Bachgraben%7Cw185791077%7C7.55676%7C47.56197#16.85/47.56172/7.557161). https://cartes.app/?allez=Van+der+Merve%7Cw292639410%7C7.54712%7C47.55955#17.5/47.558887/7.548341

4. alexandreG

   J'ai enquêté en comparant plusieurs piscines :

   Lien Cartes	Données OSM	Rendu Cartes	Données Tuiles
   Piscine Léo-Lagrange	leisure=sports_centre + sport=swimming
   Piscine de la Petite Amazonie	leisure=sports_centre + sport=swimming
   Piscine Jules Verne	leisure=sports_centre + sport=swimming
   Piscine du Petit Port	leisure=sports_centre + sport=swimming

   Bien qu'elles aient toutes les mêmes tags OSM, on voit que les class et subclass attribuées dans les tuiles ne sont pas les mêmes.

   Il semblerait que le script de génération des tuiles choisisse de privilégier soit leisure soit sport de manière un peu aléatoire. C'est probablement ce qui fait que @gregori a remarqué que le problème s'était résolu tout seul sur une piscine. Potentiellement le problème pourrait revenir lors de la prochaine génération des tuiles.

   Je n'ai pas tout compris au fonctionnement des tuiles mais il semble que cette PR s'adresse à ce problème : https://github.com/systemed/tilemaker/pull/889

5. alexandreG

   Potentiellement le problème se pose pour tous les POI qui combinent leisure=sports_centre + sport=*

6. etienneJr

   @alexandreG wrote in https://codeberg.org/cartes/web/issues/2986#issuecomment-23940399:

   cette PR s'adresse à ce problème : https://github.com/systemed/tilemaker/pull/889

   bien vu, j'ai relancé la PR sur github en disant qu'elle nous intéresse.

7. alexandreG

   La génération des tuiles de cartes.app est basée sur un process-openmaptiles.lua personnalisé c'est bien ça ? Tu attends quand même que ce type de modification soit faite dans le fichier d'origine avant de l'intégrer ?

8. etienneJr

   @alexandreG wrote in https://codeberg.org/cartes/web/issues/2986#issuecomment-24023163:

   La génération des tuiles de cartes.app est basée sur un process-openmaptiles.lua personnalisé c'est bien ça ? Tu attends quand même que ce type de modification soit faite dans le fichier d'origine avant de l'intégrer ?

   aaahhh j'avais même pas regardé la MR en détail 😅 Je pensais que les modifs seraient dans le C++, pas dans le lua. En effet, c'est quasi rien, on pourrait l'appliquer pour le tester lors du prochain calcul des tuiles.

9. alexandreG

   Une autre solution a été proposée dans cette PR : https://github.com/systemed/tilemaker/pull/942 Je ne sais pas si elle est mieux que celle-ci : https://github.com/systemed/tilemaker/pull/889 A tester ?