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

loosen join expressions on v.voie to account for TOPO not having ccovoi #743

Merged
merged 1 commit into from
Jul 31, 2024

Conversation

landryb
Copy link
Member

@landryb landryb commented Jul 23, 2024

should help with #742, and backwards compatible

@landryb
Copy link
Member Author

landryb commented Jul 23, 2024

j'ai testé cette PR en important les données de la commune 63001 dans le schéma qadastre via le plugin qgis, et crée les vues avec les scripts du repo cadastrapp:

dans TOPO/FANTOIR, il y'a 3 nouvelles voies créees en 2024 pour cette commune:

    6300010194BRUE DE LA CHAUSSADE
    6300010195CALL DE LA CHAUSSADE
    6300010200HIMP DE LA CROIX PREMIERE

et j'ai bien des parcelles qui se retrouvent dans ces nouvelles voies:

[localhost:5432] georchestra@georchestra=> 
    select parcelle,natvoi,voie,dvoilib from cadastrapp.proprietebatie  where
        dvoilib in ('LA CROIX PREMIERE','DE LA CHAUSSADE');
    parcelle     | natvoi |      voie       |      dvoilib      
-----------------+--------+-----------------+-------------------
 630001000AD0517 |        | 63000100000B022 | LA CROIX PREMIERE
 630001000AD0536 | ALL    | 630001000000195 | DE LA CHAUSSADE
 630001000AE0084 |        | 63000100000B022 | LA CROIX PREMIERE
 630001000AE0084 |        | 63000100000B022 | LA CROIX PREMIERE
 630001000ZI0178 |        | 63000100000B022 | LA CROIX PREMIERE
 630001000ZI0200 |        | 63000100000B022 | LA CROIX PREMIERE

@landryb
Copy link
Member Author

landryb commented Jul 29, 2024

@MaelREBOUX on merge ?

@landryb
Copy link
Member Author

landryb commented Jul 30, 2024

je viens de generer les mv sur mes 12 départements, et la jointure se passe bien, on ramene bien les dvoilib sur les enregistrements de proprietebatie et proprietenonbatie, on a environ le meme nombre de records ou la jointure réussit:

[localhost:5432] qadastre@qadastre=> select count(*) from cad2023.proprietebatie ;
 11640775
[localhost:5432] qadastre@qadastre=> select count(*) from cad2024.proprietebatie ;
 11758084
[localhost:5432] qadastre@qadastre=> select count(*) from cad2023.proprietebatie where voie is not null;
 10025116
[localhost:5432] qadastre@qadastre=> select count(*) from cad2024.proprietebatie where voie is not null;
 10134874
[localhost:5432] qadastre@qadastre=> select count(*) from cad2023.proprietenonbatie ;
 16215655
[localhost:5432] qadastre@qadastre=> select count(*) from cad2024.proprietenonbatie ;
 16274531

@MaelREBOUX
Copy link
Member

ok pour moi pour le merge.

Il faudrait un milestone ? @pierrejego ?

@MaelREBOUX MaelREBOUX added this to the v 2.1 milestone Jul 30, 2024
@landryb landryb merged commit 85304fb into georchestra:master Jul 31, 2024
1 of 2 checks passed
@landryb landryb linked an issue Jul 31, 2024 that may be closed by this pull request
@landryb landryb modified the milestones: v 2.1, v 2.2 Aug 5, 2024
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.

adaptations nécessaires pour TOPO ?
2 participants