Skip to content

Conversation

philippecarle
Copy link
Contributor

2ème PR avec les cantons. J'espère que le dépôt est toujours "vivant", les dates de derniers commits étant assez anciennes… !

@philippecarle
Copy link
Contributor Author

La visualisation de Github me permet de constater deux trous (j'ai un peu travaillé à l'aveugle sur ces cartos) : Lyon et Paris… Pourquoi, comment, aucune idée PR à mettre en attente et à corriger ?

@philippecarle philippecarle mentioned this pull request Aug 15, 2016
@jmleroux
Copy link
Contributor

jmleroux commented Sep 15, 2016

J'ai un peu bossé sur le sujet et ces deux "trous" sont récurrents.
Il semblerait que ces deux zones ne sont plus des cantons.
A ce jour, je n'ai pas trouvé la parade.

@jmleroux
Copy link
Contributor

jmleroux commented Sep 15, 2016

Cependant, il semble que ce dépot soit plus ou moins abandonné.
Je vais m'intéresser à votre dépôt https://github.com/populin/france-geojson.

Vous pourriez peut-être demander le transfert d'ownership.

@jmleroux
Copy link
Contributor

@gregoiredavid a redonné signe de vie. Super cool, merci 👍

@gregoiredavid
Copy link
Owner

Merci @philippecarle. Du point de vue administratif il n'y a pas de cantons à Paris et Lyon, donc à priori c'est normal. C'est un peu comme le département parisien qui est fusionné avec la ville (sans en avoir toutes les compétences) ou de Lyon qui a récupéré récemment toutes les prérogatives du département sur son territoire.

C'est un peu un casse-tête à cartographier, je suis d'avis de faire du cas par cas. Pour les départements ça n'a pas de sens de laisser un trou à l'emplacement de Paris et de Lyon, par contre pour les cantons ce serait trompeur d'ajouter des cantons qui n'existent pas pour combler les trous je pense.

Je merge tel quel parce qu'il n'y a pas de point bloquant pour moi, tout est OK. Pour info, je prévois simplement d'uniformiser les méta-données (clé "properties") et de créer des fichiers individuels par région et département dans la prochaine release (2.x).

@gregoiredavid gregoiredavid merged commit 1eb7cbd into gregoiredavid:master Sep 23, 2016
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