Skip to content

Conversation

@etienneJr
Copy link

@etienneJr etienneJr commented Jun 14, 2025

Cette PR participe à la résolution de cartesapp/cartes#968 en gérant l'inclusion des fleuves dans les tuiles de faible zoom (il faudra aussi modifier france.ts dans \cartes)

Les rivières sont en triple dans les tuiles :

  • l'emprise du lit en Polygon dans le layer water
  • le nom en Point et LineString dans le layer water_name
  • le tracé en LineString dans le layer waterway

Donc pas besoin d'étendre des Polygon de water pour les voir à faible zoom, on pourra utiliser les LineString de waterway sauf que pas de bol ils ne sont présents que dans les tuiles de zoom >= 8.

Donc pour inclure les fleuves dans les tuiles, j'ai :

  • créé une liste de nom de fleuves qu'on veut afficher
    • pas hyper satisfaisant comme méthode, mais il n'y a pas de tag spécifique pour les distinguer
  • ajouté un test isBigRiver qui permet de les ajouter dans un layer provisoire waterway_big
  • et ce layer provisoire est renvoyé dans les tuiles de zoom 3 à 7 inclus du layer waterway

@laem je n'ai pas pu tester sur la france entière, seulement sur les différents extraits .osm.pbf de régions qui m'ont permis de mettre au point le script. Ca marche bien, je suis assez confiant pour le cas France augmentée (d'autant que ça ne fait que ajouter des données en plus dans des tuiles, et tout l'existant est conservé à l'identique).

Exemple : 1 morceau de la Seine et 2 morceaux de la Meuse qui passent en Champagne-Ardennes, à zoom 6
image

@Athozus
Copy link

Athozus commented Jun 15, 2025

Ça m'a l'air super 👍

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.

2 participants