3liz / QgisCadastrePlugin

A QGIS plugin which helps users to import the french land registry ('cadastre') data into a database. It is meant to ease the use of the data in QGIS by providing search tools and appropriate layer symbology.
GNU General Public License v2.0
60 stars 41 forks source link

Rajouter un code de commune sur les tables geo_ #115

Open pnropf opened 6 years ago

pnropf commented 6 years ago

bonjour, souhaitant associer à chacun de mes bâtiments le code insee de sa commune de rattachement, je pensais y arriver en utilisant la table geo_batiment_parcelle qui laisse supposer que chaque batiment est associé à sa parcelle... toutefois quelques tests rapide montre que le champs identifiant geo_batiment n'est pas unique, et peut être relié à plusieurs bâtiments différents (de communes différentes). est-ce normal ? est-ce possible d'envisager un identifiant vraiment unique qui corresponde à un seul objet géographique ? (je sais que c'est pas évident car les fichiers fiscaux sont construits avec des identifiants communaux)

table_geo_batiment_parcelle

mdouchin commented 6 years ago

J'ai normalement repensé les identifiants pour la prochaine version du cadastre. Je vais vérifier si j'ai bien ajouté le code département et direction dans ces tables

pnropf commented 6 years ago

Super ! merci pour votre réactivité…

j’ai vu les messages pour la version « master » en cours : si j’ai bien compris, pour intégrer les derniers fichiers MAJIC en train d’être publié par les services fiscaux (j’ai reçu les premiers aujourd’hui), il faut forcement installer la version « master » ? ou est-ce que l’on peut se « contenter » de la version officielle actuelle pour intégrer les nouveaux fichiers ?

cordialement,

Pascal LAMBERT Chargé de mission SIG/EVALUATION Tel. : 03 44 63 65 62

Parc naturel régional Oise - Pays de France Château de la Borne Blanche 48, rue d'Hérivaux- B.P 6 60560 ORRY-LA-VILLE Tel. : 03 44 63 65 65 Fax : 03 44 63 65 60 http://www.parc-oise-paysdefrance.frhttp://www.parc-oise-paysdefrance.fr/ visitez aussi notre WEBSIG : http://sig.parc-oise-paysdefrance.frhttp://sig.parc-oise-paysdefrance.fr/ [cid:image001.jpg@01CDDD4A.3231DF20] Retrouvez notre actualité sur Facebookhttp://www.facebook.com/platform#!/pages/Parc-naturel-r%C3%A9gional-Oise-Pays-de-France/290114747711775?sk=wall

De : mdouchin [mailto:notifications@github.com] Envoyé : mardi 10 octobre 2017 15:37 À : 3liz/QgisCadastrePlugin Cc : Pascal LAMBERT; Author Objet : Re: [3liz/QgisCadastrePlugin] la table geo_batiment_parcelle n'est pas fiable du tout en cas d'import multicommunale (#115)

J'ai normalement repensé les identifiants pour la prochaine version du cadastre. Je vais vérifier si j'ai bien ajouté le code département et direction dans ces tables

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/3liz/QgisCadastrePlugin/issues/115#issuecomment-335474828, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AfKUzBxT2KLi3dPQuDhoVMmiBAtUp4z_ks5sq3MRgaJpZM4Pz78R.

pnropf commented 6 years ago

Pour fiabilisé complétement l’identifiant, c’est le code insee complet de la commune qu’il faut concatener à l’identifiant « communal » (la direction n’est pas assez discriminante je pense).

Cordialement,

Pascal LAMBERT Chargé de mission SIG/EVALUATION Tel. : 03 44 63 65 62

Parc naturel régional Oise - Pays de France Château de la Borne Blanche 48, rue d'Hérivaux- B.P 6 60560 ORRY-LA-VILLE Tel. : 03 44 63 65 65 Fax : 03 44 63 65 60 http://www.parc-oise-paysdefrance.frhttp://www.parc-oise-paysdefrance.fr/ visitez aussi notre WEBSIG : http://sig.parc-oise-paysdefrance.frhttp://sig.parc-oise-paysdefrance.fr/ [cid:image001.jpg@01CDDD4A.3231DF20] Retrouvez notre actualité sur Facebookhttp://www.facebook.com/platform#!/pages/Parc-naturel-r%C3%A9gional-Oise-Pays-de-France/290114747711775?sk=wall

De : mdouchin [mailto:notifications@github.com] Envoyé : mardi 10 octobre 2017 15:37 À : 3liz/QgisCadastrePlugin Cc : Pascal LAMBERT; Author Objet : Re: [3liz/QgisCadastrePlugin] la table geo_batiment_parcelle n'est pas fiable du tout en cas d'import multicommunale (#115)

J'ai normalement repensé les identifiants pour la prochaine version du cadastre. Je vais vérifier si j'ai bien ajouté le code département et direction dans ces tables

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/3liz/QgisCadastrePlugin/issues/115#issuecomment-335474828, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AfKUzBxT2KLi3dPQuDhoVMmiBAtUp4z_ks5sq3MRgaJpZM4Pz78R.

jdesboeufs commented 6 years ago

Il suffit d'ajouter les deux premiers caractères du code département (97 dans le cas des DROM). Cela permet de reconstruire le code INSEE commune en préfixe, ce qui suffit pour l'unicité.

pnropf commented 6 years ago

Non, cela ne suffit pas !

Voici un exemple : le bâtiment dont l’id est 20166000003521 correspond à plusieurs bâtiments situés sur plusieurs communes du même département (ou pas)… le problème est lié au fait que rien ne relie l’identifiant du bâtiment à sa commune (aucun lien avec une portion du code insee communal). Mon propos étant d’associer à chaque bâtiment ses caractéristiques d’habitat situé dans différentes tables sans géométrie (surface totale, etc… ), je suis coincé par cet identifiant non unique qui ne me permet pas d’associer les informations au bon bâtiment… [https://user-images.githubusercontent.com/32675020/31388946-e34bfce0-adcf-11e7-826f-b8bf444291bf.jpg]

Pascal LAMBERT Chargé de mission SIG/EVALUATION Tel. : 03 44 63 65 62

Parc naturel régional Oise - Pays de France Château de la Borne Blanche 48, rue d'Hérivaux- B.P 6 60560 ORRY-LA-VILLE Tel. : 03 44 63 65 65 Fax : 03 44 63 65 60 http://www.parc-oise-paysdefrance.frhttp://www.parc-oise-paysdefrance.fr/ visitez aussi notre WEBSIG : http://sig.parc-oise-paysdefrance.frhttp://sig.parc-oise-paysdefrance.fr/ [cid:image001.jpg@01CDDD4A.3231DF20] Retrouvez notre actualité sur Facebookhttp://www.facebook.com/platform#!/pages/Parc-naturel-r%C3%A9gional-Oise-Pays-de-France/290114747711775?sk=wall

De : Jérôme Desboeufs [mailto:notifications@github.com] Envoyé : mardi 24 octobre 2017 15:46 À : 3liz/QgisCadastrePlugin Cc : Pascal LAMBERT; Author Objet : Re: [3liz/QgisCadastrePlugin] la table geo_batiment_parcelle n'est pas fiable du tout en cas d'import multicommunale (#115)

Il suffit d'ajouter les deux premiers caractères du code département (97 dans le cas des DROM). Cela permet de reconstruire le code INSEE commune en préfixe, ce qui suffit pour l'unicité.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/3liz/QgisCadastrePlugin/issues/115#issuecomment-338995247, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AfKUzKCmjnDtScFn7pkOUX0wHorR3JS8ks5sveoZgaJpZM4Pz78R.

mdouchin commented 6 years ago

J'ai proposé une correction dans la dernière version. Je réalise une intersection en plus comme critère de jointure. cf 352b752

pnropf commented 6 years ago

Bonjour,

Je pense que cela ne suffira pas à régler le problème ! en effet le soucis vient du fait que geo_batiment n’est pas unique : plusieurs bâtiments de communes différentes possèdent le même identifiant… sauf erreur de ma part, votre code ne modifie pas cet état de fait et il ne sera toujours pas possible d’associer un bâtiment bien déterminé à sa parcelle.

Il faut que l’identifiant geo_batiment soit unique à chaque batiment de la base : il faudrait associer le code insee communal à l’identifiant actuel des batiments pour obtenir un identifiant « robuste » (a priori sur une même commune, il n’y a pas 2 bâtiments avec le même code) à la fois dans la table geo_batiment et la table geo_batiment_parcelle

[cid:image002.jpg@01D35D62.2E873F30][cid:image003.jpg@01D35D62.AF1C9320]

Cordialement, Pascal LAMBERT Chargé de mission SIG/EVALUATION Tel. : 03 44 63 65 62

Parc naturel régional Oise - Pays de France Château de la Borne Blanche 48, rue d'Hérivaux- B.P 6 60560 ORRY-LA-VILLE Tel. : 03 44 63 65 65 Fax : 03 44 63 65 60 http://www.parc-oise-paysdefrance.frhttp://www.parc-oise-paysdefrance.fr/ visitez aussi notre WEBSIG : http://sig.parc-oise-paysdefrance.frhttp://sig.parc-oise-paysdefrance.fr/ [cid:image001.jpg@01CDDD4A.3231DF20] Retrouvez notre actualité sur Facebookhttp://www.facebook.com/platform#!/pages/Parc-naturel-r%C3%A9gional-Oise-Pays-de-France/290114747711775?sk=wall

De : mdouchin [mailto:notifications@github.com] Envoyé : mardi 14 novembre 2017 15:27 À : 3liz/QgisCadastrePlugin Cc : Pascal LAMBERT; Author Objet : Re: [3liz/QgisCadastrePlugin] la table geo_batiment_parcelle n'est pas fiable du tout en cas d'import multicommunale (#115)

J'ai proposé une correction dans la dernière version. Je réalise une intersection en plus comme critère de jointure. cf 352b752https://github.com/3liz/QgisCadastrePlugin/commit/352b75284fe48d00dca855cda9236cc48f7966c7

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/3liz/QgisCadastrePlugin/issues/115#issuecomment-344275039, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AfKUzMG-prn5RlyLlcYrV-W2UvhnxzfOks5s2aNDgaJpZM4Pz78R.

mdouchin commented 6 years ago

@pnropf Merci de tester . L'intersection réalisée via https://github.com/3liz/QgisCadastrePlugin/commit/352b75284fe48d00dca855cda9236cc48f7966c7#diff-b5c1e96fdcd57d42db78e9d468839d97R151 permet bien de régler le problème, en remplissant la table geo_parcelle_batiment avec un critère de plus.

Il faudrait peut-être affiner en utilisant l'intersection entre le centroide du batiment et la parcelle, pour éviter des effets de frontière.

NB: il vaut mieux éviter de répondre via son application de messagerie à un email de Github, car cela intègre tout le texte dans la demande sur le portail Github (cf page : https://github.com/3liz/QgisCadastrePlugin/issues/115 )

pnropf commented 6 years ago

@mdouchin je ne peux pas tester le résultat de la modification proposée, car la table edigeo_rel est vide chez moi : il m'est donc difficile de juger de l'efficacité du code proposé. Concernant l'intersection, il faut effectivement utiliser le centroïde car les bâtiments sont fréquemment en limite des communes, voire même à cheval sur plusieurs communes. l'intersection va effectivement logiquement réduire considérablement le nombre de ligne puisque on va se limiter aux objets qui s'intersectent geographiquement, mais si l'identifiant des bâtiments n'est pas unique je ne vois pas trop comment on pourra associer la géométrie des bâtiments (située dans la table geo_batiment) avec l'identifiant de sa parcelle (située dans la table geo_batiment_parcelle) ou les autres informations intéressantes des autres tables. geo_batiment geo_batiment_parcelle

mdouchin commented 6 years ago

Merci de vérifier avec la nouvelle version 1.5.0 , en relançant un nouvel import.

MaelREBOUX commented 4 years ago

Bonjour,

Nous avons noté en point en "irritant" dans notre utilisation du modèle de données QGIS l'absence d'un code de commune sur la table geo_parcelle. Afin de pouvoir faire des sélections plus rapide de données il serait en effet utile d'avoir un attribut supplémentaire ccocom sur la table geo_batiment

La table geo_batiment est alimentée ici : https://github.com/3liz/QgisCadastrePlugin/blob/master/scripts/plugin/edigeo_formatage_donnees.sql#L149-L152

et la table geo_batiment _parcelle (que je découvre) ici : https://github.com/3liz/QgisCadastrePlugin/blob/master/scripts/plugin/edigeo_formatage_donnees.sql#L161-L166

MaelREBOUX commented 4 years ago

D'une manière plus générale : il faudrait rajouter un attribut ccocom sur toutes les tables geo_*

Des utilisateurs nous demande également de rajouter un attribut inseecom sur ces mêmes tables géo pour simplifier des exploitations.

MaelREBOUX commented 4 years ago

Pour suivi : on est en train de tester cette évolution en interne.

L'attribut s'appellera geo_commune.

MaelREBOUX commented 3 years ago

@EtienneRouvin : à traiter.